#21430 [NEW]: BuildManipSQL is not returning the correct query when building an update stmt

From: ritzcoder@yahoo.com Operating system: Windows XP PHP version: 4.3.0 PHP Bug Type: PEAR related Bug description: BuildManipSQL is not returning the correct query when building an update stmt Hello. I would like to report a bug in common.php of the pear:DB class. In the function buildManipSQL, the code is incorrectly building the sql statement. You have the following starting on line 528: if ($where) { $sql .= " WHERE $sql"; } This code repeats the entire query after the where clause. Instead it should read: if ($where) { $sql .= " WHERE $where"; } Just wanted to report this - it was frustrating me for a while and I decided to peek inside and see what the problem was. Hope this helps. By the way, I searched for this bug but did not see it anywhere yet. If someone has reported it already, sorry for the double post. -- Edit bug report at http://bugs.php.net/?id=21430&edit=1 -- Try a CVS snapshot: http://bugs.php.net/fix.php?id=21430&r=trysnapshot Fixed in CVS: http://bugs.php.net/fix.php?id=21430&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=21430&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=21430&r=needtrace Try newer version: http://bugs.php.net/fix.php?id=21430&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=21430&r=support Expected behavior: http://bugs.php.net/fix.php?id=21430&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=21430&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=21430&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=21430&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=21430&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=21430&r=dst IIS Stability: http://bugs.php.net/fix.php?id=21430&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=21430&r=gnused