wjunkin / moodle-quiz_liveviewgrid

Dynamic quiz spreadsheet
2 stars 2 forks source link

Warning: count(): Parameter must be an array #22

Open RoyalZefa opened 4 years ago

RoyalZefa commented 4 years ago

Hellow, this is great plugin and it help us to create nice open lessons. But we have error on moodle v. 3.9: Warning: count(): Parameter must be an array or an object that implements Countable in /var/www/moodle/mod/quiz/report/liveviewgrid/report.php on line 633 It pop up on page every time while screen updates. Thanks a lot, if we need to change somthing in code it will be nice too. image

wjunkin commented 4 years ago

Thanks, RoyalZefa, for finding this bug. (And thanks, also for your kind words about this plugin.) I am unable to duplicate this error, so if you can provide a little more information, I would appreciate it. The version of Live Report here (in github) is v1.2.43 (2020062200). When I examine the code for this version, I do not find any use of the count() function on line 633 of /liveviewgrid/report.php code. So I suspect that you are using a different version of this plugin. If that is the case, if you upgrade to the version on github and then let me know if the error still persists, I will greatly appreciate it. However, if you are using the version v1.2.43 (2020062200) of this plugin or don't want to upgrade the plugin at this time, if you will send me a copy of the /liveviewgrid/report.php code that you are using I hope I will be able to correct this bug very quickly. Thanks.

RoyalZefa commented 4 years ago

I update it to version what you told, before i have older version. I still got a error, but error change a litle bit. Now it point to line 763. For file, i can provide it on next Day(right now cant acsess server). I have moodle theme fordson and russian language installed. Screenshot_20200913_002058

wjunkin commented 4 years ago

Hi, RoyalZefa, Thanks for doing the update of this plugin. I think I have been able to duplicate your error. I am now working to fix the bug. If you would like to communicate with me directly, my email is junkinwf@eckerd.edu