MDL-58332 message: remove concat in get_popup_notifications sql
authorRyan Wyllie <ryan@moodle.com>
Mon, 27 Mar 2017 05:49:27 +0000 (05:49 +0000)
committerRyan Wyllie <ryan@moodle.com>
Mon, 10 Jul 2017 02:02:26 +0000 (02:02 +0000)
commit7f805edb3866b4ddae2d00971806a7ea4fdd78bc
tree4ca99fc316728859c46f8ce284f05715633cfb09
parent350700bf8b509f5269b0fefd34fec0d3d5393c99
MDL-58332 message: remove concat in get_popup_notifications sql

Removed the concat to generate the uniqueid field for the popup
notifications data. The concat can't be used directly in the SQL because
the syntax changes between databases. The sql_concat helper can't be
used because it assumes all values are database columns (which they
aren't in this case).

Instead I've just removed the uniqueid field because it isn't required
for the union all to work and the field isn't being used by anything.
This should fixed the compatibility issues between databases.
message/output/popup/classes/api.php