Adoption of Timestat block and MindMap course format

Adoption of Timestat block and MindMap course format

od Łukasz Sanokowski -
Število odgovorov: 13

Hello All

Since almost year I don't use Moodle any more.

I am willing to give following plugins:

https://moodle.org/plugins/block_timestat
https://moodle.org/plugins/format_mindmap

Into good hands nasmeh

They are not shared in any git repo. A new maintainer should simply download packages and make initial commit using them.

Povprečje ocenitev:Useful (2)
V odgovor na Łukasz Sanokowski

Re: Adoption of Timestat block and MindMap course format

od Virasat Solutions -
Hello Lukasz,

I am looking forward to maintaining this plugin. As first things would like to upgrade it to the latest moodle version. Let me know when can we discuss about it.


Virasat
V odgovor na Virasat Solutions

Odp: Re: Adoption of Timestat block and MindMap course format

od Łukasz Sanokowski -
Hi!

At some point of time (I don't remember which Moodle version it was) there was introduced new logging API. The previous logging mechanism was named "Legacy log" and it is only log mechanism supported by this plugin. You need to start from there.
V odgovor na Łukasz Sanokowski

Re: Odp: Re: Adoption of Timestat block and MindMap course format

od Virasat Solutions -
Oky, I will review and get back to you
V odgovor na Łukasz Sanokowski

Re: Adoption of Timestat block and MindMap course format

od Jorge C. -
Slika Plugin developers
Hello Łukasz,

I am interested in maintaining this plugin (block_timestat). I have experience working with the new Logging API, so I would like to adopt this plugin and update it to the latest Moodle version.
V odgovor na Jorge C.

Re: Adoption of Timestat block and MindMap course format

od Dan Marsden -
Slika Core developers Slika Particularly helpful Moodlers Slika Peer reviewers Slika Plugin developers Slika Plugins guardians Slika Testers Slika Translators
Hi Jorge - thanks for offering to take over maintenance! - a good place to start with would be to drop the current code into your own public git repository (github/gitlab etc) - I note that Łukasz doesn't appear to have this in a github repository anymore so you should make the first commit in your repository with his code and also manually modify that commit so that the author is set to Łukasz rather than yourself for that very first commit. Then add some further commits on top of that original code to get it working in a newer release and post the link here to your github work in progress - that will show us that you have the skills and motivation to take on maintenance longer-term and we can then look at assigning the moodle.org plugins db entry to you.
V odgovor na Dan Marsden

Re: Adoption of Timestat block and MindMap course format

od Jorge C. -
Slika Plugin developers
Hi Dan,

Thank you very much for the detailed instructions. I have uploaded the plugin to the following repository: https://github.com/jcorel/block_timestat
I have made the minimal changes to make the plugin work with Moodle 3.11, but of course there is a lot of work to be done to upgrade the plugin to Moodle standards - specifically regarding the Logging API.
V odgovor na Jorge C.

Re: Adoption of Timestat block and MindMap course format

od David Mudrák -
Slika Core developers Slika Documentation writers Slika Moodle HQ Slika Particularly helpful Moodlers Slika Peer reviewers Slika Plugin developers Slika Plugins guardians Slika Testers Slika Translators

Many thanks Jorge for the offer and for the update! Please note the comment I left at https://github.com/jcorel/block_timestat/commit/ac4b4111e46cc0bef3e8164c3783e9b1c4943e2e#r69654101 - direct inclusion of passed arguments in SQL is not safe and should not be needed in this case.

V odgovor na David Mudrák

Re: Adoption of Timestat block and MindMap course format

od Jorge C. -
Slika Plugin developers
Thank you for taking the time to review it, David.

I have updated the repository to include your suggestion.
V odgovor na Łukasz Sanokowski

Re: Adoption of Timestat block and MindMap course format

od Jorge C. -
Slika Plugin developers
Hello everyone,

I have updated the repository with important changes. At this moment the plugin is fully compatible with Moodle 3.11. Could someone check it?

Important changes still need to be made to adapt the plugin to Moodle 3.11 standard (for example, regarding AMD modules) but I consider that an acceptable first goal is to get the plugin to do its job in this version.
V odgovor na Jorge C.

Re: Adoption of Timestat block and MindMap course format

od Dan Marsden -
Slika Core developers Slika Particularly helpful Moodlers Slika Peer reviewers Slika Plugin developers Slika Plugins guardians Slika Testers Slika Translators
Thanks Jorge! - I've just re-assigned the moodle.org plugins db entry to yourself and updated the source control and tracker links in the plugin entry too.

Can you please upload the new version to the plugins db entry:
https://moodle.org/plugins/block_timestat

(you should now have access the developer zone link which has an "add new version" button)

Then once you've done that - hit the "request re-approval" button on that same page and it will get added to our queue for review.

thanks!
.*Edit* - I took a very quick look at your code in your plugins repo and reported a few issues - not all are blockers for approval though, so feel free to leave the non-blockers and just sort out the ones I've flagged as blockers before you send through for us to take a look.