Changeset 11625

Timestamp:

Jul 4, 2011, 5:08:07 PM (13 years ago)

Author:

J.Commelin

Message:

Fixed bug concerning multiple copyrights with same name.

Location:

extensions/Copyrights

Files:

• TODO (modified) (1 diff)
• admin.php (modified) (1 diff)

extensions/Copyrights/TODO

@@ -10 +10 @@
 
 === Bugs ===
-        - If you create a new copyright with a name that's already used
+        - If you create a new copyright with a name that's already used (fixed??)

extensions/Copyrights/admin.php

@@ -50 +50 @@
     $visible = (isset($_REQUEST['visible']) ? 1 : 0);
     $query = sprintf(
-      'INSERT INTO %s
-      (`name`,`url`,`descr`,`visible`) VALUES
-      ("%s","%s","%s",%d)
+      'SELECT COUNT(*)
+      FROM %s
+      WHERE `name` = '%s'
       ;',
-      COPYRIGHTS_ADMIN, $name, $url, $descr, $visible);
-    pwg_query($query);
+      COPYRIGHTS_ADMIN, $name);
+    list($counter) = pwg_db_fetch_row(pwg_query($query));
+    if ($counter != 0) {
+      array_push($page['errors'], l10n('This copyright already exists'));
+    } else {
+      $query = sprintf(
+        'INSERT INTO %s
+        (`name`,`url`,`descr`,`visible`) VALUES
+        ("%s","%s","%s",%d)
+        ;',
+        COPYRIGHTS_ADMIN, $name, $url, $descr, $visible);
+      pwg_query($query);
+    }
   }