Changeset 312

Timestamp:

Jan 22, 2004, 11:46:46 PM (20 years ago)

Author:

z0rglub

Message:

• Php warnings correction

• code refactoring

Location:

branches/release-1_3

Files:

• picture.php (modified) (5 diffs)
• profile.php (modified) (4 diffs)

branches/release-1_3/picture.php

@@ -181 +181 @@
   $next_lien_thumbnail.= $conf['prefix_thumbnail'].$file.".".$row['tn_ext'];
   
-  if ( $row['name'] != "" )
+  if ( isset( $row['name'] ) and $row['name'] != '' )
   {
     $next_alt_thumbnail = $row['name'];
@@ -191 +191 @@
   $next_title = $lang['next_image']." : ".$next_alt_thumbnail;
 
-  $next_url_link = './picture.php?image_id='.$row['id'].'&cat='.$page['cat'];
-  $next_url_link.= '&expand='.$_GET['expand'];
+  $next_url_link = './picture.php?image_id='.$row['id'];
+  $next_url_link.= '&cat='.$page['cat'];
+  if ( isset( $_GET['expand'] ) )
+    $next_url_link.= '&expand='.$_GET['expand'];
   if ( $page['cat'] == 'search' )
   {
@@ -291 +293 @@
 
   $prev_title = $lang['previous_image'].' : ';
-  $alt_thumbnaill = '';
-  if ( $row['name'] != '' ) $alt_thumbnail = $row['name'];
-  else                      $alt_thumbnail = $file;
+
+  if ( isset( $row['name'] ) and $row['name'] != '' )
+    $alt_thumbnail = $row['name'];
+  else
+    $alt_thumbnail = $file;
+
   $prev_title.= $alt_thumbnail;
   
   $url_link = './picture.php?image_id='.$row['id'].'&cat='.$page['cat'];
-  $url_link.= '&expand='.$_GET['expand'];
+  if ( isset( $_GET['expand'] ) ) $url_link.= '&expand='.$_GET['expand'];
   if ( $page['cat'] == 'search' )
   {
@@ -471 +476 @@
   $keywords = explode( ',', $page['keywords'] );
   $content = '';
-  $url = './category.php?cat=search&expand='.$_GET['expand'];
+  $url = './category.php?cat=search';
+  if ( isset( $_GET['expand'] ) ) $url.= '&expand='.$_GET['expand'];
   $url.= '&mode=OR&search=';
   foreach ( $keywords as $i => $keyword ) {
@@ -490 +496 @@
 {
   // verify if the picture is already in the favorite of the user
-  $query = 'SELECT COUNT(*) AS nb_fav FROM '.PREFIX_TABLE.'favorites WHERE image_id = '.$page['id'];
+  $query = 'SELECT COUNT(*) AS nb_fav';
+  $query.= ' FROM '.PREFIX_TABLE.'favorites';
+  $query.= ' WHERE image_id = '.$page['id'];
   $query.= ' AND user_id = '.$user['id'].';';
   $result = mysql_query( $query );
   $row = mysql_fetch_array( $result );
   if (!$row['nb_fav'])
-{
-  $url = './picture.php?cat='.$page['cat'].'&image_id='.$page['id'];
-  if (isset($_GET['expand']))
-          $url.= '&expand='.$_GET['expand'];
-  $url.='&add_fav=1';
-  if ( $page['cat'] == 'search' )
-  {
-    $url.= '&search='.$_GET['search'].'&mode='.$_GET['mode'];
-  }
-  $vtp->addSession( $handle, 'favorite' );
-  $vtp->setVar( $handle, 'favorite.link', add_session_id( $url ) );
-  $vtp->setVar( $handle, 'favorite.title', $lang['add_favorites_hint'] );
-  $vtp->setVar( $handle, 'favorite.src',
-                './template/'.$user['template'].'/theme/favorite.gif' );
-  $vtp->setVar( $handle, 'favorite.alt','[ '.$lang['add_favorites_alt'].' ]' );
-  $vtp->closeSession( $handle, 'favorite' );
-}
-else
-{
-  $url = './picture.php?cat='.$page['cat'].'&image_id='.$page['id'];
-  $url.= '&expand='.$_GET['expand'].'&add_fav=0';
-  $vtp->addSession( $handle, 'favorite' );
-  $vtp->setVar( $handle, 'favorite.link', add_session_id( $url ) );
-  $vtp->setVar( $handle, 'favorite.title', $lang['del_favorites_hint'] );
-  $vtp->setVar( $handle, 'favorite.src',
-                './template/'.$user['template'].'/theme/del_favorite.gif' );
-  $vtp->setVar( $handle, 'favorite.alt','[ '.$lang['del_favorites_alt'].' ]' );
-  $vtp->closeSession( $handle, 'favorite' );
-}
+  {
+    $url = './picture.php?cat='.$page['cat'].'&image_id='.$page['id'];
+    if (isset($_GET['expand']))
+      $url.= '&expand='.$_GET['expand'];
+    $url.='&add_fav=1';
+    if ( $page['cat'] == 'search' )
+    {
+      $url.= '&search='.$_GET['search'].'&mode='.$_GET['mode'];
+    }
+    $vtp->addSession( $handle, 'favorite' );
+    $vtp->setVar( $handle, 'favorite.link', add_session_id( $url ) );
+    $vtp->setVar( $handle, 'favorite.title', $lang['add_favorites_hint'] );
+    $vtp->setVar( $handle, 'favorite.src',
+                  './template/'.$user['template'].'/theme/favorite.gif' );
+    $vtp->setVar($handle,'favorite.alt','[ '.$lang['add_favorites_alt'].' ]');
+    $vtp->closeSession( $handle, 'favorite' );
+  }
+  else
+  {
+    $url = './picture.php?cat='.$page['cat'].'&image_id='.$page['id'];
+    $url.= '&expand='.$_GET['expand'].'&add_fav=0';
+    $vtp->addSession( $handle, 'favorite' );
+    $vtp->setVar( $handle, 'favorite.link', add_session_id( $url ) );
+    $vtp->setVar( $handle, 'favorite.title', $lang['del_favorites_hint'] );
+    $vtp->setVar( $handle, 'favorite.src',
+                  './template/'.$user['template'].'/theme/del_favorite.gif' );
+    $vtp->setVar($handle,'favorite.alt','[ '.$lang['del_favorites_alt'].' ]');
+    $vtp->closeSession( $handle, 'favorite' );
+  }
 }
 //------------------------------------ admin link for information modifications

branches/release-1_3/profile.php

@@ -68 +68 @@
   }
   $mail_error = validate_mail_address( $_POST['mail_address'] );
-  if ( $mail_error != '' )
-  {
-    array_push( $errors, $mail_error );
-  }
-  if ( $_POST['use_new_pwd'] == 1 )
-  {
-    // password must be the same as its confirmation
-    if ( $_POST['password'] != $_POST['passwordConf'] )
-    {
-      array_push( $errors, $lang['reg_err_pass'] );
-    }
-  }
-
+  if ( $mail_error != '' ) array_push( $errors, $mail_error );
+  // password must be the same as its confirmation
+  if ( isset( $_POST['use_new_pwd'] )
+       and $_POST['password'] != $_POST['passwordConf'] )
+    array_push( $errors, $lang['reg_err_pass'] );
+  
   if ( count( $errors ) == 0 )
   {
@@ -96 +89 @@
     mysql_query( $query );
 
-    if ( $_POST['use_new_pwd'] == 1 )
+    if ( isset( $_POST['use_new_pwd'] ) )
     {
       $query = 'UPDATE '.PREFIX_TABLE.'users';
@@ -104 +97 @@
       mysql_query( $query );
     }
-    if ( $_POST['create_cookie'] == 1 )
+    if ( isset( $_POST['create_cookie'] ) )
     {
       setcookie( 'id',$page['session_id'],$_POST['cookie_expiration'],
@@ -117 +110 @@
     // redirection
     $url = 'category.php';
-    if ( $_POST['create_cookie'] != 1 ) $url = add_session_id( $url, true );
+    if ( isset( $_POST['create_cookie'] ) ) $url = add_session_id( $url,true );
     header( 'Request-URI: '.$url );  
     header( 'Content-Location: '.$url );