cleaned up system messages display code

git-svn-id: http://code.elgg.org/elgg/trunk@7548 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/engine/lib/elgglib.php b/engine/lib/elgglib.php
index dcc468cd77a2a9390cb72ef65dc7756bb7ca42aa..e6bbb3c013bf682595c938581d69079d81dbc0d5 100644 (file)
--- a/engine/lib/elgglib.php
+++ b/engine/lib/elgglib.php
@@ -608,7 +608,7 @@ function get_register($register_name) {
  *                          whether the message addition was successful.
  * @todo Clean up. Separate registering messages and retrieving them.
  */
-function system_messages($message = null, $register = "messages", $count = false) {
+function system_messages($message = null, $register = "success", $count = false) {
        if (!isset($_SESSION['msg'])) {
                $_SESSION['msg'] = array();
        }
@@ -665,10 +665,10 @@ function count_messages($register = "") {
  *
  * @param string|array $message Message or messages to add
  *
- * @return Bool
+ * @return bool
  */
 function system_message($message) {
-       return system_messages($message, "messages");
+       return system_messages($message, "success");
 }
 
 /**
@@ -678,10 +678,10 @@ function system_message($message) {
  *
  * @param string|array $error Error or errors to add
  *
- * @return true|false Success response
+ * @return bool
  */
 function register_error($error) {
-       return system_messages($error, "errors");
+       return system_messages($error, "error");
 }
 
 /**

diff --git a/engine/lib/views.php b/engine/lib/views.php
index d2e63d1a5732c0fa821531aee574c5e3f6c73f65..17b93926cb9521cc60932a59e48170cc8b9d33ce 100644 (file)
--- a/engine/lib/views.php
+++ b/engine/lib/views.php
@@ -563,10 +563,10 @@ function elgg_view_page($title, $body, $page_shell = 'default', $vars = array())
 
        if (count_messages()) {
                // get messages - try for errors first
-               $sysmessages = system_messages(NULL, "errors");
-               if (count($sysmessages["errors"]) == 0) {
+               $messages = system_messages(NULL, "error");
+               if (count($messages["error"]) == 0) {
                        // no errors so grab rest of messages
-                       $sysmessages = system_messages(null, "");
+                       $messages = system_messages(null, "");
                } else {
                        // we have errors - clear out remaining messages
                        system_messages(null, "");
@@ -575,7 +575,7 @@ function elgg_view_page($title, $body, $page_shell = 'default', $vars = array())
 
        $vars['title'] = $title;
        $vars['body'] = $body;
-       $vars['sysmessages'] = $sysmessages;
+       $vars['sysmessages'] = $messages;
 
        // Draw the page
        $output = elgg_view("page_shells/$page_shell", $vars);

diff --git a/views/default/messages/list.php b/views/default/messages/list.php
deleted file mode 100644 (file)
index dad1c23..0000000
--- a/views/default/messages/list.php
+++ /dev/null
@@ -1,23 +0,0 @@
-<?php
-/**
- * Elgg global system message list
- * Lists all system messages
- *
- * @package Elgg
- * @subpackage Core
- *
- * @uses $vars['object'] The array of message registers
- */
-
-if (!empty($vars['object']) && is_array($vars['object']) && sizeof($vars['object']) > 0) {
-?>
-
-<ul class="elgg-system-messages">
-<?php 
-       foreach ($vars['object'] as $register => $list ) {
-               echo elgg_view("messages/{$register}/list", array('object' => $list));
-       }
-?>
-</ul>
-<?php
-}

diff --git a/views/default/page_elements/messages.php b/views/default/page_elements/messages.php
new file mode 100644 (file)
index 0000000..f44d120
--- /dev/null
+++ b/views/default/page_elements/messages.php
@@ -0,0 +1,28 @@
+<?php
+/**
+ * Elgg global system message list
+ * Lists all system messages
+ *
+ * @package Elgg
+ * @subpackage Core
+ *
+ * @uses $vars['object'] The array of message registers
+ */
+
+if (isset($vars['object']) && is_array($vars['object']) && sizeof($vars['object']) > 0) {
+
+       echo '<ul class="elgg-system-messages">';
+
+       foreach ($vars['object'] as $type => $list ) {
+               foreach ($list as $message) {
+                       echo "<li class=\"elgg-state-$type radius8\">";
+                       echo elgg_view('output/longtext', array(
+                               'value' => $message,
+                               'parse_urls' => false
+                       ));
+                       echo '</li>';
+               }
+       }
+
+       echo '</ul>';
+}

diff --git a/views/default/page_shells/admin.php b/views/default/page_shells/admin.php
index 9fdbf2e289caa3c7fdcb153b3ff0bdc452ebb461..ff427a6b7d6db46da581013d6099183730d8d19e 100644 (file)
--- a/views/default/page_shells/admin.php
+++ b/views/default/page_shells/admin.php
@@ -24,6 +24,6 @@ if (empty($vars['title'])) {
 }\r
 \r
 echo elgg_view('page_elements/html_begin', $vars);\r
-echo elgg_view('messages/list', array('object' => $vars['sysmessages']));\r
+echo elgg_view('page_elements/messages', array('object' => $vars['sysmessages']));\r
 echo elgg_view('page_elements/content', $vars);\r
 echo elgg_view('page_elements/html_end', $vars);
\ No newline at end of file

diff --git a/views/default/page_shells/default.php b/views/default/page_shells/default.php
index 3c643933ae7e275c96228d32e0868bd1a60144b4..3380188d9e8db33e5938e010e16ec777f7ee664e 100644 (file)
--- a/views/default/page_shells/default.php
+++ b/views/default/page_shells/default.php
@@ -27,7 +27,7 @@ if (empty($vars['title'])) {
 echo elgg_view('page_elements/html_begin', $vars);
 
 echo '<div class="elgg-page">';
-echo elgg_view('messages/list', array('object' => $vars['sysmessages']));
+echo elgg_view('page_elements/messages', array('object' => $vars['sysmessages']));
 echo elgg_view('page_elements/topbar', $vars);
 echo elgg_view('page_elements/header', $vars);
 echo elgg_view('page_elements/body', $vars);

diff --git a/views/default/page_shells/walled_garden.php b/views/default/page_shells/walled_garden.php
index 5c4e54b755ee7b456347717355f0400600f9776e..dbd8d3ac05fe9e99c340fe9aa1ccf4daadf25c1e 100644 (file)
--- a/views/default/page_shells/walled_garden.php
+++ b/views/default/page_shells/walled_garden.php
@@ -268,7 +268,7 @@ body {background: white;}
 </style>
 
 <?php
-$view = elgg_view('messages/list', array('object' => $vars['sysmessages']));
+$view = elgg_view('page_elements/messages', array('object' => $vars['sysmessages']));
 
 echo "<div id='walledgarden_sysmessages' class='clearfix'>$view</div>";       
 echo '<div id="walledgarden_container"><div id="walledgarden" class="clearfix">';