OpenPNE 3

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

マイグレーションの処理に問題が見つかったため戻します。

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

• コミュニティカテゴリは opStringValidator により、文頭・文末の全角/全角スペースを除去するべきでしょう。

些細な問題ですが、

$this->setValidator('name', new opValidatorString(array('max_length' => 64, 'ltrim' => true, 'rtrim' => true)));

のオプションで、 ltrim, rtrim を指定していますが、
trim を true にするだけで良いはずです。

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

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

修正箇所( 2340f1059 bb98bf99 4097c1b4 b147fa25 )確認しました。

一点気になる点がありますので、他の方にも意見を求めるべく一旦留保します。

メモ¶

b147fa25 (fixed 'required' message) について、

適用前：

• 既存のカテゴリ名を空欄にして「編集」　→ ページ上部に赤い字で「name [Required.]」
• 新規カテゴリ名を空欄で「追加」　→ カテゴリ名フィールドの真上に赤い字で「必須項目です。」

適用後：

• 既存のカテゴリ名を空欄にして「編集」　→ ページ上部に赤い字で「必須項目です。」
• 新規カテゴリ名を空欄で「追加」　→ カテゴリ名フィールドの真上に赤い字で「必須項目です。」

のようにメッセージがflash表示されるようになりますが、必須項目なのがカテゴリ名であるという情報が欠落してしまうのは不親切ではないでしょうか。

レビューしました．

本チケットの主題である「コミュニティカテゴリを空欄で追加できてしまう」という問題の修正については問題ないと思います．ただし， note-21 で上げられているような修正はこのチケットとは関係ない部分と言えるため，本来ならば別チケットで対応が正しいと思います．note-21 の問題は安定版である OpenPNE 3.6 にも含まれている状態のため別のバグとして扱うのが妥当であると思い，本チケットについてはレビューOKとしてステータスを「Pending Testing」に変更します．

テストOKです。