Всегда пожаллста! Если что - обращайтесь
Alie, думаю, что стоило поместить ссылку на выход в конец блока. ИМХО выглядит красивше и юзеру промахнуться меньше вероятность. Не приятно при навигации по сайту нечаянно выход кликнуть. Потом входить по-новой...
PHP код:
/* Now we make the Modules block with the correspondent links */
//Ссылка на главную страницу:
$content .= "<strong><big>·</big></strong> <a href=\"index.php\">"._HOME."</a><br>\n";
//Запрос и вывод активных модулей из БД:
$result3 = $db->sql_query("SELECT title, custom_title, view FROM " . $prefix . "_modules WHERE active='1' AND title!='$def_module' AND inmenu='1' ORDER BY custom_title ASC");
while ($row3 = $db->sql_fetchrow($result3)) {
$m_title = stripslashes($row3['title']);
$custom_title = $row3['custom_title'];
$view = intval($row3['view']);
$m_title2 = ereg_replace("_", " ", $m_title);
if ($custom_title != "") {
$m_title2 = $custom_title;
}
if ($m_title != $main_module) {
if ((is_admin($admin) AND $view == 2) OR $view != 2) {
$content .= "<strong><big>·</big></strong> <a href=\"modules.php?name=$m_title\">$m_title2</a><br>\n";
}
}
}
//Ссылка на выход:
if(is_user($user)){
$content .= "<strong><big>·</big></strong> <a href=\"modules.php?name=Your_Account&op=logout\">Выйти</a><br>\n";
}
И еще, зачем лишний код в строчке?:
PHP код:
//Ссылка на выход:
if(is_user($user)){$content .= "<strong><big>·</big></strong> <a href=\"modules.php?name=Your_account&op=logout\">Выйти</a><br>\n";}else{};
Оператор else тут совсем ни к чему и неопытного программера только в заблуждение вводит. Ну и точку с запятой после условных операторов обычно не ставят.
Но это лишь стили программирования - ничего против не имею. Пусть каждый поступает, как ему удобнее.