Bonjour,

Je pense que ce patch ne concerne que le travail que j'effectue en ce
moment sur le plugin GenericObject. Pour faire court : les fichiers
générés par le plugin vont dans le répertoire
files/_plugins/genericobject qui est normalement dédié à cet effet de
résolution de chemins 'front'.

Ainsi, les URLs générées par la classe Search sont toutes issues des
fonctions Toolbox::getItemTypeSearchURL() et
Toolbox::getItemTypeFormURL() au lieu des fonctions de la classe de
base CommonDBTM.

Malgré ma problématique, la majorité des fonctions du framework et des
plugins (inc et front confondus) consommant ce type d'URL utilise les
méthodes communes de CommonGLPI et je me dis que la classe Search
n'est pas une exception et que le patch proposé essaie de corriger.

PS: Mes développement actuels sur le plugin genericobject se trouve
sur https://github.com/kiniou/glpi-plugin-genericobject (branche
glpi0.85).

Cordialement,
--
Kevin Roy
Index: inc/search.class.php
===================================================================
--- inc/search.class.php	(revision 23099)
+++ inc/search.class.php	(working copy)
@@ -137,7 +137,7 @@
       $p['start']               = 0;//
       $p['is_deleted']          = 0;
       $p['export_all']          = 0;
-      $p['target']              = Toolbox::getItemTypeSearchURL($itemtype);
+      $p['target']              = $itemtype::getSearchURL();
       $p['display_type']        = self::HTML_OUTPUT;
       $p['list_limit']          = $_SESSION['glpilist_limit'];
       $p['massiveactionparams'] = array();
@@ -1688,7 +1688,7 @@
       $p['is_deleted']   = 0;
       $p['criteria']     = array();
       $p['metacriteria'] = array();
-      $p['target']       = Toolbox::getItemTypeSearchURL($itemtype);
+      $p['target']       = $itemtype::getSearchURL();
       $p['showreset']    = true;
       $p['showbookmark'] = true;
       $p['addhidden']    = array();
@@ -1768,7 +1768,10 @@
          }
 
          if ($p['showreset']) {
-            echo "<a href='".$p['target']."?reset=reset' >";
+            echo "<a href='"
+               .$p['target']
+               .(strpos($p['target'],'?') ? '&' : '?')
+               ."reset=reset' >";
             echo "&nbsp;&nbsp;<img title=\"".__s('Blank')."\" alt=\"".__s('Blank')."\" src='".
                   $CFG_GLPI["root_doc"]."/pics/reset.png' class='calendrier'></a>";
          }
@@ -4327,7 +4330,7 @@
                         $out .= $separate;
                      }
                      $count_display++;
-                     $page  = Toolbox::getItemTypeFormURL($linkitemtype);
+                     $page  = $linkitemtype::getFormUrl();
                      $page .= (strpos($page,'?') ? '&id' : '?id');
                      $name  = Dropdown::getValueWithUnit($data[$num][$k]['name'],$unit);
                      if ($_SESSION["glpiis_ids_visible"] || empty($data[$num][$k]['name'])) {
@@ -5832,4 +5835,4 @@
    }
 
 }
-?>
\ No newline at end of file
+?>
_______________________________________________
Glpi-dev mailing list
[email protected]
https://mail.gna.org/listinfo/glpi-dev

Reply via email to