criteria.phpでのIN句対応変更で仕様が不明確

• 回報者: toshimitsu
• 負責人: (無)
• 類型: 問題回報
• 狀態: 關閉
• 元件: (無)
• 里程碑: (無)
• 優先權: 5 - 中
• 嚴重程度: 5 - 中
• 處理結果: 無
• 檔案: 無

細節 回覆

影響はまだでていないはずだし
要望？質問ともつかないの、こまかい話で、
バグとは言えないので
後で消してもらっても良いです

class/criteria.php
現在
345
if ( '' === ($value = trim($this->value)) ) {
　return '';
}
の
「値の指定が空の場合、条件として整形しない」
という仕様だと思うのですが

IN句対応にした変更の際に
IN句の場合はこの判定していません

なんだか、仕様として不自然
以前と同様に
function render() {
先頭で判定した方が良いよう思います
（これは仕様バグなのでは？）

---
同様に、不自然なのが
xoops2.0.11JP RC1にて
350
$value = $db->quoteString($value);
がされましたが
IN句の値に対してもするべきだと思います

Ticket History (2/2 Histories)

2005-06-30 16:50 Updated by: toshimitsu

評語 回覆

Logged In: YES
user_id=6033

nobunobuさんが、本家の投稿で
/modules/system/admin/findusers/main.php
とかで、すでにAdd-Slashed しているから・・うまく動かないものが
あるらしいことが書いてありました

http://www.xoops.org/modules/news/article.php?
storyid=2383&com_mode=flat&com_order=0&start=60

Xoops2.0.11JP RC1の場合も
quoteStringでAdd-Slasheしているので
同様のことになるはず

2005-06-30 16:51 Updated by: toshimitsu

• Ticket Close date is changed to 2005-06-30 16:51
• 狀態 Update from 開啟 to 關閉