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

Replace VALUE_OPTIONAL with VALUE_DEFAULT in get_chapters_parameters() #40

Open
vithushau opened this issue May 23, 2024 · 0 comments
Open

Comments

@vithushau
Copy link

This issue addresses a bug where the VALUE_OPTIONAL parameter in the get_chapters_parameters() function was causing errors during unit testing.

core\externallib_test::test_all_external_info with data set "pulseaction_notification_get_chapters" (stdClass Object (...))
Unexpected debugging() call detected.
Debugging: External function parameters: invalid OPTIONAL value specified.
* line 730 of /lib/externallib.php: call to debugging()

To resolve this issue, VALUE_OPTIONAL has been replaced with VALUE_DEFAULT, ensuring that the parameter remains optional while providing a default value when omitted.

Additionally, this issue includes the addition of missing privacy providers.

Moodle Version: 4.1.10

@vithushau vithushau changed the title Fix: Replace VALUE_OPTIONAL with VALUE_DEFAULT in get_chapters_parameters() Replace VALUE_OPTIONAL with VALUE_DEFAULT in get_chapters_parameters() May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant