Error accessing the logs

izendegi commented 3 years ago

Just acknowledging that there is an issue with this plugin when accesing the course logs on Moodle 3.11:

https://moodle.org/mod/forum/discuss.php?d=423173

I've experienced it and as a workaround I have tried to disable the block but it hasn't solved it, so I've uninstalled the block.

marceloschmitt commented 3 years ago

I have a new version. I will test this issue before publishing it. I hope you can help me solve it.

Marcelo Augusto Rauh Schmitt @.***

Em 2 de set de 2021, à(s) 11:17, Iñigo Zendegi @.***> escreveu:

Just acknowledging that there is an issue with this plugin when accesing the course logs on Moodle 3.11:

https://moodle.org/mod/forum/discuss.php?d=423173 https://moodle.org/mod/forum/discuss.php?d=423173 I've experienced it and as a workaround I have tried to disable the block but it hasn't solved it, so I've uninstalled the block.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/marceloschmitt/moodle-block_analytics_graphs/issues/41, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGPGZ3G6DN3ST3M2FZH5JDT76BRJANCNFSM5DJHIELQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

izendegi commented 3 years ago

Sure, tell me when it's ready and I'll test it.

izendegi commented 3 years ago

I've seen you've published a new version of the plugin at moodle.org and tried to install it but I get this error:

syntax error, unexpected '$plugin' (T_VARIABLE) Debug: Error code: generalexceptionmessage

line 24 of /blocks/analytics_graphs/version.php: ParseError thrown
line 1104 of /lib/classes/component.php: call to core_component::get_all_versions()
line 8104 of /lib/moodlelib.php: call to core_component::get_all_versions_hash()
line 60 of /index.php: call to moodle_needs_upgrading()

marceloschmitt commented 3 years ago

I make a mistake with the releases version. Ridiculous. Sorry. Fixed.

izendegi commented 3 years ago

Yes, with the new version I can install the plugin correctly, but when doing it the error with the log report happens again so it seems like the original issue isn't fixed yet. :-(

marceloschmitt commented 3 years ago

Ok, Izendigi. Let's solve it now, ok? Tell me exactly what happens. You installed the new plugin. Then in what situation the error occurs? Can you talk to me using marcelo.schmit@poa.ifrs.edu.br? It may be faster to solve the issue.

izendegi commented 3 years ago

To reproduce the issue, once the block is installed, I just have to navigate to one course's activity log report (doesn't matter if the course has the block added or not): yourmoodleurl.org/report/log/index.php?id=99999

https://docs.moodle.org/311/en/Logs#Logs_of_course_activity

Doing that the report doesn't show up, and instead of that I get this message: Exception - Call to a member function out() on null

And I've found this message on the PHP log: Error code: generalexceptionmessage

line 54 of /lib/classes/report_helper.php: Error thrown
line 196 of /report/log/index.php: call to core\report_helper::print_report_selector()

P.S.: I've tried to send this by email but the mail server answered that the email account does not exist

marceloschmitt commented 3 years ago

I think it is ok now in 4.2.1 release. Thanks.

izendegi commented 3 years ago

Yes, the issue is gone on 4.2.1, thank you @marceloschmitt

marceloschmitt / moodle-block_analytics_graphs

Error accessing the logs #41