Bug(バグ) #2337
部分一致検索を行う箇所でワイルドカード検索ができてしまう
100%
説明
Overview (現象)¶
メンバー画面の「メンバー検索」や「コミュニティ検索」、あるいは管理画面の「メンバー管理」などでは、ニックネームやキーワード等で部分一致検索が行える。ここで、アンダースコア("_")やパーセント("%")を含めて検索すると、文字としてではなくメタ文字(ワイルドカード)として検索が行なわれてしまう。
これは仕様と考えれば問題ではないと言えるが、この動作は意図した仕様ではなく、不具合として扱うべきであると考えられる。
Causes (原因)¶
LIKE検索を行っている箇所では
$query->addWhere('name LIKE ?', '%'.$value.'%');
のように(部分一致)検索文字列として入力された値を直接渡しているためメタ文字がそのままメタ文字として扱われてしまっている。
Way to fix (修正内容)¶
LIKE検索は関数化されていない(LIKEメソッドがあるわけではない)ため、LIKE文を用いている箇所全てで入力文字列中のメタ文字をエスケープするか、最終的にSQLが生成される時点でエスケープを施すという対応が考えられる。
具体的な修正箇所や修正方針については検討できていない。
Note (補足)¶
バックスラッシュを含む文字列での検索についての挙動も気になる。例えば "foo\bar" というニックネームのメンバーがいる場合、メンバー検索で "foo" と検索すればヒットするが、 "foo\" と検索してもヒットせず、 "foo\\" と検索するとヒットするような動作を確認している。
関連するチケット
関係しているリビジョン
added escape function to like query. (fixes #2337)
moved function of DoctrineQuery::andWhere() to opDoctrineQuery::andWhereLike() and Doctrine_Formatter::escapePattern() to opDoctrineQuery::escapePattern(). (fixes #2337)
履歴
#1 Hiroshi Kato が12年以上前に更新
pull request 出しました。
https://github.com/balibali/opDiaryPlugin/pull/13
#2 Hiroshi Kato が12年以上前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 b6892446ab5c000e8c6a831fb5a3c067d1f4fae6 で適用されました。
#3 Shingo Yamada が12年以上前に更新
- 360対象 を RC1 にセット
#4 Hiroshi Kato が12年以上前に更新
- 対象バージョン を 205 にセット
#5 Hiroshi Kato が12年以上前に更新
更新履歴 6a06c0bd5df4982fb7bd97edc046f9dff85dd65a で適用されました。
#6 Rimpei Ogawa が12年以上前に更新
- 対象バージョン を 205 から v1.4.0 に変更
#7 Rimpei Ogawa が12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Pull request 取り込みました。
#8 Yuma Sakata が12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKです。