Adoption of Timestat block and MindMap course format

Adoption of Timestat block and MindMap course format

par Łukasz Sanokowski,
Nombre de réponses : 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 sourire

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

Moyenne des évaluations Useful (2)
En réponse à Łukasz Sanokowski

Re: Adoption of Timestat block and MindMap course format

par 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
En réponse à Virasat Solutions

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

par Ł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.
En réponse à Łukasz Sanokowski

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

par Virasat Solutions,
Oky, I will review and get back to you
En réponse à Virasat Solutions

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

par Alexis Saturno,

Hi, there are Any advance in the actualization of the plugin to newest versions?

En réponse à Łukasz Sanokowski

Re: Adoption of Timestat block and MindMap course format

par Jorge C.,
Avatar 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.
Moyenne des évaluations Useful (1)
En réponse à Jorge C.

Re: Adoption of Timestat block and MindMap course format

par Dan Marsden,
Avatar Core developers Avatar Particularly helpful Moodlers Avatar Peer reviewers Avatar Plugin developers Avatar Plugins guardians Avatar Testers Avatar 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.
Moyenne des évaluations Useful (5)
En réponse à Dan Marsden

Re: Adoption of Timestat block and MindMap course format

par Jorge C.,
Avatar 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.
Moyenne des évaluations Useful (1)
En réponse à Jorge C.

Re: Adoption of Timestat block and MindMap course format

par David Mudrák,
Avatar Core developers Avatar Documentation writers Avatar Moodle HQ Avatar Particularly helpful Moodlers Avatar Peer reviewers Avatar Plugin developers Avatar Plugins guardians Avatar Testers Avatar 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.

Moyenne des évaluations Useful (1)
En réponse à David Mudrák

Re: Adoption of Timestat block and MindMap course format

par Jorge C.,
Avatar Plugin developers
Thank you for taking the time to review it, David.

I have updated the repository to include your suggestion.
En réponse à Łukasz Sanokowski

Re: Adoption of Timestat block and MindMap course format

par Jorge C.,
Avatar 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.
En réponse à Jorge C.

Re: Adoption of Timestat block and MindMap course format

par Dan Marsden,
Avatar Core developers Avatar Particularly helpful Moodlers Avatar Peer reviewers Avatar Plugin developers Avatar Plugins guardians Avatar Testers Avatar 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.