move hints to toolbar

includes/controller/user_angeltypes_controller.php

@@ -16,7 +16,7 @@ function user_angeltypes_unconfirmed_hint() {
   foreach ($unconfirmed_user_angeltypes as $user_angeltype)
     $unconfirmed_links[] = '<a href="' . page_link_to('angeltypes') . '&action=view&angeltype_id=' . $user_angeltype['angeltype_id'] . '">' . $user_angeltype['name'] . '</a>';
   
-  return error(sprintf(ngettext("There is %d unconfirmed angeltype.", "There are %d unconfirmed angeltypes.", count($unconfirmed_user_angeltypes)), count($unconfirmed_user_angeltypes)) . " " . _('Angel types which need approvals:') . ' ' . join(', ', $unconfirmed_links));
+  return info(sprintf(ngettext("There is %d unconfirmed angeltype.", "There are %d unconfirmed angeltypes.", count($unconfirmed_user_angeltypes)), count($unconfirmed_user_angeltypes)) . " " . _('Angel types which need approvals:') . ' ' . join(', ', $unconfirmed_links), true);
 }
 
 /**

includes/pages/admin_questions.php

@@ -10,7 +10,7 @@ function admin_new_questions() {
     $new_messages = sql_num_query("SELECT * FROM `Questions` WHERE `AID` IS NULL");
     
     if ($new_messages > 0)
-      info('<a href="' . page_link_to("admin_questions") . '">Es gibt unbeantwortete Fragen!</a>');
+      return info('<a href="' . page_link_to("admin_questions") . '">' . _('There are unanswered questions!') . '</a>', true);
   }
   
   return "";

includes/sys_menu.php

@@ -14,7 +14,7 @@ function page_link_to_absolute($page) {
  * Renders the header toolbar containing search, login/logout, user and settings links.
  */
 function header_toolbar() {
-  global $p, $privileges, $user;
+  global $p, $privileges, $user, $enable_tshirt_size;
   
   $toolbar_items = array();
   
@@ -30,6 +30,35 @@ function header_toolbar() {
   if(isset($user) && in_array('user_messages', $privileges))
     $toolbar_items[] = toolbar_item_link(page_link_to('user_messages'), 'envelope', user_unread_messages());
   
+  $hints = [];
+  if (isset($user)) {
+    if (User_is_freeloader($user))
+      $hints[] = error(sprintf(_("You freeloaded at least %s shifts. Shift signup is locked. Please go to heavens desk to be unlocked again."), $max_freeloadable_shifts), true);
+      
+      // Hinweis für Engel, die noch nicht angekommen sind
+    if ($user['Gekommen'] == 0)
+      $hints[] = error(_("You are not marked as arrived. Please go to heaven's desk, get your angel badge and/or tell them that you arrived already."), true);
+    
+    if ($enable_tshirt_size && $user['Size'] == "")
+      $hints[] = error(_("You need to specify a tshirt size in your settings!"), true);
+    
+    if ($user['DECT'] == "")
+      $hints[] = error(_("You need to specify a DECT phone number in your settings! If you don't have a DECT phone, just enter \"-\"."), true);
+  
+    // Erzengel Hinweis für unbeantwortete Fragen
+    if ($p != "admin_questions") {
+      $new_questions = admin_new_questions();
+     if($new_questions != "")
+       $hints[] = $new_questions; 
+    }
+  
+    $unconfirmed_hint = user_angeltypes_unconfirmed_hint();
+    if($unconfirmed_hint != '')
+      $hints[] = $unconfirmed_hint;
+  }
+  if(count($hints) > 0)
+    $toolbar_items[] = toolbar_popover('warning-sign text-danger', '', $hints, 'bg-danger');
+  
   $user_submenu = make_langselect();
   $user_submenu[] = toolbar_item_divider();
   if (in_array('user_myshifts', $privileges))

includes/sys_template.php

@@ -5,13 +5,13 @@
  */
 $themes = array(
     "0" => "Engelsystem light",
-    "1" => "Engelsystem dark"
+    "1" => "Engelsystem dark" 
 );
 
 /**
  * Render glyphicon
  *
- * @param string $glyph_name
+ * @param string $glyph_name          
  */
 function glyph($glyph_name) {
   return ' <span class="glyphicon glyphicon-' . $glyph_name . '"></span> ';
@@ -20,7 +20,7 @@ function glyph($glyph_name) {
 /**
  * Renders a tick or a cross by given boolean
  *
- * @param boolean $boolean
+ * @param boolean $boolean          
  */
 function glyph_bool($boolean) {
   return '<span class="text-' . ($boolean ? 'success' : 'danger') . '">' . glyph($boolean ? 'ok' : 'remove') . '</span>';
@@ -34,7 +34,7 @@ function div($class, $content = array(), $id = "") {
 /**
  * Render a toolbar.
  *
- * @param array $items
+ * @param array $items          
  * @return string
  */
 function toolbar($items = array(), $right = false) {
@@ -44,10 +44,10 @@ function toolbar($items = array(), $right = false) {
 /**
  * Render a link for a toolbar.
  *
- * @param string $href
+ * @param string $href          
- * @param string $glyphicon
+ * @param string $glyphicon          
- * @param string $label
+ * @param string $label          
- * @param bool $selected
+ * @param bool $selected          
  * @return string
  */
 function toolbar_item_link($href, $glyphicon, $label, $selected = false) {
@@ -58,12 +58,21 @@ function toolbar_item_divider() {
   return '<li class="divider"></li>';
 }
 
-function toolbar_dropdown($glyphicon, $label, $submenu) {
+function toolbar_dropdown($glyphicon, $label, $submenu, $class = '') {
-  return '<li class="dropdown">
+  return '<li class="dropdown ' . $class . '">
           <a href="#" class="dropdown-toggle" data-toggle="dropdown">' . ($glyphicon != '' ? '<span class="glyphicon glyphicon-' . $glyphicon . '"></span> ' : '') . $label . ' <span class="caret"></span></a>
           <ul class="dropdown-menu" role="menu">' . join("\n", $submenu) . '</ul></li>';
 }
 
+function toolbar_popover($glyphicon, $label, $content, $class = '') {
+  $id = md5(microtime() . $glyphicon . $label);
+  return '<li class="dropdown ' . $class . '">
+          <a id="' . $id . '" href="#">' . ($glyphicon != '' ? '<span class="glyphicon glyphicon-' . $glyphicon . '"></span> ' : '') . $label . ' <span class="caret"></span></a>
+          <script type="text/javascript">
+          $(function(){$("#' . $id . '").effect("highlight", {color: "#A94442"}, 500); $("#' . $id . '").popover({trigger: "focus", html: true, content: "' . addslashes(join('', $content)) . '", placement: "bottom", container: "body"})});    
+          </script></li>';
+}
+
 function form_hidden($name, $value) {
   return '<input type="hidden" name="' . $name . '" value="' . $value . '" />';
 }
@@ -111,9 +120,9 @@ function form_checkboxes($name, $label, $items, $selected) {
   $html = form_element($label, '');
   foreach ($items as $key => $item)
     $html .= form_checkbox($name . '_' . $key, $item, array_search($key, $selected) !== false);
-
+  
   return $html;
-
+  
   $html = "<ul>";
   foreach ($items as $key => $item) {
     $id = $name . '_' . $key;
@@ -234,7 +243,7 @@ function form_select($name, $label, $values, $selected) {
 function form_element($label, $input, $for = "") {
   if ($label == '') {
     return '<div class="form-group">' . $input . '</div>';
-  }else {
+  } else {
     return '<div class="form-group">' . '<label for="' . $for . '">' . $label . '</label>' . $input . '</div>';
   }
 }
@@ -269,17 +278,17 @@ function table($columns, $rows_raw, $data = true) {
   // If only one column is given
   if (! is_array($columns)) {
     $columns = array(
-        'col' => $columns
+        'col' => $columns 
     );
-
+    
     $rows = array();
     foreach ($rows_raw as $row)
       $rows[] = array(
-          'col' => $row
+          'col' => $row 
       );
   } else
     $rows = $rows_raw;
-
+  
   if (count($rows) == 0)
     return info(_("No data found."), true);
   $html = "";
@@ -313,9 +322,9 @@ function button($href, $label, $class = "") {
 /**
  * Rendert einen Knopf mit Glyph
  */
- function button_glyph($href, $glyph, $class= "") {
+function button_glyph($href, $glyph, $class = "") {
-   return button($href, glyph($glyph), $class);
+  return button($href, glyph($glyph), $class);
- }
+}
 
 /**
  * Rendert eine Toolbar mit Knöpfen
@@ -367,7 +376,7 @@ function html_options($name, $options, $selected = "") {
   $html = "";
   foreach ($options as $value => $label)
     $html .= '<input type="radio"' . ($value == $selected ? ' checked="checked"' : '') . ' name="' . $name . '" value="' . $value . '"> ' . $label;
-
+  
   return $html;
 }
 
@@ -405,7 +414,7 @@ function ReplaceSmilies($neueckig) {
   $neueckig = str_replace(";P", "<img src=\"pic/smiles/icon_mad.gif\">", $neueckig);
   $neueckig = str_replace(";oP", "<img src=\"pic/smiles/icon_mad.gif\">", $neueckig);
   $neueckig = str_replace("?)", "<img src=\"pic/smiles/icon_question.gif\">", $neueckig);
-
+  
   return $neueckig;
 }
 ?>

public/index.php

@@ -93,27 +93,6 @@ if (isset($_REQUEST['p']) && preg_match("/^[a-z0-9_]*$/i", $_REQUEST['p']) && (i
   $title = $p;
   $content = "";
   
-  if (isset($user)) {
-    if (User_is_freeloader($user))
-      error(sprintf(_("You freeloaded at least %s shifts. Shift signup is locked. Please go to heavens desk to be unlocked again."), $max_freeloadable_shifts));
-  
-    // Hinweis für Engel, die noch nicht angekommen sind
-    if ($user['Gekommen'] == 0)
-      error(_("You are not marked as arrived. Please go to heaven's desk, get your angel badge and/or tell them that you arrived already."));
-  
-    if ($enable_tshirt_size && $user['Size'] == "")
-      error(_("You need to specify a tshirt size in your settings!"));
-  
-    if ($user['DECT'] == "")
-      error(_("You need to specify a DECT phone number in your settings! If you don't have a DECT phone, just enter \"-\"."));
-  
-    // Erzengel Hinweis für unbeantwortete Fragen
-    if ($p != "admin_questions")
-      admin_new_questions();
-  
-    user_angeltypes_unconfirmed_hint();
-  }
-  
   if ($p == "api") {
     require_once realpath(__DIR__ . '/../includes/controller/api.php');
     error("Api disabled temporily.");

templates/layout.html

@@ -13,6 +13,7 @@
 <link rel="stylesheet" type="text/css" href="css/theme%theme%.css" />
 <link rel="stylesheet" type="text/css" href="vendor/icomoon/style.css" />
 <script type="text/javascript" src="vendor/jquery-2.1.1.min.js"></script>
+<script type="text/javascript" src="vendor/jquery-ui.min.js"></script>
 %atom_link%
 </head>
 <body>