Changeset 2265 for trunk/upload.php

Timestamp:

Mar 8, 2008, 2:38:37 AM (16 years ago)

Author:

rvelices

Message:

• upload.tpl goes smarty
• start some language cleanup and a small attempt to standardize a bit ...
• debug_language now calls trigger_error instead of echo when missing language key

File:

• trunk/upload.php (modified) (12 diffs)

trunk/upload.php

@@ -3 +3 @@
 // | PhpWebGallery - a PHP based picture gallery                           |
 // | Copyright (C) 2002-2003 Pierrick LE GALL - pierrick@phpwebgallery.net |
-// | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
+// | Copyright (C) 2003-2008 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
 // | file          : $Id$
@@ -104 +104 @@
       case 3 : $result['type'] = 'png'; break;
       default :
-        array_push( $result['error'], l10n('upload_advise_filetype') );  
+        array_push( $result['error'], l10n('upload_advise_filetype') );
       }
     }
@@ -135 +135 @@
   $category = get_cat_info( $page['category'] );
   $category['cat_dir'] = get_complete_dir( $page['category'] );
-  
+
   if (url_is_remote($category['cat_dir']) or !$category['uploadable'])
   {
-    die('Fatal: you take a wrong way, bye bye');
+    page_forbidden('upload not allowed');
   }
 }
 else { // $page['category'] may be set by a futur plugin but without it
-  die('Fatal: you take a wrong way, bye bye');
+  bad_request('invalid parameters');
 }
 
@@ -174 +174 @@
     array_push( $error, l10n('upload_err_username') );
   }
-  
+
   $date_creation = '';
   if ( !empty($_POST['date_creation']) )
@@ -203 +203 @@
     array_push( $error, l10n('update_wrong_dirname') );
   }
-  
+
   if ( sizeof( $error ) == 0 )
   {
@@ -316 +316 @@
 }
 
-$template->assign_vars(
+$template->assign(
   array(
-    'U_HOME' => make_index_url(),
-
     'ADVISE_TITLE' => $advise_title,
     'NAME' => $username,
@@ -333 +331 @@
     )
   );
-  
+
+$template->assign('errors', $error);
+$template->assign('UPLOAD_SUCCESSFUL', $page['upload_successful'] );
+
 if ( !$page['upload_successful'] )
 {
-  $template->assign_block_vars('upload_not_successful',array());
-//-------------------------------------------------------------- errors display
-if ( sizeof( $error ) != 0 )
-{
-  $template->assign_block_vars('upload_not_successful.errors',array());
-  for ( $i = 0; $i < sizeof( $error ); $i++ )
-  {
-    $template->assign_block_vars('upload_not_successful.errors.error',array('ERROR'=>$error[$i]));
-  }
-}
-
 //--------------------------------------------------------------------- advises
   if ( !empty($conf['upload_maxfilesize']) )
@@ -352 +342 @@
     $content = l10n('upload_advise_filesize');
     $content.= $conf['upload_maxfilesize'].' KB';
-    $template->assign_block_vars('upload_not_successful.advise',array('ADVISE'=>$content));
+    $template->append('advises', $content);
   }
 
@@ -361 +351 @@
       $content = l10n('upload_advise_width');
       $content.= $conf['upload_maxwidth_thumbnail'].' px';
-      $template->assign_block_vars('upload_not_successful.advise',array('ADVISE'=>$content));
+      $template->append('advises', $content);
     }
     if ( $conf['upload_maxheight_thumbnail'] != '' )
@@ -367 +357 @@
       $content = l10n('upload_advise_height');
       $content.= $conf['upload_maxheight_thumbnail'].' px';
-      $template->assign_block_vars('upload_not_successful.advise',array('ADVISE'=>$content));
+      $template->append('advises', $content);
     }
   }
@@ -376 +366 @@
       $content = l10n('upload_advise_width');
       $content.= $conf['upload_maxwidth'].' px';
-      $template->assign_block_vars('upload_not_successful.advise',array('ADVISE'=>$content));
+      $template->append('advises', $content);
     }
     if ( $conf['upload_maxheight'] != '' )
@@ -382 +372 @@
       $content = l10n('upload_advise_height');
       $content.= $conf['upload_maxheight'].' px';
-      $template->assign_block_vars('upload_not_successful.advise',array('ADVISE'=>$content));
-    }
-  }
-  $template->assign_block_vars('upload_not_successful.advise',array('ADVISE'=>l10n('upload_advise_filetype')));
-  
+      $template->append('advises', $content);
+    }
+  }
+  $template->append('advises', l10n('upload_advise_filetype'));
+
 //----------------------------------------- optionnal username and mail address
   if ( !isset( $page['waiting_id'] ) )
   {
-    $template->assign_block_vars('upload_not_successful.fields',array());
-    $template->assign_block_vars('note',array());
-  }
-}
-else
-{
-  $template->assign_block_vars('upload_successful',array());
+    $template->assign('SHOW_FORM_FIELDS', true);
+  }
 }