Changeset 307

Timestamp:

Jan 21, 2004, 11:03:20 PM (20 years ago)

Author:

z0rglub

Message:

Php warnings correction

File:

• branches/release-1_3/admin/infos_images.php (modified) (12 diffs)

branches/release-1_3/admin/infos_images.php

@@ -24 +24 @@
 
 check_cat_id( $_GET['cat_id'] );
+
+$errors = array();
+
 if ( isset( $page['cat'] ) )
 {
@@ -29 +32 @@
   if ( isset( $_POST['submit'] ) )
   {
-    $errors = array();
     if ( isset( $_POST['associate'] ) )
     {
@@ -48 +50 @@
     }
 
+    $associate = false;
+    
     $query = 'SELECT id,file';
     $query.= ' FROM '.PREFIX_TABLE.'images';
@@ -54 +58 @@
     $query.= ';';
     $result = mysql_query( $query );
-    $i = 1;
     while ( $row = mysql_fetch_array( $result ) )
     {
@@ -91 +94 @@
 
         $query.= ', keywords = ';
+
         $keywords_array = get_keywords( $_POST[$keywords] );
-        if ( count( $keywords_array ) == 0 )
-          $query.= 'NULL';
-        else
-        {
-          $query.= "'";
-          foreach ( $keywords_array as $i => $keyword ) {
-            if ( $i > 0 ) $query.= ',';
-            $query.= $keyword;
-          }
-          $query.= "'";
-        }
+        if ( count( $keywords_array ) == 0 ) $query.= 'NULL';
+        else $query.= "'".implode( ',', $keywords_array )."'";
 
         $query.= ' WHERE id = '.$row['id'];
@@ -109 +104 @@
       }
       // add link to another category
-      if ( $_POST['check-'.$row['id']] == 1 and count( $errors ) == 0 )
+      if ( isset( $_POST['check-'.$row['id']] ) and count( $errors ) == 0 )
       {
         $query = 'INSERT INTO '.PREFIX_TABLE.'image_category';
@@ -116 +111 @@
         $query.= ';';
         mysql_query( $query );
+        $associate = true;
       }
     }
     update_category( $_POST['associate'] );
+    if ( $associate ) synchronize_all_users();
 //------------------------------------------------------ update general options
-    if ( $_POST['use_common_author'] == 1 )
+    if ( isset( $_POST['use_common_author'] ) )
     {
       $query = 'SELECT image_id';
@@ -143 +140 @@
       }
     }
-    if ( $_POST['use_common_date_creation'] == 1 )
+    if ( isset( $_POST['use_common_date_creation'] ) )
     {
       if ( check_date_format( $_POST['date_creation_cat'] ) )
@@ -234 +231 @@
   }
 
-  if ( is_numeric( $_GET['num'] ) and $_GET['num'] >= 0 )
+  if ( isset($_GET['num']) and is_numeric($_GET['num']) and $_GET['num'] >= 0 )
   {
     $page['start'] =
@@ -277 +274 @@
   $array_cat_directories = array();
 
-  $query = 'SELECT id,file,comment,author,tn_ext,name,date_creation,keywords';
-  $query.= ',storage_category_id,category_id';
+  $infos = array( 'id','file','comment','author','tn_ext','name'
+                  ,'date_creation','keywords','storage_category_id'
+                  ,'category_id' );
+  
+  $query = 'SELECT '.implode( ',', $infos );
   $query.= ' FROM '.PREFIX_TABLE.'images';
   $query.= ' INNER JOIN '.PREFIX_TABLE.'image_category ON id = image_id';
@@ -288 +288 @@
   while ( $row = mysql_fetch_array( $result ) )
   {
+    foreach ($infos as $info) { if (!isset($row[$info])) $row[$info] = ''; }
+    
     $vtp->addSession( $sub, 'picture' );
     $vtp->setVar( $sub, 'picture.id', $row['id'] );
@@ -300 +302 @@
     $vtp->setVar( $sub, 'picture.default_name', $file );
     // creating url to thumbnail
-    if ( $array_cat_directories[$row['storage_category_id']] == '' )
+    if ( !isset( $array_cat_directories[$row['storage_category_id']] ) )
     {
       $array_cat_directories[$row['storage_category_id']] =