Skip to content

Commit 4facc33

Browse files
Mael AlleMael Alle
authored andcommitted
Fix a bug that ignored read and write permissions on profiles
1 parent f4a8964 commit 4facc33

3 files changed

Lines changed: 27 additions & 12 deletions

File tree

ajax/config.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,15 @@
2121
exit;
2222
}
2323

24+
// Check if user has READ right on the plugin
25+
if (!Session::haveRight('plugin_wikitsemanticschat_config', READ)) {
26+
echo json_encode([
27+
'success' => false,
28+
'error' => 'Access denied',
29+
]);
30+
exit;
31+
}
32+
2433
// Get configuration
2534
$config = Config::getConfig();
2635

setup.php

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
use GlpiPlugin\Wikitsemanticschat\Config;
99
use GlpiPlugin\Wikitsemanticschat\Profile;
1010

11-
define('PLUGIN_WIKITSEMANTICSCHAT_VERSION', '1.0.0');
11+
define('PLUGIN_WIKITSEMANTICSCHAT_VERSION', '1.0.1');
1212
// Minimal GLPI version, inclusive
1313
define('PLUGIN_WIKITSEMANTICSCHAT_MIN_GLPI_VERSION', '10.0.0');
1414
// Maximum GLPI version, exclusive
@@ -33,25 +33,26 @@ function plugin_init_wikitsemanticschat(): void {
3333
return;
3434
}
3535

36-
// Add JavaScript on all pages for logged-in users only
36+
// Register plugin classes (always needed for profile management)
3737
if (Session::getLoginUserID()) {
38-
// Load plugin configuration and main script
39-
$PLUGIN_HOOKS[Hooks::ADD_JAVASCRIPT]['wikitsemanticschat'] = [
40-
'public/js/config.js.php',
41-
'public/js/wikitsemanticschat.js'
42-
];
43-
44-
// Register plugin classes
4538
Plugin::registerClass(Config::class);
4639
Plugin::registerClass(
4740
Profile::class,
4841
['addtabon' => 'Profile']
4942
);
5043

5144
// Configuration page in Setup > Plugins
52-
if (Session::haveRight('config', UPDATE)) {
53-
$PLUGIN_HOOKS['config_page']['wikitsemanticschat'] = 'front/config.form.php';
54-
}
45+
if (Session::haveRight('config', UPDATE)) {
46+
$PLUGIN_HOOKS['config_page']['wikitsemanticschat'] = 'front/config.form.php';
47+
}
48+
49+
// Add JavaScript only for users with READ right
50+
if (Session::haveRight('plugin_wikitsemanticschat_config', READ)) {
51+
$PLUGIN_HOOKS[Hooks::ADD_JAVASCRIPT]['wikitsemanticschat'] = [
52+
'public/js/config.js.php',
53+
'public/js/wikitsemanticschat.js'
54+
];
55+
}
5556
}
5657
}
5758

wikitsemanticschat.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,11 @@
9292
<compatibility>~11.0.0</compatibility>
9393
<download_url>https://github.com/wikit-ai/GLPI-Semantics-Chat/releases/download/v2.0.0/wikitsemanticschat-2.0.0.tar.bz2</download_url>
9494
</version>
95+
<version>
96+
<num>1.0.1</num>
97+
<compatibility>~10.0.0</compatibility>
98+
<download_url>https://github.com/wikit-ai/GLPI-Semantics-Chat/releases/download/v1.0.1/wikitsemanticschat-1.0.1.tar.bz2</download_url>
99+
</version>
95100
<version>
96101
<num>1.0.0</num>
97102
<compatibility>~10.0.0</compatibility>

0 commit comments

Comments
 (0)