Skip to content

Update doublons.php - Fix GLPIv11#2

Open
ilionel wants to merge 2 commits into
InfotelGLPI:masterfrom
ilionel:master
Open

Update doublons.php - Fix GLPIv11#2
ilionel wants to merge 2 commits into
InfotelGLPI:masterfrom
ilionel:master

Conversation

@ilionel
Copy link
Copy Markdown

@ilionel ilionel commented May 15, 2026

Hello,

doublon.php
In getLastInventory(), the WHERE clause incorrectly used computers_id as the column name when querying glpi_computers, but the primary key of that table is id — computers_id is the foreign key column name specific to the OCS plugin's glpi_plugin_ocsinventoryng_ocslinks table.

profile.php
In GLPI 11's Symfony routing, $_SERVER['PHP_SELF'] always resolves to /index.php (the front controller), so using it as a form action caused the POST to be intercepted by CatchInventoryAgentRequestListener — which expects inventory agent XML — instead of being handled by the reports plugin; replacing it with $_SERVER['REQUEST_URI'] ensures the form submits to the actual requested URL.

Lionel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant