Check settings that require int only in text field
Issue #841
resolved
A potential issue exists in the max course title length setting. This expects an integer but the text field will allow anything. If anything other that int is put in there, it could provide unpredictable results. Check other similar settings.
Comments (3)
-
-
reporter - changed status to resolved
Fix
#841→ <<cset c94c21402114>>
-
reporter I checked all settings and only found two cases where the setting requires a number so those were updated to accept PARAM_INT only, which should be enough. As the realistic number range was fairly wide on these it didn't seem worth making it a dropdown. There was a third which is the Matomo site ID in Analytics but I'm unsure if that might need to be a number only.
- Log in to comment
Will you be changing this to a select drop down list instead of a free text field?