Project

General

Profile

Enhancement(機能追加・改善) #3561

画像アップロード時に上限サイズを超えるファイルは自動的に縮小してアップロードする

Added by Mutsumi Imamura over 5 years ago. Updated over 2 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2014-02-21
Due date:
% Done:

50%


Description

画像アップロード時に上限サイズを超えるファイルは自動的に縮小してアップロードする。
OpenPNEおよびサーバー側のアップロードサイズ制限の上限をアップすることで回避することはできますが、
サーバーのディスク容量の切迫にも繋がるため、リサイズし保存する機能を追加する。


Related issues

Related to OpenPNE 3 - Enhancement(機能追加・改善) #3219: 投稿画像を自動的に圧縮する機能を実装する New(新規) 2012-09-28
Copied to OpenPNE 3 - Backport(バックポート) #3563: 画像アップロード時に上限サイズを超えるファイルは自動的に縮小してアップロードする Fixed(完了) 2014-02-21
Copied to OpenPNE 3 - Backport(バックポート) #3565: 画像アップロード時に上限サイズを超えるファイルは自動的に縮小してアップロードする New(新規) 2014-02-21

History

#1 Updated by Mutsumi Imamura over 5 years ago

#2 Updated by Mutsumi Imamura over 5 years ago

#3 Updated by Yuya Watanabe over 5 years ago

対象 pull request

able to resize the uploaded image when its size is larger than assigned. by AkihiroKOBAYASHI · Pull Request #104 · openpne/OpenPNE3
https://github.com/openpne/OpenPNE3/pull/104

#4 Updated by Yuya Watanabe over 5 years ago

同じような名前のコミットが複数あり,最終的に必要なコミットが不明です.最終的なマージコンフリクトの解消は master マージ時に行いますので本チケットの修正に則した内容のコミットのみのプルリクエストでお願いします.

b202426 able to resize the uploaded image when its size is larger than assigned.
28047ed start
4ad8194 able to resize the uploaded image when its size is larger than assigned.
dc57650 start
d897a1d Merge branch 'master' of https://github.com/AkihiroKOBAYASHI/OpenPNE3
30fb274 Merge branch 'master' of https://github.com/AkihiroKOBAYASHI/OpenPNE3
29cecef able to compress the uploaded image when its size is larger than assigned size
3b8dc51 able to resize the uploaded image when the size of it is larger than that of assigned
f20e9a3 able to read exif data and orient the direction to repair of the uploaded image

#5 Updated by Yuya Watanabe over 5 years ago

フィードバック

下記部分 ssl_selectable_actions に default/customizingCss が追加されている意図はなんでしょうか?

config/OpenPNE.yml.sample
137 # SSL を使用するかどうかをメンバーが選択できるアクション
138 # Actions that are SSL selectable by SNS member
139 ssl_selectable_actions:
140 
141   # PC 版設定
142   # for pc_frontend
143   pc_frontend: ["default/customizingCss"]

#6 Updated by Yuya Watanabe over 5 years ago

フィードバック

すべて save() メソッドの中に記述されていますが,この辺りは「縮小する」という明確な目的の一連の流れが存在するため,名前付けしてメソッドとして抽出してもらったほうがよいです.

メソッド抽出の具体的なメリットは リファクタリングという書籍の 「Extract Method(メソッドの抽出)」の項目が参考になります.

#7 Updated by Yuya Watanabe over 5 years ago

  • Status changed from New(新規) to Pending Fixing(修正待ち)

#8 Updated by Mutsumi Imamura over 5 years ago

https://github.com/openpne/OpenPNE3/pull/108 でプルリクエストが来ています

#10 Updated by Mutsumi Imamura over 5 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

プルリクエストが来ていますのでステータスを変更します。

#11 Updated by Mutsumi Imamura over 5 years ago

  • Assignee set to Akihiro KOBAYASHI

#12 Updated by Akihiro KOBAYASHI almost 5 years ago

#13 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee deleted (Akihiro KOBAYASHI)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

Also available in: Atom PDF