OpenPNE 3

対象 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

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

フィードバック¶

下記部分 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"]

フィードバック¶

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

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

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

https://github.com/openpne/OpenPNE3/pull/191
更新しました
https://github.com/AkihiroKOBAYASHI/OpenPNE3/commit/e04f627bf6b22ba4d7f48f7e0e7f1577226931de(一応)