Bug 286392: String concatenation on postgres does not always work

Patch By Tomas Kopal <Tomas.Kopal@altap.cz> r=mkanat, a=justdave

2 files changed, 2 insertions, 2 deletions

diff --git a/Bugzilla/DB.pm b/Bugzilla/DB.pm
index 0d41bbd01..1c2bcbf99 100644
--- a/Bugzilla/DB.pm
+++ b/Bugzilla/DB.pm
@@ -225,7 +225,7 @@ sub sql_group_by {
 sub sql_string_concat {
     my ($self, @params) = @_;
     
-    return join(' || ', @params);
+    return '(' . join(' || ', @params) . ')';
 }
 
 sub sql_fulltext_search {
diff --git a/Bugzilla/DB/Pg.pm b/Bugzilla/DB/Pg.pm
index de3260e7a..be921f4d1 100644
--- a/Bugzilla/DB/Pg.pm
+++ b/Bugzilla/DB/Pg.pm
@@ -137,7 +137,7 @@ sub sql_string_concat {
     # Postgres 7.3 does not support concatenating of different types, so we
     # need to cast both parameters to text. Version 7.4 seems to handle this
     # properly, so when we stop support 7.3, this can be removed.
-    return 'CAST(' . join(' AS text) || CAST(', @params) . ' AS text)';
+    return '(CAST(' . join(' AS text) || CAST(', @params) . ' AS text))';
 }
 
 sub bz_lock_tables {