Postgres concat operator has relatively low priority, we need to put braces around it to avoid problems when used in complex expressions. This problem does not affect MySQL, as it does not use operator but function. Patch will follow.
Added braces around string concatenation for postgres and generic ANSI.
Attachment #177603 - Flags: review?(mkanat)
Comment on attachment 177603 [details] [diff] [review] V1 Oh funny, I was thinking about that when we wrote the CONCAT patch, too, I just must have forgotten to mention it in review. :-)
Attachment #177603 - Flags: review?(mkanat) → review+
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.20
Checking in Bugzilla/DB.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB.pm,v <-- DB.pm new revision: 1.33; previous revision: 1.32 done Checking in Bugzilla/DB/Pg.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Pg.pm,v <-- Pg.pm new revision: 1.6; previous revision: 1.5 done
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.