Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moodle 4.1 - PHP 8.1 | Work #241

Open
gemguardian opened this issue Dec 8, 2023 · 7 comments
Open

Moodle 4.1 - PHP 8.1 | Work #241

gemguardian opened this issue Dec 8, 2023 · 7 comments

Comments

@gemguardian
Copy link

@jleyva We just want to inform you that we will be working on a M4.1 / PHP 8.1 release on the Configurable Reports block. This work will be done by Luuk Verhoeven.

The forcus will be:

  • Test the plugin for 4.1 with PHP 8.1 and solve any issues they encounter
  • Solve security risks that have been detected
  • Add codecheck CI Validation, though because of some legacy pieces it does mean we have to skips some test.

What won't be done is:

  • Solve any issues that are mentioned on Github that have nothing to do with 4.1
  • Change any of the current functionalities
  • Remove features
  • Rewrite the full code so it's uptodate with Moodle coding styles
  • Add new librarys
  • Removing the jQuery code since it still works in 4.1, but please do realize this will have to be rewritten in the future.
  • Maintaining the further code.

We expect to be able to test the new version around January 12th 2023, when we feel comfortable enough we will create a pull request and hopefull you are willing to check (if necessary) and release this new branche/request to Moodle.org/plugins

In case you have any questions or feel like certain Issues mentioned in this tracker can be part of the "Focus will be" section, It would be great if you can add it this issue.

Greetings
Gemma Lesterhuis - Lesterhuis Training & Consultancy

@jleyva
Copy link
Owner

jleyva commented Dec 13, 2023

Hi Gemma,

that's great new, I will review the changes once submitted

@fabitas
Copy link

fabitas commented Jan 9, 2024

Wow, this is great. Question, because this is not clear from what won't be done "Maintaining the further code.". Will it be compatible with 4.3 too?
BTW, we are using it now in 4.1.
Thanks!

@gemguardian
Copy link
Author

@fabitas what we mean is that we won't be actively maintaining the code or adopting the plugin. So to answer your questions: if it works with Moodle 4.3 need to be tested by the users and issues should be reported in this github.

We are right now testing 4.1 version

@gemguardian
Copy link
Author

@jleyva we have finished the work for Moodle 4.1 , php 8.1 version. We are currently testing the plugin with a small community group. We are expecting to hand over the code to you at the end of February so you can do a follow up check and release it to Moodle.org , would that work for you?

@gemguardian
Copy link
Author

@jleyva I just send a pull request. Even though we tested it, I think a code review would be a good idea before you merge it and release it to the community

if there are any questions please let me know.

@jleyva
Copy link
Owner

jleyva commented Mar 5, 2024

Thanks, I'm reviewing this but it is a huge patch! So it will take some time

@luisdev
Copy link

luisdev commented May 31, 2024

@jleyva I just send a pull request. Even though we tested it, I think a code review would be a good idea before you merge it and release it to the community

if there are any questions please let me know.

Are your changes incorporated in the new 4.1.0 release (https://github.com/jleyva/moodle-block_configurablereports/releases/tag/4.1.0)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants