summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--misc/drupal.css46
-rw-r--r--modules/admin.module2
-rw-r--r--modules/help.module2
-rw-r--r--modules/help/help.module2
-rw-r--r--modules/user.module25
-rw-r--r--modules/user/user.module25
6 files changed, 64 insertions, 38 deletions
diff --git a/misc/drupal.css b/misc/drupal.css
index 936715445..881657227 100644
--- a/misc/drupal.css
+++ b/misc/drupal.css
@@ -259,12 +259,40 @@ pre, code {
color: red;
}
-#menu { padding: 0 0 0 0.5em; }
-#menu ul { list-style: none; margin: 0; padding: 0 0 0.25em 1em; border: none; text-align:left;}
-#menu li { margin: 0; padding: 0.2em 0.5em 0 0; }
-#menu li a { display: block; width: 100%; text-decoration: none; }
-#menu li a.current { color: #000; }
-#menu li a:hover { color: #009; }
-#menu li.expanded { list-style-image: url(menu-expanded.gif); }
-#menu li.collapsed { list-style-image: url(menu-collapsed.gif); }
-#menu li.leaf { list-style-image: url(menu-leaf.gif); }
+#menu {
+ padding: 0 0 0 0.5em;
+}
+#menu ul {
+ list-style: none;
+ margin: 0;
+ padding: 0 0 0.25em 1em;
+ border: none;
+ text-align:left;
+}
+#menu li {
+ margin: 0;
+ padding: 0.2em 0.5em 0 0;
+}
+#menu li a {
+ display: block;
+ width: 100%;
+ text-decoration: none;
+}
+#menu li a.current {
+ color: #000;
+}
+#menu li a:hover {
+ color: #009;
+}
+#menu li.expanded {
+ list-style-type: circle;
+ list-style-image: url(menu-expanded.gif);
+}
+#menu li.collapsed {
+ list-style-type: disc;
+ list-style-image: url(menu-collapsed.gif);
+}
+#menu li.leaf {
+ list-style-type: square;
+ list-style-image: url(menu-leaf.gif);
+}
diff --git a/modules/admin.module b/modules/admin.module
index 1f2994d0f..5057c1f60 100644
--- a/modules/admin.module
+++ b/modules/admin.module
@@ -36,7 +36,7 @@ function admin_system($field){
}
function admin_link($type) {
- if ($type == "system") {
+ if ($type == "system" && user_access("access administration pages")) {
menu("admin", t("administer %a", array("%a" => variable_get("site_name", "drupal"))) , NULL, NULL, 9);
menu("admin/overview", "sitemap", "overview_callback", admin_help("admin/overview"), 8);
}
diff --git a/modules/help.module b/modules/help.module
index 9719ba9e9..cb1bb4a73 100644
--- a/modules/help.module
+++ b/modules/help.module
@@ -7,7 +7,7 @@ function help_system($field){
}
function help_link($type) {
- if ($type == "system") {
+ if ($type == "system" && user_access("access administration pages")) {
menu("admin/help/glossary", "glossary", "help_glossary", NULL, 8);
menu("admin/help", "help", "help_admin", NULL, 9);
}
diff --git a/modules/help/help.module b/modules/help/help.module
index 9719ba9e9..cb1bb4a73 100644
--- a/modules/help/help.module
+++ b/modules/help/help.module
@@ -7,7 +7,7 @@ function help_system($field){
}
function help_link($type) {
- if ($type == "system") {
+ if ($type == "system" && user_access("access administration pages")) {
menu("admin/help/glossary", "glossary", "help_glossary", NULL, 8);
menu("admin/help", "help", "help_admin", NULL, 9);
}
diff --git a/modules/user.module b/modules/user.module
index 35c8fcf61..0ec7d834a 100644
--- a/modules/user.module
+++ b/modules/user.module
@@ -373,7 +373,10 @@ function user_block($op = "list", $delta = 0) {
else {
switch ($delta) {
case 0:
- if (!$user->uid) {
+ if ($user->uid) {
+ $block["subject"] = $user->name;
+ }
+ else {
/*
** For usability's sake, avoid showing two login forms on one
** page.
@@ -422,18 +425,11 @@ function user_block($op = "list", $delta = 0) {
$block["subject"] = t("User login");
$block["content"] = "<div class=\"user-login-link\">$output</div>";
- return $block;
}
- break;
- case 1:
- if ($user->uid) {
- $block["subject"] = $user->name;
- menu_build("system");
- $block["content"] .= "<div id=\"menu\">". menu_tree()."</div>";
-
- return $block;
- }
+ menu_build("system");
+ $block["content"] .= "<div id=\"menu\">". menu_tree()."</div>";
+ return $block;
break;
case 2:
if (user_access("access content")) {
@@ -469,8 +465,11 @@ function user_link($type) {
}
if ($type == "system") {
- menu("user/edit", t("edit account"), NULL, NULL, 8);
- menu("user/logout", t("logout"), NULL, NULL, 10);
+ global $user;
+ if ($user->uid) {
+ menu("user/edit", t("edit account"), NULL, NULL, 8);
+ menu("user/logout", t("logout"), NULL, NULL, 10);
+ }
if (user_access("administer users")) {
menu("admin/user", "user management", "user_admin", user_help("admin/user"), 2);
menu("admin/user/create", "create new account", "user_admin", user_help("admin/user/create"), 1);
diff --git a/modules/user/user.module b/modules/user/user.module
index 35c8fcf61..0ec7d834a 100644
--- a/modules/user/user.module
+++ b/modules/user/user.module
@@ -373,7 +373,10 @@ function user_block($op = "list", $delta = 0) {
else {
switch ($delta) {
case 0:
- if (!$user->uid) {
+ if ($user->uid) {
+ $block["subject"] = $user->name;
+ }
+ else {
/*
** For usability's sake, avoid showing two login forms on one
** page.
@@ -422,18 +425,11 @@ function user_block($op = "list", $delta = 0) {
$block["subject"] = t("User login");
$block["content"] = "<div class=\"user-login-link\">$output</div>";
- return $block;
}
- break;
- case 1:
- if ($user->uid) {
- $block["subject"] = $user->name;
- menu_build("system");
- $block["content"] .= "<div id=\"menu\">". menu_tree()."</div>";
-
- return $block;
- }
+ menu_build("system");
+ $block["content"] .= "<div id=\"menu\">". menu_tree()."</div>";
+ return $block;
break;
case 2:
if (user_access("access content")) {
@@ -469,8 +465,11 @@ function user_link($type) {
}
if ($type == "system") {
- menu("user/edit", t("edit account"), NULL, NULL, 8);
- menu("user/logout", t("logout"), NULL, NULL, 10);
+ global $user;
+ if ($user->uid) {
+ menu("user/edit", t("edit account"), NULL, NULL, 8);
+ menu("user/logout", t("logout"), NULL, NULL, 10);
+ }
if (user_access("administer users")) {
menu("admin/user", "user management", "user_admin", user_help("admin/user"), 2);
menu("admin/user/create", "create new account", "user_admin", user_help("admin/user/create"), 1);