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

Add support for core dynamic_form #115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add support for core dynamic_form #115

wants to merge 1 commit into from

Conversation

sumaiyamannan
Copy link
Contributor

@sumaiyamannan sumaiyamannan commented Jan 6, 2025

Hi,

This fixes the issue 104 and 112

To replicate the issue do the below:
Add multiblock plugin to a Moodle 4.2 site and add a block
Go to 'Manage contents' and click on the wheel icon to edit the block instance
Screenshot from 2025-01-06 17-26-38
It will lead to

Fatal error: Cannot override final method core_form\dynamic_form::__construct() in blocks/multiblock/classes/form/editblock_base.php on line 60

Apply the change and see that the issue is resolved

Note: This is for Moodle site 4.2 and above only.

Cheers,
Sumaiya

@sumaiyamannan sumaiyamannan force-pushed the issue104 branch 3 times, most recently from 6727e56 to 755734b Compare January 7, 2025 02:16
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.

1 participant