Project

General

Profile

Backport(バックポート) #3075

activity/post.json の画像投稿対応

Added by Shouta Kashiwagi about 7 years ago. Updated almost 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2012-06-01
Due date:
% Done:

100%


Description

概要

activity/post.json の画像投稿対応
現状OpenPNE3に従来からある「アクティビティ」は「ActivityData」テーブルと連携させる為に用意されたと考えられる「ActivityImage」テーブルがあるにも関わらず,
アクティビティの画像添付に対応していないので,API側で優先的に対応させる(タイムラインプラグインの画像投稿対応も視野に入れた想定)

仕様

"images" パラメーターに画像データを載せることにより投稿する

  • 実装予定

+    if ($request->getFiles('images'))
+    {
+      $imageFile = $request->getFiles('images');
+      $obj = new sfValidatedFile($imageFile['name'], $imageFile['type'], $imageFile['tmp_name'], $imageFile['size']);
+      $file = new File();
+      $file->setFromValidatedFile($obj);
+      $file->setName('ac_'.$this->getUser()->getMemberId().'_'.$file->getName());
+      $file->save();
+      $options['images'] = array('file_id' => $file->getId());
+    }
+


Related issues

Related to OpenPNE 3 - Enhancement(機能追加・改善) #3064: activity/post.json の画像投稿対応 Won't fix(対応せず) 2012-06-01
Blocked by OpenPNE 3 - Backport(バックポート) #3397: opBrowser::post() でファイルのアップロードをテストできるようにする Fixed(完了) 2013-09-10

Associated revisions

Revision 6138554a (diff)
Added by Shouta Kashiwagi about 7 years ago

add an implement of image post onto activity/post.json (fixes #3075, BP from #3064)

Revision c2104678 (diff)
Added by Yuya Watanabe about 7 years ago

Revert "add an implement of image post onto activity/post.json (fixes #3075, BP from #3064)"

This reverts commit 6138554af6200edc11d05195c27bd2d1ba07ecac.

Revision a1e1a104 (diff)
Added by Shouta Kashiwagi about 7 years ago

add an implement of image post onto activity/post.json (fixes #3075, BP from #3064)

Revision e4812f72 (diff)
Added by Yuya Watanabe about 7 years ago

Revert "add an implement of image post onto activity/post.json (fixes #3075, BP from #3064)"

This reverts commit a1e1a104cbef3a56ffa23493cba86276badf87f1.

Revision 6652dbb1 (diff)
Added by Shouta Kashiwagi almost 6 years ago

add an implement of image post onto activity/post.json (fixes #3075, BP from #3064)

(cherry picked from commit 91745b9e8bb9c56460b04d561e5be926920ad083)

Revision fbffaba3 (diff)
Added by Youichi Kimura almost 6 years ago

some cleanup in activity/post.json (fixes #3075, BP from #3064)

(cherry picked from commit 96df88d12b0e15f418b3a6a6da4aaae6240b40cc)

Revision 248c2898 (diff)
Added by Youichi Kimura almost 6 years ago

add testcases for activity/post.json (refs #3075, BP from #3064)

(cherry picked from commit bbf9839510c25ad9da71c5a1a464f46745d2a370)

History

#1 Updated by Shouta Kashiwagi about 7 years ago

  • Tracker changed from Enhancement(機能追加・改善) to Backport(バックポート)

#2 Updated by Shouta Kashiwagi about 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 6138554af6200edc11d05195c27bd2d1ba07ecac で適用されました。

#3 Updated by Yuya Watanabe about 7 years ago

更新履歴 c2104678d167beab63f85b467974a2328a7b3f65 で適用されました。

#4 Updated by Shouta Kashiwagi about 7 years ago

更新履歴 a1e1a104cbef3a56ffa23493cba86276badf87f1 で適用されました。

#5 Updated by Yuya Watanabe about 7 years ago

更新履歴 e4812f72b9fd7177df5047773bba8d8b4dfffe22 で適用されました。

#6 Updated by Mutsumi Imamura about 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.8

公式SNS等で要望が多いので、OpenPNE3.8.8にて対応をします

#7 Updated by Mutsumi Imamura almost 6 years ago

  • Assignee changed from Shouta Kashiwagi to Youichi Kimura

#8 Updated by Mutsumi Imamura almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

親チケットが差し戻しになっていますのでステータス変更します。

#9 Updated by Shouta Kashiwagi almost 6 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 6652dbb1a7b6f647e9d749f21b65cc1ca2fbcd3f で適用されました。

#10 Updated by Youichi Kimura almost 6 years ago

更新履歴 fbffaba3ddcd0e9e1c311e624d8df29c91c20e13 で適用されました。

#11 Updated by Youichi Kimura almost 6 years ago

#3203 の修正が間に入っていない影響でバックポート元のコミットと差分が若干異なっています(images パラメータが配列でないなど)

#12 Updated by Shinichi Urabe almost 6 years ago

確認点としてこちらは機能追加に当たると考えます。安定版に取り込んでもよいものでしょうか。

#13 Updated by Shinichi Urabe almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

#note-12 についての懸念がありましたので、念のため差し戻しにします。

#14 Updated by Mutsumi Imamura almost 6 years ago

Shinichi Urabe は書きました:

確認点としてこちらは機能追加に当たると考えます。安定版に取り込んでもよいものでしょうか。

本修正はDB変更を伴わないですし、安定版の動作に不具合を起こすものではないため、トラッカーはEnhancementですが、取り込むことに問題はありません。

#15 Updated by Mutsumi Imamura almost 6 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

#16 Updated by Shinichi Urabe almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです

#17 Updated by Mutsumi Imamura almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

動作確認OKです。

Also available in: Atom PDF