Skip to content

Fix: front/ check#29

Merged
Rom1-B merged 2 commits intopluginsGLPI:mainfrom
Rom1-B:front_check
May 5, 2025
Merged

Fix: front/ check#29
Rom1-B merged 2 commits intopluginsGLPI:mainfrom
Rom1-B:front_check

Conversation

@Rom1-B
Copy link

@Rom1-B Rom1-B commented May 2, 2025

Add missing Session::checkLoginUser();

@Rom1-B Rom1-B requested a review from stonebuzz May 2, 2025 10:01
Comment on lines 35 to 36
Session::checkLoginUser();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Session::checkLoginUser();
Session::checkLoginUser();
Session::checkRight('plugin_pdf', READ)

front/export.php Outdated
Comment on lines 38 to 39
Session::checkLoginUser();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Session::checkLoginUser();
Session::checkLoginUser();
Session::checkRight('plugin_pdf', READ)

@Rom1-B Rom1-B requested a review from stonebuzz May 5, 2025 07:53
@Rom1-B Rom1-B merged commit 7131185 into pluginsGLPI:main May 5, 2025
3 checks passed
@Rom1-B Rom1-B deleted the front_check branch May 5, 2025 08:00
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.

2 participants