Project

General

Profile

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

携帯で表示する画像の最大サイズを設定可能にする

Added by Masato Nagasawa about 8 years ago. Updated over 7 years ago.

Status:
Fixed(完了)
Priority:
Urgent(急いで)
Target version:
Start date:
2011-05-12
Due date:
% Done:

100%


Description

Overview (概要)

携帯で表示する画像の最大サイズは 320x320 で固定となっているが、
管理画面からサイズの変更を可能にする。

Spec (仕様)

  • デフォルト値は現在の 320x320 にする
  • 入力欄を空欄にした場合、フルサイズにする
  • sfImageHandlerPlugin で許可されているサイズ以外を指定した場合はエラーにする

Related issues

Related to OpenPNE 3 - Bug(バグ) #1861: 携帯のプロフィール画像変更画面などの画像の大サイズ表示がアクセスエラーになる Fixed(完了) 2011-01-05

Associated revisions

Revision 0e6df5a2 (diff)
Added by Masato Nagasawa about 8 years ago

added the image max size of the mobile to the pc_backend (refs #2089)

Revision cf16768e (diff)
Added by Masato Nagasawa about 8 years ago

fixed description of class file (fixes #2089)

Revision 0021f0b7 (diff)
Added by Masato Nagasawa over 7 years ago

added the image max size of the mobile to the pc_backend (refs #2089)

Revision c3d3433b (diff)
Added by Masato Nagasawa over 7 years ago

fixed description of class file (fixes #2089)

Revision 6376c1bb (diff)
Added by Shouta Kashiwagi over 7 years ago

modified image_size setting to choice selectable value. (fixes #2089)

Revision 5db51482 (diff)
Added by Shouta Kashiwagi over 7 years ago

modified image_size setting to choice selectable value. (fixes #2089)

Revision 60b0aec1 (diff)
Added by Shouta Kashiwagi over 7 years ago

added full size option in setting of mobile max image size (fixes #2089)

Revision 7ab4f92a (diff)
Added by Shouta Kashiwagi over 7 years ago

added full size option in setting of mobile max image size (fixes #2089)

History

#1 Updated by Masato Nagasawa about 8 years ago

  • Subject changed from 携帯で表示する画像の最大サイズを指定可能にする to 携帯で表示する画像の最大サイズを設定可能にする

#2 Updated by Masato Nagasawa about 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Masato Nagasawa

#3 Updated by Masato Nagasawa about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#4 Updated by Masato Nagasawa about 8 years ago

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

#5 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#6 Updated by Youichi Kimura over 7 years ago

  • Priority changed from High(高め) to Urgent(急いで)

#7 Updated by Yuya Watanabe over 7 years ago

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

この実装ではテキストフィールドとしてフォームが設置されており,自由にサイズを選べるように見えますが実際にそういうわけではなく,下記の部分で定義されているように設定できる項目が固定されています.
設定できるサイズを知る方法がソースを見るだけという状態ではソフトウェアとして不親切であると思うため,実際にサイズを自由に設定できるようにするか,選択式にして他のものが設定されないようにするようにすべきなのではないでしょうか?

lib/validator/opValidatorImageSize.class.php

 36     if (!in_array($value, sfImageHandler::getAllowedSize()))
 37     {
 38       throw new sfValidatorError($this, 'Not allowed');
 39     }

lib/vendor/symfony/lib/plugins/sfImageHandlerPlugin/lib/image/sfImageHandler.class.php

 44   public static function getAllowedSize()
 45   {
 46     if (self::$allowedSize)
 47     {
 48       return self::$allowedSize;
 49     }
 50 
 51     self::$allowedSize = array_merge(
 52       array('48x48', '76x76', '120x120', '180x180', '240x320', '320x320', '600x600'),
 53       sfConfig::get('sf_image_handler_allowed_size', array())
 54     );
 55 
 56     return self::$allowedSize;
 57   }

#8 Updated by Yuma Sakata over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8beta1

#9 Updated by Masato Nagasawa over 7 years ago

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

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

#10 Updated by Yuya Watanabe over 7 years ago

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

別ブランチにコミットしたものでステータスが変わっているため再度「Rejected」に戻します.

#11 Updated by Shouta Kashiwagi over 7 years ago

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

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

#12 Updated by Shouta Kashiwagi over 7 years ago

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

#13 Updated by Yuya Watanabe over 7 years ago

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

#14 Updated by Yuya Watanabe over 7 years ago

  • Assignee changed from Masato Nagasawa to Shouta Kashiwagi

#15 Updated by Yuma Sakata over 7 years ago

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

テストOKです。

#16 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Fixed(完了) to Rejected(差し戻し)
  • % Done changed from 100 to 50

すみませんフルサイズ指定できるのを忘れていたので一旦差し戻します

#17 Updated by Shouta Kashiwagi over 7 years ago

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

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

#18 Updated by Shouta Kashiwagi over 7 years ago

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

#19 Updated by Yuya Watanabe over 7 years ago

master

0e6df5a238b75abc84f9a3b1d378410646f871b8 added the image max size of the mobile to the pc_backend (refs #2089)
cf16768ec4e8bc6b62fdb121d033e5bc169e4d21 fixed description of class file (fixes #2089)
5db51482cf94f88356bee0593cd25c9933b8eff9 modified image_size setting to choice selectable value. (fixes #2089)
60b0aec1135edbf88a4d50ed60d35010606b12d6 added full size option in setting of mobile max image size (fixes #2089)

release-3.8beta1

0e6df5a238b75abc84f9a3b1d378410646f871b8 added the image max size of the mobile to the pc_backend (refs #2089)
cf16768ec4e8bc6b62fdb121d033e5bc169e4d21 fixed description of class file (fixes #2089)
cc8ea818f66bc399435a58d83eae0091b2d0c2ad Revert (#2089) "fixed description of class file"
12ac5cbb3c01d613fb0ecbfb74e52fa26beb3a4d Revert (#2089) "added the image max size of the mobile to the pc_backend"
0021f0b7656bae96b6cd63992f93c65b5700c552 added the image max size of the mobile to the pc_backend (refs #2089)
c3d3433bb5e97ab1f2df59022d9f4419e503b7cb fixed description of class file (fixes #2089)
6376c1bbd05e320a32544f4d1d8427b4f933f086 modified image_size setting to choice selectable value. (fixes #2089)
7ab4f92a3e5511816e815f661c2b2043b4292eaf added full size option in setting of mobile max image size (fixes #2089)

#20 Updated by Yuya Watanabe over 7 years ago

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

#21 Updated by Yuya Watanabe over 7 years ago

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

下記コマンド等で note-19 の結果を見ましたが revert コミットがmasterに含まれていなかったので少し調査します.

git log --grep "#2089" master

#22 Updated by Yuya Watanabe over 7 years ago

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

release-3.8beta1 では一旦 revert したものを再適用したものということがわかりましたのでレビュー待ちに変更します.

#23 Updated by Kiwa Sakai over 7 years ago

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

【3.8beta1テスト表】https://docs.google.com/a/openpne.jp/spreadsheet/ccc?key=0AsdDZCDQSqiYdDFCSGdTZzlYQWV6bDFkTmI2N2p2emc#gid=0
テスト完了しているようなのでこのチケットは閉じます。

Also available in: Atom PDF