Fixed issue #12059: Internal Server Error Property 'UniqueNotificatio…

…n.hash' is not defined
LimeSurvey · Jan 16, 2017 · 9fd7f17 · 9fd7f17 · Shnoulle · Jan 25, 2017
1 parent d104a79
commit 9fd7f17
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/application/config/version.php b/application/config/version.php
@@ -13,7 +13,7 @@
  */
 
 $config['versionnumber'] = '2.59.0';
-$config['dbversionnumber'] = 260;
+$config['dbversionnumber'] = 261;
 $config['buildnumber'] = '';
 $config['updatable'] = true;
 $config['assetsversionnumber'] = '2590';

diff --git a/application/helpers/update/updatedb_helper.php b/application/helpers/update/updatedb_helper.php
@@ -1438,11 +1438,24 @@ function db_upgrade_all($iOldDBVersion, $bSilent=false) {
             alterColumn('{{participant_attribute_names_lang}}','attribute_name',"string(255)",false);
             $oDB->createCommand()->update('{{settings_global}}',array('stg_value'=>260),"stg_name='DBVersion'");
         }
+
+        /*
+         * The hash value of a notification is used to calculate uniqueness.
+         * @since 2016-08-10
+         * @author Olle Haerstedt
+         */
+        if ($iOldDBVersion < 261) {
+            addColumn('{{notifications}}', 'hash', 'string(64)');
+            $oDB->createCommand()->createIndex('notif_hash_index', '{{notifications}}', 'hash', false);
+
+            $oDB->createCommand()->update('{{settings_global}}',array('stg_value'=>262),"stg_name='DBVersion'");
+        }
+
         // Inform superadmin about update
         $superadmins = User::model()->getSuperAdmins();
         Notification::broadcast(array(
             'title' => gT('Database update'),
-            'message' => sprintf(gT('The database has been updated from version %s to version %s.'), $iOldDBVersion, '260')
+            'message' => sprintf(gT('The database has been updated from version %s to version %s.'), $iOldDBVersion, '261')
         ), $superadmins);