Changeset 280

Timestamp:

Jan 15, 2004, 6:06:45 PM (20 years ago)

Author:

gweltas

Message:

Correction of php warnings

Location:

branches/release-1_3

Files:

• admin/stats.php (modified) (3 diffs)
• admin/thumbnail.php (modified) (1 diff)
• admin/user_list.php (modified) (1 diff)
• admin/user_perm.php (modified) (1 diff)
• include/functions_category.inc.php (modified) (3 diffs)
• picture.php (modified) (2 diffs)
• register.php (modified) (2 diffs)
• template/default/htmlfunctions.inc.php (modified) (2 diffs)

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

@@ -43 +43 @@
 $vtp->setGlobalVar( $sub, 'user_template', $user['template'] );
 //--------------------------------------------------- number of days to display
-if ( isset( $_GET['last_days'] ) ) define( MAX_DAYS, $_GET['last_days'] );
-else                               define( MAX_DAYS, 0 );
+if ( isset( $_GET['last_days'] ) ) define( 'MAX_DAYS', $_GET['last_days'] );
+else                               define( 'MAX_DAYS', 0 );
 
 foreach ( $conf['last_days'] as $option ) {
   $vtp->addSession( $sub, 'last_day_option' );
   $vtp->setVar( $sub, 'last_day_option.option', $option );
-  $url = './admin.php?page=stats&expand='.$_GET['expand'];
+  $url = './admin.php?page=stats';
+  if (isset($_GET['expand']))
+          $url .='&expand='.$_GET['expand'];
   $url.= '&last_days='.($option - 1);
   $vtp->setVar( $sub, 'last_day_option.link', add_session_id( $url ) );
@@ -60 +62 @@
 //---------------------------------------------------------------- log  history
 // empty link
-$url = './admin.php?page=stats&last_days='.$_GET['last_days'];
-$url.= '&expand='.$_GET['expand'];
+$url = './admin.php?page=stats';
+if (isset($_GET['last_days']))
+        $url .='&last_days='.$_GET['last_days'];
+// expand array management
+$expand_days = array();
+if (isset($_GET['expand']))
+{
+        $url.= '&expand='.$_GET['expand'];
+        $expand_days = explode( ',', $_GET['expand'] );
+}
 $url.= '&act=empty';
 $vtp->setVar( $sub, 'emply_url', add_session_id( $url ) );
-// expand array management
-$expand_days = explode( ',', $_GET['expand'] );
 $page['expand_days'] = array();
 foreach ( $expand_days as $expand_day ) {
@@ -100 +108 @@
     array_push( $local_expand, $i );
   }
-  $url = './admin.php?page=stats&last_days='.$_GET['last_days'];
+  $url = './admin.php?page=stats';
+  if (isset($_GET['last_days']))
+        $url.= '&last_days='.$_GET['last_days'];
   $url.= '&expand='.implode( ',', $local_expand );
   $vtp->setVar( $sub, 'day.url', add_session_id( $url ) );

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

@@ -254 +254 @@
     $vtp->closeSession( $sub, 'dir' );
     // recursive call
-    $dirs.= get_displayed_dirs( $dir.'/'.$sub_dir,
-                                $indent+30 );
-    
+    get_displayed_dirs( $dir.'/'.$sub_dir,
+                                $indent+30 );    
   }
 }

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

@@ -110 +110 @@
   $action = './admin.php?'.$_SERVER['QUERY_STRING'];
   $vtp->setVar( $sub, 'add_user.form_action', $action );
-  $vtp->setVar( $sub, 'add_user.f_username', $_POST['username'] );
+  if (isset( $_POST['username']))
+          $vtp->setVar( $sub, 'add_user.f_username', $_POST['username'] );
   $vtp->closeSession( $sub, 'add_user' );
   

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

@@ -60 +60 @@
 }
 //---------------------------------------------------------------- form display
-$restrictions = get_restrictions( $_GET['user_id'], $page['user_status'],
+$restrictions = get_user_restrictions( $_GET['user_id'], $page['user_status'],
                                   false, false );
 $action = './admin.php?page=user_perm&user_id='.$_GET['user_id'];

branches/release-1_3/include/functions_category.inc.php

@@ -407 +407 @@
 {
   pwg_debug( 'start initialize_category' );
-  global $page,$lang,$user,$conf;
+  global $page,$lang,$user,$conf, $where_append;
 
   if ( isset( $page['cat'] ) )
@@ -538 +538 @@
         $conf['order_by'] = ' ORDER BY hit DESC, file ASC';
         $page['cat_nb_images'] = $conf['top_number'];
-        if ( $page['start'] + $user['nb_image_page'] >= $conf['top_number'] )
+        if ( isset($page['start']) &&  ($page['start']+ $user['nb_image_page'] >= $conf['top_number'] ))
         {
           $page['nb_image_page'] = $conf['top_number'] - $page['start'];
@@ -544 +544 @@
       }
 
-      if ( $query != '' )
+      if ( isset($query))
       {
         $result = mysql_query( $query );

branches/release-1_3/picture.php

@@ -72 +72 @@
 $page['width']          = $row['width'];
 $page['height']         = $row['height'];
-$page['category_id']    = $row['category_id'];
+if (is_numeric( $page['cat'] ))
+        $page['category_id']    = $row['category_id'];
 $page['keywords']       = $row['keywords'];
 $page['storage_category_id'] = $row['storage_category_id'];
@@ -493 +494 @@
   $row = mysql_fetch_array( $result );
 
-  if ( $array_cat_directories[$row['storage_category_id']] == '' )
+  if ( !isset($array_cat_directories[$row['storage_category_id']]))
   {
     $array_cat_directories[$row['storage_category_id']] =

branches/release-1_3/register.php

@@ -71 +71 @@
 $vtp->addSession( $handle, 'text' );
 $vtp->setVar( $handle, 'text.name', 'login' );
-$vtp->setVar( $handle, 'text.value', $_POST['login'] );
+if (isset( $_POST['login']))
+        $vtp->setVar( $handle, 'text.value', $_POST['login'] );
 $vtp->closeSession( $handle, 'text' );
 $vtp->closeSession( $handle, 'line' );
@@ -95 +96 @@
 $vtp->addSession( $handle, 'text' );
 $vtp->setVar( $handle, 'text.name', 'mail_address' );
-$vtp->setVar( $handle, 'text.value', $_POST['mail_address'] );
+if (isset( $_POST['mail_address']))
+        $vtp->setVar( $handle, 'text.value', $_POST['mail_address'] );
 $vtp->closeSession( $handle, 'text' );
 $vtp->closeSession( $handle, 'line' );

branches/release-1_3/template/default/htmlfunctions.inc.php

@@ -178 +178 @@
   $vtp->addSession( $handle, 'category' );
   $vtp->setVar( $handle, 'category.indent', $indent );
-
   if ( $user['expand'] or $category['nb_sub_categories'] == 0 )
   {
@@ -194 +193 @@
         $url .='?cat='.$page['cat'];
     $url.= '&expand='.$category['expand_string'];
-    if ( $page['cat'] == 'search' )
-    {
-      $url.= '&search='.$_GET['search'].'&mode='.$_GET['mode'];
-    }
+        }
+        else if ($category['expand_string']<>'')
+        {
+                $url.= '?expand='.$category['expand_string'];
         }
     $vtp->setVar( $handle, 'bullet_w_link.bullet_link', add_session_id($url) );