Forum GLPI-Project

J2C99

Member

Registered: 2020-08-11

Posts: 112

[GLPI 9.5.2/Appliances 3.1.0] Divers Bugs.

Bonjour

  Voici ma liste:

[BUG]     * Non prise en compte de  quotes  dans les champs  comment et name .
Dans le  fichier plugins/appliances/hook.php
Ligne 192 :       '".$data['comment']."', '".$data['locations_id']."',
    A remplacer par
            '".addslashes($data['comment'])."', '".$data['locations_id']."',

Lignes 257 :    '".$data['comment']."', '".$data['locations_id']."',
    A remplacer par
               '".addslashes($data['comment'])."', '".$data['locations_id']."',
               
Ligne 311 :      '".$data['comment']."')";
    A remplacer par
               '".addslashes($data['comment'])."')";

Ligne 189:      '".$data['is_recursive']."', '".$data['name']."',
    A remplacer par
                     '".$data['is_recursive']."', '".addslashes($data['name'])."',

[BUG]     * SQL Error Syntax (manque le ')' de fermeture)
Dans le  fichier plugins/appliances/hook.php
Ligne  153         VALUES ('".$pref['num']."', '".$pref['rank']."', '".$pref['users_id']."'";
      A remplacer par
                        VALUES ('".$pref['num']."', '".$pref['rank']."', '".$pref['users_id']."')";
               

[BUG]   * Table glpi_appliancerelations inexistante.
         A partir de la 9.5.2 , la table  glpi_appliancerelations a été renommé en 'glpi_appliances_items_relations'
         Le repport n'a pas été fait dans le plugin appliance en 3.1.0.
Dans le  fichier plugins/appliances/hook.php
Ligne  275:        If (countElementsInTable('glpi_appliancerelations') > 0) {
         A remplacer par
                          If (countElementsInTable('glpi_appliances_items_relations') > 0) {

Ligne  284:        $queryrel = "INSERT INTO `glpi_appliancerelations`
         A remplacer par
                        $queryrel = "INSERT INTO `glpi_appliances_items_relations`

[BUG]   * Pertes de certains droits au niveau des profils.
          En cours d'analyse, pas de correctifs
hooks.php [lgn:135]
       // migration rights
      foreach ($DB->request(['FROM'   => 'glpi_profilerights',
                             'WHERE'  => ['name' => 'plugin_appliances']]) AS $right) {
         $queryright = "UPDATE `glpi_profilerights`
                        SET `rights` = '".$right['rights']."'
                        WHERE `profiles_id` = '".$right['profiles_id']."'";
         $DB->query($queryright);
      }

      $delqueryright = "DELETE FROM `glpi_profilerights`
                        WHERE `name` LIKE 'plugin_appliances%' ";
      $DB->query($delqueryright);

  Je ne comprend pas la raison du  update . A la lecture de celui-ci , on fait un update d'une table avec les mêmes valeurs.
Cela n'a pas de sens.

Cordialement
J2C

Last edited by J2C99 (2020-10-23 11:20:11)

yllen

GLPI-DEV

From: Sillery (51)

Registered: 2008-01-14

Posts: 15,278

Re: [GLPI 9.5.2/Appliances 3.1.0] Divers Bugs.

Merci de votre retour.
Ticket ouvert sur la forge du plugin : https://forge.glpi-project.org/issues/5611 et corrigé.

Concernant les droits des profils l'update doit être fait car avant le droit se nommait "plugin_appliance' et maintenant il se nomme 'appliance'

---

CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

J2C99

Member

Registered: 2020-08-11

Posts: 112

Re: [GLPI 9.5.2/Appliances 3.1.0] Divers Bugs.

Dans , ce cas ,ceci ne serait il pas plus judicieux ?

      foreach ($DB->request(['FROM'   => 'glpi_profilerights',
                             'WHERE'  => ['name' => 'plugin_appliances']]) AS $right) {
         $queryright = "UPDATE `glpi_profilerights`
                        SET `rights` = '".$right['rights']."'
                        WHERE `profiles_id` = '".$right['profiles_id']."'";
         $DB->query($queryright);
      }

      $delqueryright = "DELETE FROM `glpi_profilerights`
                        WHERE `name` LIKE 'plugin_appliances%' ";
      $DB->query($delqueryright);

A remplacer par

         $queryright = "UPDATE `glpi_profilerights`
                        SET   name = 'appliance'                                                           
                        WHERE name= 'plugin_appliances'";
       $DB->query($queryright);

Malheureusement si j'execute ,   j'ai l'erreur suivante
    ERROR 1062 (23000): Duplicate entry '1-appliance' for key 'unicity'

Last edited by J2C99 (2020-10-23 18:32:53)

J2C99

Member

Registered: 2020-08-11

Posts: 112

Re: [GLPI 9.5.2/Appliances 3.1.0] Divers Bugs.

Cela me semble plus étrange ,  je n'ai pas de doublon dans la table en question
  Je penche à un bug de mariadb .

Pour contourner le probleme , je recrée l'index.
ALTER TABLE glpi_profilerights DROP  KEY `unicity` ;
ALTER TABLE glpi_profilerights ADD   KEY `unicity` (`profiles_id`,`name`) ;

ET CA MARCHE ....

Last edited by J2C99 (2020-10-23 18:42:30)