Changeset 11675 for extensions/Copyrights/modify.php

Timestamp:

Jul 9, 2011, 3:30:34 PM (13 years ago)

Author:

Mattias

Message:

Fixed bugs in modify.php, it works now, though it still looks awfull :-/
Added language for the error message in admin.php

File:

• extensions/Copyrights/modify.php (modified) (3 diffs)

extensions/Copyrights/modify.php

@@ -3 +3 @@
 // Add a prefilter
 add_event_handler('loc_begin_admin', 'CR_set_prefilter_modify', 50 );
-add_event_handler('loc_end_admin', 'CR_modify_submit', 50 );
+add_event_handler('loc_begin_admin', 'CR_modify_submit', 50 );
 
 // Change the variables used by the function that changes the template
-add_event_handler('loc_end_admin', 'CR_add_modify_vars_to_template');
+add_event_handler('loc_begin_admin', 'CR_add_modify_vars_to_template');
 
 function CR_set_prefilter_modify()
@@ -23 +23 @@
                         <select id="copyright" name="copyright">
                                 <option value="0">--</option>
-                                {html_options options=$CRoptions selected=$CRcopyrights[$CRid]}
+                                {html_options options=$CRoptions selected=$CRid}
                         </select>
                 </td>
@@ -36 +36 @@
 function CR_add_modify_vars_to_template()
 {
-        global $template;
+        // Check if we are at the modify page (the problem is that this will send the stuff to every admin page that has an ?image_id...
+        if (isset($_GET['image_id']))
+        {
+                global $template;
 
-        load_language('plugin.lang', dirname(__FILE__).'/');
+                load_language('plugin.lang', dirname(__FILE__).'/');
 
-        // Fetch all the copyrights and assign them to the template
-        $query = sprintf(
-                'SELECT `cr_id`,`name`
-                FROM %s
-                WHERE `visible`<>0
-                ;',
-                COPYRIGHTS_ADMIN);
-        $result = pwg_query($query);
+                // Fetch all the copyrights and assign them to the template
+                $query = sprintf(
+                        'SELECT `cr_id`,`name`
+                        FROM %s
+                        WHERE `visible`<>0
+                        ;',
+                        COPYRIGHTS_ADMIN);
+                $result = pwg_query($query);
 
-        $CRoptions = array();
-        while ($row = pwg_db_fetch_assoc($result)) {
-                $CRoptions[$row['cr_id']] = $row['name'];
+                $CRoptions = array();
+                while ($row = pwg_db_fetch_assoc($result)) {
+                        $CRoptions[$row['cr_id']] = $row['name'];
+                }
+                $template->assign('CRoptions', $CRoptions);
+                
+                // Get the current Copyright
+                $image_id = $_GET['image_id'];
+                $query = sprintf(
+                        'SELECT `media_id`, `cr_id`
+                        FROM %s
+                        WHERE `media_id` = %d
+                        ;',
+                        COPYRIGHTS_MEDIA, $image_id);
+                $result = pwg_query($query);
+                
+                $CRid = 0; // Default is '--'
+                while ($row = pwg_db_fetch_assoc($result)) {
+                        $CRid = $row['cr_id'];
+                }
+                $template->assign('CRid', $CRid);
         }
-        $template->assign('CRoptions', $CRoptions);
-        
-        // Get the current Copyright
-        $image_id = $_GET['image_id']);
-        $query = sprintf(
-                'SELECT `media_id`, `cr_id`
-                FROM %s
-    WHERE `media_id` = %d
-                ;',
-                COPYRIGHTS_MEDIA, $image_id);
-        $result = pwg_query($query);
-        
-  $CRid = 0; // Default is '--'
-        while ($row = pwg_db_fetch_assoc($result)) {
-                $CRid = $row['cr_id'];
-        }
-        $template->assign('CRid', $CRid);
 }
 
 function CR_modify_submit()
 {
-        if (isset($_POST['submit']))
+        // Check if we are at the modify page (the problem is that this will send the stuff to every admin page that has an ?image_id...
+        if (isset($_GET['image_id']))
         {
-                // The data from the submit
-                $image_id = $_GET['image_id']);
-    $CRid = $_POST['copyright'];
+                if (isset($_POST['submit']))
+                {
+                        // The data from the submit
+                        $image_id = $_GET['image_id'];
+                        $CRid = $_POST['copyright'];
 
-                // Delete the Copyright if it allready exists
+                        // Delete the Copyright if it allready exists
+                        $query = sprintf(
+                                'DELETE
+                                FROM %s
+                                WHERE `media_id` = %d
+                                ;',
+                                COPYRIGHTS_MEDIA, $image_id);
+                        pwg_query($query);
+
+                // Insert the Copyright
                 $query = sprintf(
-                        'DELETE
-                        FROM %s
-                        WHERE `media_id` = %d
-                        ;',
-                        COPYRIGHTS_MEDIA, $image_id);
+                  'INSERT INTO %s
+                  VALUES (%d, %d)
+                  ;',
+                  COPYRIGHTS_MEDIA, $image_id, $CRid);
                 pwg_query($query);
-
-    // Insert the Copyright
-    $query = sprintf(
-      'INSERT INTO %s
-      VALUES (%d, %d)
-      ;',
-      COPYRIGHTS_MEDIA, $image_id, $CRid);
-    pwg_query($query);
+                }
         }
 }