Project

General

Profile

Bug(バグ) #1602

コミュニティカテゴリを空欄で追加できてしまう

Added by Itsuro Tajima about 9 years ago. Updated about 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2010-09-21
Due date:
% Done:

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

コミュニティカテゴリ設定
/pc_backend.php/community/categoryList
で、「大カテゴリ」「小カテゴリ」の入力フォームでカテゴリ名に何も入力せずに「追加」をクリックすると、
カテゴリ名のないカテゴリが追加されてしまう。

ピクチャ_9.png View (60.9 KB) Itsuro Tajima, 2010-09-21 17:54


Related issues

Related to OpenPNE 3 - Backport(バックポート) #1625: コミュニティカテゴリを空欄で追加できてしまう Fixed(完了) 2010-09-21

Associated revisions

Revision d869769b (diff)
Added by Masato Nagasawa about 9 years ago

fixed deleted "default" parameter in the community_category table (refs #1602)

Revision 2f4eb182 (diff)
Added by Masato Nagasawa about 9 years ago

added migration file. (fixes #1602)

Revision f75a37d6 (diff)
Added by Masato Nagasawa about 9 years ago

added migration file.(again) (fixes #1602)

Revision 1f96ae05 (diff)
Added by Masato Nagasawa about 9 years ago

Revert "added migration file.(again) (fixes #1602)"

This reverts commit f75a37d6f947e8aecb2bcf8433a52191a9b1c06d.

Revision 50d24897 (diff)
Added by Masato Nagasawa about 9 years ago

Revert "added migration file. (fixes #1602)"

This reverts commit 2f4eb182ecd04fb9460ddb582b6bd97d4ede9164.

Revision 91194b3e (diff)
Added by Masato Nagasawa about 9 years ago

Revert "fixed deleted "default" parameter in the community_category table (refs #1602)"

This reverts commit d869769b41f462fae23d4d70ff983dfa2dbdd021.

Revision 2340f105 (diff)
Added by Masato Nagasawa about 9 years ago

fixed modified to require input of name in the community category form. (fixes #1602)

Revision bb98bf99 (diff)
Added by Masato Nagasawa about 9 years ago

fixed to trim use the opValidatorString (refs #1602)

Revision 4097c1b4 (diff)
Added by Masato Nagasawa about 9 years ago

fixed trim option and useless processing of a 'unset()' (refs #1602)

Revision b147fa25 (diff)
Added by Masato Nagasawa about 9 years ago

fixed 'required' message (fixes #1602)

History

#1 Updated by Masato Nagasawa about 9 years ago

  • Assignee set to Masato Nagasawa

#2 Updated by Masato Nagasawa about 9 years ago

  • Status changed from New(新規) to Accepted(着手)

#3 Updated by Masato Nagasawa about 9 years ago

アプリ側で空欄を許可しない修正で対応しようと思いましたが、
そもそもschema.ymlで以下のように「default: ""」としてデフォルト値が空欄となっていることが原因であると思います。

name: { type: string(64), default: "", notnull: true, comment: "Category name" }

よって、3.7では「default: ""」を削除し、DB変更を行えない3.6ではアプリ側で許可しないように修正します。

#4 Updated by Masato Nagasawa about 9 years ago

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

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

#5 Updated by Masato Nagasawa about 9 years ago

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

#6 Updated by Masato Nagasawa about 9 years ago

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

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

#7 Updated by Masato Nagasawa about 9 years ago

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

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

#8 Updated by Masato Nagasawa about 9 years ago

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

#9 Updated by Masato Nagasawa about 9 years ago

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

#10 Updated by Masato Nagasawa about 9 years ago

「default: ""」が指定されていた場合にDB側に変更があるものと思い込んでいましたが、
DB側は指定しない場合と一切変更がなく、生成されるform側で空欄を許可しないようになるだけでした。
そのため、一度すべての変更を変更(3コミット)を戻し、3.6で適応したform側を修正する変更を取り込む方向で修正しました。

#11 Updated by Shogo Kawahara about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • コミュニティカテゴリは opStringValidator により、文頭・文末の全角/全角スペースを除去するべきでしょう。

#12 Updated by Masato Nagasawa about 9 years ago

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

#13 Updated by Masato Nagasawa about 9 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

#14 Updated by Masato Nagasawa about 9 years ago

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

#15 Updated by Shogo Kawahara about 9 years ago

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

些細な問題ですが、

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

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

#16 Updated by Shogo Kawahara about 9 years ago

さらに、 CommunityCategoryForm::configure() で 2箇所で unset() しているのが不可解です。

#17 Updated by Masato Nagasawa about 9 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

#18 Updated by Masato Nagasawa about 9 years ago

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

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

#19 Updated by Shogo Kawahara about 9 years ago

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

#20 Updated by Masato Nagasawa about 9 years ago

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

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

#21 Updated by Naoya Tozuka over 8 years ago

  • Assignee deleted (Masato Nagasawa)

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

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

メモ

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

適用前:

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

適用後:

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

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

#22 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70
  • 3.6 で発生するか changed from Yes to Unknown (未調査)
  • 3.4 で発生するか set to Unknown (未調査)

レビューしました.

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

#23 Updated by Shouta Kashiwagi over 7 years ago

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

テストOKです。

#24 Updated by kaoru n about 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF