vadimonus/moodle-report_extendedlog

moodle-report_extendedlog's Issues

Help strings can use Markdown format

This is not a bug, just a suggestion. All the help strings in moodle (those that define the text of the help popups) are implicitly considered to be Markdown formatted. You can make use of it for strings like filter_ip4_help. From the Moodle translators' perspective, translating Markdown formatted texts is easier than having to deal with HTML tags for paragraphs, lists etc.

Offers to use disabled log stores

When displaying the list of available log stores in report_extendedlog\filter_form::definition(), the report calls

$logreaders = get_log_manager()->get_supported_logstores('report_extendedlog');

I realised that the method get_supported_logstores() returns the external database log store, even if that one is not enabled on the site. So I can select it. But then in index.php, the code

$manager = get_log_manager();
$readers = $manager->get_readers();
$logreader = $readers[$pageparams['logreader']];

starts to throw errors because the method get_readers() does not contain the selected log store.

It is not fully clear if the behaviour of get_supported_logstores() is a Moodle core feature or a bug. After having this discussed in the moodle dev jabber chat, I tend to think this is a feature and expected behaviour. To get the list for this field, you actually want intersection of "supported" and "enabled" log stores.

Add "course id" field to the report

Hello
Congratulations and thank you for this wonderful plugin.
It has been very useful for me to provide information to the application that I am developing: https://moodle-py.herokuapp.com (in Spanish for now)
There is only something that would be helpful to add, the course id, since there are global analyzes that I need the course id to make analysis and comparisons, for example, courses within a category.
Thank you in advance for your help.

Moodle 4 support

Hi,
Could you please add support for Moodle 4?
Even if the current version of the plugin would work on Moodle 4, an official support would be great.

Thanks even for considering it :-)