diff options
| author | Frédéric Buclin <LpSolit@gmail.com> | 2015-01-23 12:07:34 +0100 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@gmail.com> | 2015-01-23 12:07:34 +0100 |
| commit | db8c5141a90a910dca5600ba094b6f7d04b749bf (patch) | |
| tree | fd2b0f57196b47c371ec8fd9624543b31f8a1e83 /Bugzilla | |
| parent | cd902140949e2e65e4d99dba94613a282c17aa8f (diff) | |
| download | bugzilla-db8c5141a90a910dca5600ba094b6f7d04b749bf.tar.gz bugzilla-db8c5141a90a910dca5600ba094b6f7d04b749bf.tar.xz | |
Bug 493027: Update the comment in DB::Pg::sql_string_concat()
r=gerv a=glob
Diffstat (limited to 'Bugzilla')
| -rw-r--r-- | Bugzilla/DB/Pg.pm | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/Bugzilla/DB/Pg.pm b/Bugzilla/DB/Pg.pm index 4ec95725d..59231e2a6 100644 --- a/Bugzilla/DB/Pg.pm +++ b/Bugzilla/DB/Pg.pm @@ -187,10 +187,9 @@ sub sql_date_math { sub sql_string_concat { my ($self, @params) = @_; - - # 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. + + # PostgreSQL 8.3 and newer require an explicit coercion to text + # to support concatenation of different data types. return '(CAST(' . join(' AS text) || CAST(', @params) . ' AS text))'; } |