Changeset 405

Timestamp:

Mar 31, 2004, 10:43:09 PM (21 years ago)

Author:

z0rglub

Message:

redirections modification : use of a HTML refresh page instead of header PHP
function. The purpose is to avoid redirections failure when extra characters
are found in included PHP files.

Location:

trunk

Files:

• category.php (modified) (1 diff)
• identification.php (modified) (1 diff)
• include/functions.inc.php (modified) (2 diffs)
• include/page_header.php (modified) (1 diff)
• include/user.inc.php (modified) (1 diff)
• index.php (modified) (1 diff)
• install.php (modified) (1 diff)
• picture.php (modified) (3 diffs)
• profile.php (modified) (1 diff)
• register.php (modified) (1 diff)
• search.php (modified) (1 diff)

trunk/category.php

@@ -37 +37 @@
   setcookie( 'id', '', 0, cookie_path() );
   $url = 'category.php';
-  header( 'Request-URI: '.$url );  
-  header( 'Content-Location: '.$url );  
-  header( 'Location: '.$url );
-  exit();
+  redirect( $url );
 }
 //-------------------------------------------------- access authorization check

trunk/identification.php

@@ -43 +43 @@
     $session_id = session_create( $_POST['username'] );
     $url = 'category.php?id='.$session_id;
-    header( 'Request-URI: '.$url );
-    header( 'Content-Location: '.$url );  
-    header( 'Location: '.$url );
-    exit();
+    redirect( $url );
   }
   else

trunk/include/functions.inc.php

@@ -25 +25 @@
 // | USA.                                                                  |
 // +-----------------------------------------------------------------------+
+
 include_once( PHPWG_ROOT_PATH .'include/functions_user.inc.php' );
 include_once( PHPWG_ROOT_PATH .'include/functions_session.inc.php' );
@@ -463 +464 @@
 }
 
+/**
+ * Redirects to the given URL
+ *
+ * Note : once this function called, the execution doesn't go further
+ * (presence of an exit() instruction.
+ *
+ * @param string $url
+ * @return void
+ */
+function redirect( $url )
+{
+  global $user, $template, $lang_info, $conf, $lang, $t2;
+
+  // $refresh, $url_link and $title are required for creating an automated
+  // refresh page in header.tpl
+  $refresh = 5;
+  $url_link = $url;
+  $title = 'redirection';
+  include( PHPWG_ROOT_PATH.'include/page_header.php' );
+  
+  $template->set_filenames( array( 'redirect' => 'redirect.tpl' ) );
+  $template->pparse('redirect');
+  
+  include( PHPWG_ROOT_PATH.'include/page_tail.php' );
+
+  exit();
+}
 ?>

trunk/include/page_header.php

@@ -31 +31 @@
 $template->set_filenames(array('header'=>'header.tpl'));
 
-$css = PHPWG_ROOT_PATH.'template/'.$user['template'].'/'.$user['template'].'.css';
-$template->assign_vars(array(
-  'CONTENT_ENCODING' => $lang_info['charset'],
-  'PAGE_TITLE' => $title,
-  'LANG'=>$lang_info['code'],
-  'DIR'=>$lang_info['direction'],
-  
-  'T_STYLE' => $css
-  ));
+$css = PHPWG_ROOT_PATH.'template/'.$user['template'];
+$css.= '/'.$user['template'].'.css';
+
+$template->assign_vars(
+  array(
+    'CONTENT_ENCODING' => $lang_info['charset'],
+    'PAGE_TITLE' => $title,
+    'LANG'=>$lang_info['code'],
+    'DIR'=>$lang_info['direction'],
+    
+    'T_STYLE' => $css
+    ));
 
 // refresh
-if ( isset( $refresh ) && $refresh >0 && isset($url_link))
+if ( isset( $refresh ) and $refresh > 0 and isset( $url_link ) )
 {
-  $url = $url_link.'&slideshow='.$refresh;
-  $template->assign_vars(array(
-                           'REFRESH_TIME' => $refresh,
-                           'U_REFRESH' => add_session_id( $url )
-                           ));
+  $template->assign_vars(
+    array(
+      'REFRESH_TIME' => $refresh,
+      'U_REFRESH' => add_session_id( $url_link )
+      ));
   $template->assign_block_vars('refresh', array());
 }
 
-// Work around for "current" Apache 2 + PHP module which seems to not
-// cope with private cache control setting
-if (!empty( $_SERVER['SERVER_SOFTWARE'] )
-    and strstr( $_SERVER['SERVER_SOFTWARE'], 'Apache/2'))
-{
-  header( 'Cache-Control: no-cache, pre-check=0, post-check=0, max-age=0' );
-}
-else
-{
-  header( 'Cache-Control: private, pre-check=0, post-check=0, max-age=0' );
-}
-header( 'Expires: ' . gmdate('D, d M Y H:i:s', time()) . ' GMT' );
-header( 'Last-Modified: ' . gmdate('D, d M Y H:i:s') . ' GMT' );
-
 $template->pparse('header');
 ?>

trunk/include/user.inc.php

@@ -46 +46 @@
   setcookie( 'id', '', 0, cookie_path() );
   $url = 'category.php';
-  header( 'Request-URI: '.$url );  
-  header( 'Content-Location: '.$url );  
-  header( 'Location: '.$url );
-  exit();
+  redirect( $url );
 }
 

trunk/index.php

@@ -37 +37 @@
 // redirection
 $url.= '.php';
-header( 'Request-URI: '.$url );  
-header( 'Content-Location: '.$url );  
-header( 'Location: '.$url );
-exit();
+redirect( $url );
 ?>

trunk/install.php

@@ -181 +181 @@
 // Is phpBB already installed? Yes? Redirect to the index
 if (defined("PHPWG_INSTALLED"))
-{  
-  header( 'Location: index.php' );
-  exit;
+{
+  die( 'PhpWebGallery is already installed' );
 }
 

trunk/picture.php

@@ -181 +181 @@
       // category page
       $url = add_session_id( $url_home );
-      header( 'Request-URI: '.$url );
-      header( 'Content-Location: '.$url );  
-      header( 'Location: '.$url );
-      exit();
+      redirect( $url );
     }
     else if ( $prev < 0 )
@@ -196 +193 @@
       $url = add_session_id( $url, true);
     }
-    header( 'Request-URI: '.$url );
-    header( 'Content-Location: '.$url );  
-    header( 'Location: '.$url );
-    exit();
+    redirect( $url );
   }
 }
@@ -297 +291 @@
 {
   $refresh= $_GET['slideshow'];
-  $url_link = $picture['next']['url'];
+  $url_link = $picture['next']['url'].'&amp;slideshow='.$refresh;
 }
 

trunk/profile.php

@@ -121 +121 @@
     $url = 'category.php';
     if ( !isset($_POST['create_cookie']) ) $url = add_session_id( $url,true );
-    header( 'Request-URI: '.$url );  
-    header( 'Content-Location: '.$url );  
-    header( 'Location: '.$url );
-    exit();
+    redirect( $url );
   }
 }

trunk/register.php

@@ -45 +45 @@
     $session_id = session_create( $_POST['login'] );
     $url = 'category.php?id='.$session_id;
-    header( 'Request-URI: '.$url );
-    header( 'Content-Location: '.$url );  
-    header( 'Location: '.$url );
-    exit();
+    redirect( $url );
   }
 }

trunk/search.php

@@ -56 +56 @@
     $url = 'category.php?cat=search&search='.$search.'&mode='.$_POST['mode'];
     $url = add_session_id( $url, true );
-    header( 'Request-URI: '.$url );
-    header( 'Content-Location: '.$url );  
-    header( 'Location: '.$url );
-    exit();
+    redirect( $url );
   }
 }