Skip to content

馃悰 Migrate before_footer callback to Moodle hook - #10

Merged
DamienVauchel merged 1 commit into
mainfrom
bugfix/EDFLEX1-13863
Jul 8, 2026
Merged

馃悰 Migrate before_footer callback to Moodle hook#10
DamienVauchel merged 1 commit into
mainfrom
bugfix/EDFLEX1-13863

Conversation

@DamienVauchel

Copy link
Copy Markdown
Collaborator

馃搵 Summary

  • The legacy before_footer plugin callback triggers a deprecation notice on Moodle 4.4+ (core\hook\output\before_footer_html_generation->process_legacy_callbacks).
  • Register the hook in db/hooks.php so the notice is suppressed and the callback runs through the new hook API.
  • Keep mod_edflex_before_footer() (now returning its HTML) as a fallback for Moodle < 4.4.

馃И Manual checks

  • On a Moodle 4.5 site, open a SCORM activity with an edflex mapping: the edflex block still renders and no "should be migrated to new hook callback" debug notice appears.

- Register before_footer_html_generation hook to silence the 4.4+
  deprecation notice; legacy lib.php callback kept for Moodle < 4.4.
@DamienVauchel DamienVauchel self-assigned this Jul 6, 2026
@DamienVauchel
DamienVauchel marked this pull request as ready for review July 6, 2026 14:50
@DamienVauchel
DamienVauchel merged commit df62a6e into main Jul 8, 2026
12 checks passed
@DamienVauchel
DamienVauchel deleted the bugfix/EDFLEX1-13863 branch July 8, 2026 08:38
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

Successfully merging this pull request may close these issues.

2 participants