Merge branch 'release/1.8.2'

Author: cedric-anne

inc/alert.class.php

@@ -180,10 +180,10 @@ public function rawSearchOptions() {
    public static function findAllToNotify($params = []) {
       global $DB;
 
-      $p['show_only_login_alerts'] = false;
-      $p['show_hidden_alerts']     = false;
-      $p['show_helpdesk_alerts']   = false;
-      $p['entities_id']            = false;
+      $p['show_only_login_alerts']     = false;
+      $p['show_hidden_alerts']         = false;
+      $p['show_only_helpdesk_alerts']  = false;
+      $p['entities_id']                = false;
       foreach ($params as $key => $value) {
          $p[$key] = $value;
       }
@@ -236,7 +236,7 @@ public static function findAllToNotify($params = []) {
 
       //If the alert must be displayed on helpdesk form : filter by ticket's entity
       //and not the current entity
-      if ($p['show_helpdesk_alerts']) {
+      if ($p['show_only_helpdesk_alerts']) {
          $show_helpdesk_sql = " AND `$table`.`is_displayed_onhelpdesk`='1'";
       }
       if (!$p['show_only_login_alerts']) {
@@ -416,7 +416,7 @@ public function showForm($ID, $options = []) {
 
    static function displayOnCentral() {
       echo "<tr><th colspan='2'>";
-      self::displayAlerts(['show_only_login_alerts' => false]);
+      self::displayAlerts(['show_only_helpdesk_alerts' => Session::getCurrentInterface() == 'helpdesk']);
       echo "</th></tr>";
    }
 
@@ -430,10 +430,10 @@ static function displayOnLogin() {
    static function displayAlerts($params = []) {
       global $CFG_GLPI;
 
-      $p['show_only_login_alerts'] = false;
-      $p['show_hidden_alerts']     = false;
-      $p['show_helpdesk_alerts']   = false;
-      $p['entities_id']            = false;
+      $p['show_only_login_alerts']     = false;
+      $p['show_hidden_alerts']         = false;
+      $p['show_only_helpdesk_alerts']  = false;
+      $p['entities_id']                = false;
       foreach ($params as $key => $value) {
          $p[$key] = $value;
       }
@@ -502,9 +502,9 @@ static function preItemForm($params = []) {
          $entities_id = isset($params['item']->fields['entities_id'])
             ? $params['item']->fields['entities_id']
             : false; // false to use current entity
-         self::displayAlerts(['show_helpdesk_alerts' => true,
-                              'show_hidden_alerts'   => false,
-                              'entities_id'          => $entities_id
+         self::displayAlerts(['show_only_helpdesk_alerts'   => true,
+                              'show_hidden_alerts'          => false,
+                              'entities_id'                 => $entities_id
                              ]);
          echo "</br>";
       }

plugin.xml

@@ -53,6 +53,11 @@ Fonctionnalités
       <author>TECLIB'</author>
    </authors>
    <versions>
+      <version>
+         <num>1.8.2</num>
+         <compatibility>~9.5.0</compatibility>
+         <download_url>https://github.com/pluginsGLPI/news/releases/download/1.8.2/glpi-news-1.8.2.tar.bz2</download_url>
+      </version>
       <version>
          <num>1.8.1</num>
          <compatibility>~9.5.0</compatibility>

setup.php

@@ -21,7 +21,7 @@
  --------------------------------------------------------------------------
 */
 
-define ('PLUGIN_NEWS_VERSION', '1.8.1');
+define ('PLUGIN_NEWS_VERSION', '1.8.2');
 
 // Minimal GLPI version, inclusive
 define("PLUGIN_NEWS_MIN_GLPI", "9.5");