In the module opigno_statisics_app.module, opigno_statistics_app_exit() throws undefined index on the SESSION variables lines 260-269. From looking at the code, that query's result doesn't do anything now that the $first_login code is commented out. So I have commented it out also and suggest if it will be re-enabled, that it is moved inside the isset() check for those session vars.

Comments

iLLin created an issue. See original summary.

iLLin’s picture

Project: Opigno LMS module (community version) » Opigno Statistics App
Version: 7.x-1.17 » 7.x-1.3
StatusFileSize
new1.01 KB

Moving issue to correct project and attaching a patch.

iLLin’s picture

Status: Active » Needs review

  • Amermod committed 5f902e6 on 7.x-1.x authored by iLLin
    Issue #2870411 by iLLin: Unused code causing undefined notices in...
amermod’s picture

Status: Needs review » Patch (to be ported)

Hi iLLin,

Yes, you are right. This part of code is not used anymore. I removed it and pushed it.

Thanks for spoting this !
Best regards,
Allan

amermod’s picture

Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.