diff --git a/identification.php b/identification.php index 70e46b18e..9a3128536 100644 --- a/identification.php +++ b/identification.php @@ -31,15 +31,13 @@ include_once( PHPWG_ROOT_PATH.'include/common.inc.php' ); check_status(ACCESS_FREE); //-------------------------------------------------------------- identification -$errors = array(); - $redirect_to = ''; if ( !empty($_GET['redirect']) ) { $redirect_to = urldecode($_GET['redirect']); if ( is_a_guest() ) { - array_push($errors, l10n('You are not authorized to access the requested page')); + array_push($page['errors'], l10n('You are not authorized to access the requested page')); } } @@ -48,7 +46,7 @@ if (isset($_POST['login'])) if (!isset($_COOKIE[session_name()])) { array_push( - $errors, + $page['errors'], l10n('Cookies are blocked or not supported by your browser. You must enable cookies to connect.') ); } @@ -68,7 +66,7 @@ if (isset($_POST['login'])) } else { - array_push( $errors, l10n('Invalid password!') ); + array_push($page['errors'], l10n('Invalid password!') ); } } } @@ -96,12 +94,6 @@ if ($conf['allow_user_registration']) $template->assign('U_REGISTER', get_root_url().'register.php' ); } -//-------------------------------------------------------------- errors display -if ( sizeof( $errors ) != 0 ) -{ - $template->assign('errors', $errors); -} - // include menubar $themeconf = $template->get_template_vars('themeconf'); if (!isset($themeconf['hide_menu_on']) OR !in_array('theIdentificationPage', $themeconf['hide_menu_on'])) diff --git a/include/common.inc.php b/include/common.inc.php index 32eb7920a..d4e8984e8 100644 --- a/include/common.inc.php +++ b/include/common.inc.php @@ -63,7 +63,10 @@ if ( !empty($_SERVER["PATH_INFO"]) ) // rewriting of language and otherarray values via URI params // $conf = array(); -$page = array(); +$page = array( + 'infos' => array(), + 'errors' => array(), + ); $user = array(); $lang = array(); $header_msgs = array(); diff --git a/include/page_header.php b/include/page_header.php index 506e93920..55dc3f5c5 100644 --- a/include/page_header.php +++ b/include/page_header.php @@ -98,6 +98,22 @@ if ( isset( $refresh ) and intval($refresh) >= 0 )); } + +// messages +foreach (array('errors','infos') as $mode) +{ + if (isset($_SESSION['page_'.$mode])) + { + $page[$mode] = array_merge($page[$mode], $_SESSION['page_'.$mode]); + unset($_SESSION['page_'.$mode]); + } + + if (count($page[$mode]) != 0) + { + $template->assign($mode, $page[$mode]); + } +} + trigger_action('loc_end_page_header'); header('Content-Type: text/html; charset='.get_pwg_charset()); diff --git a/include/picture_comment.inc.php b/include/picture_comment.inc.php index 064e02056..0273e7b71 100644 --- a/include/picture_comment.inc.php +++ b/include/picture_comment.inc.php @@ -53,28 +53,23 @@ if ( $page['show_comments'] and isset( $_POST['content'] ) ) include_once(PHPWG_ROOT_PATH.'include/functions_comment.inc.php'); - $comment_action = insert_user_comment($comm, @$_POST['key'], $infos ); + $comment_action = insert_user_comment($comm, @$_POST['key'], $page['infos']); switch ($comment_action) { case 'moderate': - array_push( $infos, l10n('An administrator must authorize your comment before it is visible.') ); + array_push($page['infos'], l10n('An administrator must authorize your comment before it is visible.') ); case 'validate': - array_push( $infos, l10n('Your comment has been registered')); + array_push($page['infos'], l10n('Your comment has been registered')); break; case 'reject': set_status_header(403); - array_push($infos, l10n('Your comment has NOT been registered because it did not pass the validation rules') ); + array_push($page['errors'], l10n('Your comment has NOT been registered because it did not pass the validation rules') ); break; default: trigger_error('Invalid comment action '.$comment_action, E_USER_WARNING); } - $template->assign( - ($comment_action=='reject') ? 'errors' : 'infos', - $infos - ); - // allow plugins to notify what's going on trigger_action( 'user_comment_insertion', array_merge($comm, array('action'=>$comment_action) ) diff --git a/nbm.php b/nbm.php index e9e541cad..cb6ae9997 100644 --- a/nbm.php +++ b/nbm.php @@ -37,13 +37,9 @@ trigger_action('loading_lang'); load_language('lang', PHPWG_ROOT_PATH.PWG_LOCAL_DIR, array('no_fallback'=>true, 'local'=>true) ); - // +-----------------------------------------------------------------------+ // | Main | // +-----------------------------------------------------------------------+ -$page['errors'] = array(); -$page['infos'] = array(); - if (isset($_GET['subscribe']) and preg_match('/^[A-Za-z0-9]{16}$/', $_GET['subscribe'])) { @@ -68,16 +64,7 @@ $page['body_id'] = 'theNBMPage'; $template->set_filenames(array('nbm'=>'nbm.tpl')); -// +-----------------------------------------------------------------------+ -// | errors & infos | -// +-----------------------------------------------------------------------+ -$template->assign( - array( - 'errors' => $page['errors'], - 'infos' => $page['infos'], - ) - ); - + // include menubar $themeconf = $template->get_template_vars('themeconf'); if (!isset($themeconf['hide_menu_on']) OR !in_array('theNBMPage', $themeconf['hide_menu_on'])) diff --git a/password.php b/password.php index 1dd88cd17..271026371 100644 --- a/password.php +++ b/password.php @@ -240,10 +240,6 @@ function reset_password() // +-----------------------------------------------------------------------+ // | Process form | // +-----------------------------------------------------------------------+ - -$page['errors'] = array(); -$page['infos'] = array(); - if (isset($_POST['submit'])) { check_pwg_token(); @@ -345,12 +341,6 @@ $template->assign( ) ); -// +-----------------------------------------------------------------------+ -// | infos & errors display | -// +-----------------------------------------------------------------------+ - -$template->assign('errors', $page['errors']); -$template->assign('infos', $page['infos']); // include menubar $themeconf = $template->get_template_vars('themeconf'); diff --git a/picture.php b/picture.php index fb7b19307..bcc1a2939 100644 --- a/picture.php +++ b/picture.php @@ -189,8 +189,6 @@ function default_picture_content($content, $element_info) // | initialization | // +-----------------------------------------------------------------------+ -$infos = array(); - // caching first_rank, last_rank, current_rank in the displayed // section. This should also help in readability. $page['first_rank'] = 0; @@ -338,22 +336,17 @@ UPDATE '.USER_CACHE_CATEGORIES_TABLE.' switch ($comment_action) { case 'moderate': - array_push($infos, l10n('An administrator must authorize your comment before it is visible.')); + array_push($page['infos'], l10n('An administrator must authorize your comment before it is visible.')); case 'validate': - array_push($infos, l10n('Your comment has been registered')); + array_push($page['infos'], l10n('Your comment has been registered')); break; case 'reject': set_status_header(403); - array_push($infos, l10n('Your comment has NOT been registered because it did not pass the validation rules')); + array_push($page['errors'], l10n('Your comment has NOT been registered because it did not pass the validation rules')); break; default: trigger_error('Invalid comment action '.$comment_action, E_USER_WARNING); } - - $template->assign( - ($comment_action=='reject') ? 'errors' : 'infos', - $infos - ); unset($_POST['content']); break; diff --git a/profile.php b/profile.php index 779e4ea81..ab1bcf677 100644 --- a/profile.php +++ b/profile.php @@ -64,7 +64,7 @@ SELECT '.implode(',', $fields).' $userdata = array_merge($userdata, $default_user); } - save_profile_from_post($userdata, $errors); + save_profile_from_post($userdata, $page['errors']); $title= l10n('Your Gallery Customization'); $page['body_id'] = 'theProfilePage'; @@ -75,13 +75,6 @@ SELECT '.implode(',', $fields).' make_index_url(), // for redirect $userdata ); - // +-----------------------------------------------------------------------+ - // | errors display | - // +-----------------------------------------------------------------------+ - if (count($errors) != 0) - { - $template->assign('errors', $errors); - } // include menubar $themeconf = $template->get_template_vars('themeconf'); diff --git a/register.php b/register.php index 63199e8f1..b5cdfb61c 100644 --- a/register.php +++ b/register.php @@ -37,28 +37,27 @@ if (!$conf['allow_user_registration']) page_forbidden('User registration closed'); } -$errors = array(); if (isset($_POST['submit'])) { if (!verify_ephemeral_key(@$_POST['key'])) { set_status_header(403); - array_push($errors, 'Invalid/expired form key'); + array_push($page['errors'], 'Invalid/expired form key'); } if ($_POST['password'] != $_POST['password_conf']) { - array_push($errors, l10n('please enter your password again')); + array_push($page['errors'], l10n('please enter your password again')); } - $errors = + $page['errors'] = register_user($_POST['login'], $_POST['password'], $_POST['mail_address'], true, - $errors); + $page['errors']); - if (count($errors) == 0) + if (count($page['errors']) == 0) { $user_id = get_userid($_POST['login']); log_user($user_id, false); @@ -91,12 +90,6 @@ $template->assign(array( 'obligatory_user_mail_address' => $conf['obligatory_user_mail_address'], )); -//-------------------------------------------------------------- errors display -if (count($errors) != 0) -{ - $template->assign('errors', $errors); -} - // include menubar $themeconf = $template->get_template_vars('themeconf'); if (!isset($themeconf['hide_menu_on']) OR !in_array('theRegisterPage', $themeconf['hide_menu_on'])) diff --git a/search.php b/search.php index f59323db9..201531e31 100644 --- a/search.php +++ b/search.php @@ -31,7 +31,6 @@ include_once( PHPWG_ROOT_PATH.'include/common.inc.php' ); check_status(ACCESS_GUEST); //------------------------------------------------------------------ form check -$errors = array(); $search = array(); if (isset($_POST['submit'])) { @@ -148,11 +147,11 @@ INSERT INTO '.SEARCH_TABLE.' } else { - array_push($errors, l10n('Empty query. No criteria has been entered.')); + array_push($page['errors'], l10n('Empty query. No criteria has been entered.')); } } //----------------------------------------------------------------- redirection -if (isset($_POST['submit']) and count($errors) == 0) +if (isset($_POST['submit']) and count($page['errors']) == 0) { redirect( make_index_url( @@ -222,11 +221,6 @@ SELECT id,name,global_rank,uppercats ;'; display_select_cat_wrapper($query, array(), 'category_options', false); -//-------------------------------------------------------------- errors display -if (sizeof($errors) != 0) -{ - $template->assign('errors', $errors); -} // include menubar $themeconf = $template->get_template_vars('themeconf'); diff --git a/themes/Sylvia/theme.css b/themes/Sylvia/theme.css index bc8b131a9..9093cd475 100644 --- a/themes/Sylvia/theme.css +++ b/themes/Sylvia/theme.css @@ -249,19 +249,15 @@ .infos { color:#f70; border:0; - background-color: transparent; - margin-left:125px; - padding:5px 0px 0pt 10pt; - min-height: 48px; + background:transparent url(icon/infos.png) no-repeat center right; + margin-left:40px; text-align:left; } .errors { color:#f33; border:0; - background: transparent url(icon/errors.png) no-repeat scroll right top; - margin-left:125px; - padding:5px 0px 0pt 10pt; - min-height: 48px; + background:transparent url(icon/errors.png) no-repeat center right; + margin-left:40px; text-align:left; font-weight:bold; } diff --git a/themes/default/template/about.tpl b/themes/default/template/about.tpl index 55f2ec830..407ca2839 100644 --- a/themes/default/template/about.tpl +++ b/themes/default/template/about.tpl @@ -8,6 +8,9 @@