Skip to content

Feature - GLPI 11.0 Compatibility#89

Merged
stonebuzz merged 1 commit intomainfrom
feature/glpi-11.0
Sep 16, 2025
Merged

Feature - GLPI 11.0 Compatibility#89
stonebuzz merged 1 commit intomainfrom
feature/glpi-11.0

Conversation

@stonebuzz
Copy link
Copy Markdown
Contributor

No description provided.

@stonebuzz stonebuzz force-pushed the feature/glpi-11.0 branch 7 times, most recently from 8806cff to db064a5 Compare June 24, 2025 09:13
@stonebuzz stonebuzz requested a review from Rom1-B June 24, 2025 09:13
@stonebuzz stonebuzz self-assigned this Jun 24, 2025
@stonebuzz stonebuzz force-pushed the feature/glpi-11.0 branch from db064a5 to 815d509 Compare June 24, 2025 09:17
@stonebuzz stonebuzz force-pushed the feature/glpi-11.0 branch from a39b073 to 569b05b Compare June 24, 2025 10:30
@Rom1-B Rom1-B self-assigned this Jun 24, 2025
@stonebuzz stonebuzz force-pushed the feature/glpi-11.0 branch 3 times, most recently from 98dc41b to e7a6ba9 Compare June 25, 2025 10:27
@Rom1-B Rom1-B force-pushed the feature/glpi-11.0 branch from e7a6ba9 to 78ba986 Compare July 17, 2025 10:25
@seblveb
Copy link
Copy Markdown

seblveb commented Aug 19, 2025

Hello,
I noticed for GPLI 11 that the plugin treeview has invalid relative includes in its public files.
In both files:
marketplace/treeview/public/index.php
marketplace/treeview/public/left.php

the following line is used:

include(../../inc/includes.php);

This fails with the warning:

PHP Warning: Failed to open stream: No such file or directory

Suggested fix: update the include path to be relative to the current file, for example:
include(../../../inc/includes.php');

This way, the plugin works properly when placed in marketplace/treeview.
Thanks!

@stonebuzz
Copy link
Copy Markdown
Contributor Author

Hi @seblveb

Thank you for your feedback, the includes are no longer useful and I have removed them.

@Rom1-B Rom1-B force-pushed the feature/glpi-11.0 branch 2 times, most recently from c1780c8 to 01b0078 Compare August 20, 2025 08:02
@stonebuzz stonebuzz force-pushed the feature/glpi-11.0 branch 2 times, most recently from cfa94ea to b549130 Compare September 16, 2025 08:08
@stonebuzz stonebuzz merged commit 3c435c0 into main Sep 16, 2025
3 checks passed
@stonebuzz stonebuzz deleted the feature/glpi-11.0 branch September 16, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants