Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undefined index when installing the module #78

Open
GregMage opened this issue Oct 25, 2020 · 5 comments
Open

Undefined index when installing the module #78

GregMage opened this issue Oct 25, 2020 · 5 comments

Comments

@GregMage
Copy link

Notification: Undefined index: xoopsModule dans le fichier /modules/newbb/class/PermissionCategoryHandler.php ligne 80
Notification: Undefined index: xoopsModule dans le fichier /modules/newbb/class/PermissionForumHandler.php ligne 231

@mambax7
Copy link
Contributor

mambax7 commented Oct 25, 2020

What PHP and XOOPS versions?

@GregMage
Copy link
Author

PHP 7.4 and xoops 2.5.11 (last version).
No messages if you install the module from the module manager.
The messages have appeared if you install xoops 2.5.11 and request the installation of the module during the xoops 2.511 installation process.

@mambax7
Copy link
Contributor

mambax7 commented Oct 25, 2020

Can you change both lines by adding a check:

isset($GLOBALS['xoopsModule'])

so it begin of the line looks like this:

if (isset($GLOBALS['xoopsModule']) && 

and the whole line:

if (isset($GLOBALS['xoopsModule']) && \is_object($GLOBALS['xoopsModule']) && 'newbb' === $GLOBALS['xoopsModule']->getVar('dirname')) {

and let us know if it helped?

@GregMage
Copy link
Author

I just report the errors, it is not a problem for the installation of the module. This will need to be corrected in the next version.

@mambax7
Copy link
Contributor

mambax7 commented Oct 26, 2020

But can you check if this code above fixed the issue?

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

No branches or pull requests

2 participants