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

Cannot select 'Passing grade' as the criteria for Default activity completion #12

Closed
drachels opened this issue Nov 2, 2023 · 3 comments

Comments

@drachels
Copy link

drachels commented Nov 2, 2023

In Moodle 4.3 with Podcast version "4.2 (2023041800)" with debug turned on, you should see the following along with possibly some others when you go to a course More > Completion settings and then Default activity completion:

Custom completion rule completionepisodesgroup of module pcast has wrong suffix and has been removed from the form. This has to be fixed by the developer

line 126 of /completion/classes/edit_base_form.php: call to debugging()
line 146 of /completion/classes/edit_base_form.php: call to core_completion_edit_base_form->add_custom_completion()
line 178 of /completion/classes/form/form_trait.php: call to core_completion_edit_base_form->add_completion_rules()
line 247 of /completion/classes/edit_base_form.php: call to core_completion_edit_base_form->add_completion_elements()
line 118 of /completion/classes/defaultedit_form.php: call to core_completion_edit_base_form->definition()
line 217 of /lib/formslib.php: call to core_completion_defaultedit_form->definition()
line 45 of /completion/classes/defaultedit_form.php: call to moodleform->__construct()
line 89 of /course/classes/output/bulk_activity_completion_renderer.php: call to core_completion_defaultedit_form->__construct()
line 97 of /course/defaultcompletion.php: call to core_course_bulk_activity_completion_renderer->defaultcompletion()
@drachels
Copy link
Author

drachels commented Nov 2, 2023

I had this same problem with my own plugins and had to add the suffix's to the public function add_completion_rules in the mod_form.php file. So far I've done HotQuestion and MooTyper. Still trying to decide if I need extra completion requirements for Diary.

@sbourget
Copy link
Owner

sbourget commented Nov 2, 2023

Thanks for the report and the fix. I've pushed the fix, and once GHA has confirmed that I didn't break anything in Behat, I'll publish a new version to the Plugins DB.

@drachels
Copy link
Author

drachels commented Nov 2, 2023

I wish I could work that fast. My biggest problem is that the PHP language was not even invented when I was going to college and learning how to program in about a dozen programming languages. Seems to take me forever, for instance when I was fixing suffix's I think it took me two days to do them for HotQuestion and MooTyper and I am dragging my feet when it comes to the Diary plugin, because I also need to add five more custom completion settings, plus some other stuff.

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

2 participants