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

• Auswertung: toshimitsu
• Verantwortlicher: (Keine)
• Typ: Fehler
• Status: Geschlossen
• Komponente: (Keine)
• Meilenstein: (Keine)
• Priorität: 5 - Mittel
• Schweregrad: 5 - Mittel
• Lösung: Keine
• Datei: Keine

Details Antworten

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

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-Verlauf (2/2 Historien)

2005-06-30 16:50 Aktualisiert von: toshimitsu

Kommentar Antworten

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 Aktualisiert von: toshimitsu

• Ticket Close date is changed to 2005-06-30 16:51
• Status Update from Offen to Geschlossen