プロジェクト

全般

プロフィール

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

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

Mutsumi Imamura約10年前に追加. 約4年前に更新.

ステータス:
Rejected(差し戻し)
優先度:
Normal(通常)
担当者:
-
対象バージョン:
開始日:
2014-02-21
期日:
進捗率:

50%


説明

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


関連するチケット

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

履歴

#1 Mutsumi Imamura約10年前に更新

#2 Mutsumi Imamura約10年前に更新

#3 Yuya Watanabe約10年前に更新

対象 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 Yuya Watanabe約10年前に更新

同じような名前のコミットが複数あり,最終的に必要なコミットが不明です.最終的なマージコンフリクトの解消は 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 Yuya Watanabe約10年前に更新

フィードバック

下記部分 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 Yuya Watanabe約10年前に更新

フィードバック

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

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

#7 Yuya Watanabe約10年前に更新

  • ステータスNew(新規) から Pending Fixing(修正待ち) に変更

#8 Mutsumi Imamura約10年前に更新

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

#10 Mutsumi Imamuraほぼ10年前に更新

  • ステータスPending Fixing(修正待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#11 Mutsumi Imamuraほぼ10年前に更新

  • 担当者Akihiro KOBAYASHI にセット

#12 Akihiro KOBAYASHI9年以上前に更新

#13 kaoru n約7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 担当者 を削除 (Akihiro KOBAYASHI)
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

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

#14 kaoru n約4年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE 3.10.x に変更

他の形式にエクスポート: Atom PDF