Revert "Fix tu deploy (#724)"

This reverts commit 938c71a.

Author: trasher

hook.php

@@ -1,12 +1,10 @@
 <?php
 
-use Glpi\DBAL\QueryExpression;
 use Glpi\Error\ErrorHandler;
 use Safe\Exceptions\InfoException;
 
 use function Safe\ini_set;
 use function Safe\json_decode;
-
 /**
  * ---------------------------------------------------------------------
  * GLPI Inventory Plugin
@@ -38,39 +36,6 @@
  * ---------------------------------------------------------------------
  */
 
-function plugin_glpiinventory_add_default_where($in)
-{
-    [$itemtype, $condition] = $in;
-
-    if ($itemtype === Computer::class && str_contains($_SERVER['REQUEST_URI'] ?? '', 'deploypackage.public.php')) {
-        // Specific case related to Self-Service profile rights:
-        // - As Computer object has no read permission for Self-Service profile,
-        //   Computer::getAssignableVisiblityCriteria() returns a QueryExpression("0").
-        // - This leads to a "WHERE (0)" clause in the SQL query, which discards
-        //   all results (e.g. no deployment packages shown in Self-Service).
-        // - To fix this in the plugin, we explicitly filter out QueryExpression
-        //   instances with value "0", while keeping all other valid criteria
-        //   generated by GLPI.
-        removeEmptyCondition($condition);
-    }
-
-    return [$itemtype, $condition];
-}
-
-function removeEmptyCondition(array &$conditions): void
-{
-    foreach ($conditions as $key => $condition) {
-        if (is_array($condition)) {
-            removeEmptyCondition($condition);
-            if (empty($condition)) {
-                unset($conditions[$key]);
-            }
-        } elseif ($condition instanceof QueryExpression && $condition->getValue() == 0) {
-            unset($conditions[$key]);
-        }
-    }
-}
-
 /**
  * Add search options for GLPI objects
  *

setup.php

@@ -375,8 +375,6 @@ function plugin_init_glpiinventory()
     $PLUGIN_HOOKS[Hooks::HANDLE_ESX_TASK]['glpiinventory'] = 'plugin_glpiinventory_handle_esx_task';
     $PLUGIN_HOOKS[Hooks::HANDLE_COLLECT_TASK]['glpiinventory'] = 'plugin_glpiinventory_handle_collect_task';
     $PLUGIN_HOOKS[Hooks::HANDLE_DEPLOY_TASK]['glpiinventory'] = 'plugin_glpiinventory_handle_deploy_task';
-
-    $PLUGIN_HOOKS[Hooks::ADD_DEFAULT_WHERE]['glpiinventory'] = 'plugin_glpiinventory_add_default_where';
 }
 
 

tests/Integration/Deploy/PackageSelfDeployTest.php

@@ -409,19 +409,6 @@ public function testReportMyPackage()
         // Prepare task
         PluginGlpiinventoryTask::cronTaskscheduler();
 
-        $_SERVER['REQUEST_URI'] = 'front/deploypackage.php'; // URL is used to fix addDefaultWhere
-        $packages = $pfDeployPackage->getPackageForMe($userId);
-        $packages_deploy = [];
-        foreach ($packages as $data) {
-            foreach ($data as $package_info) {
-                if (isset($package_info['taskjobs_id'])) {
-                    $packages_deploy[] = $package_info['last_taskjobstate']['state'];
-                }
-            }
-        }
-        $this->assertEquals([], $packages_deploy);
-
-        $_SERVER['REQUEST_URI'] = 'front/deploypackage.public.php'; // URL is used to fix addDefaultWhere
         $packages = $pfDeployPackage->getPackageForMe($userId);
         $packages_deploy = [];
         foreach ($packages as $data) {