OpenPNE 3

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

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   }

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

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

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

テストOKです。

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

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

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

git log --grep "#2089" master

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

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