catalyst / moodle-auth_saml2

SAML done 100% in Moodle, fast, simple, secure
https://moodle.org/plugins/auth_saml2
70 stars 132 forks source link

Calls to getString/getArray need updating #775

Open cameron1729 opened 9 months ago

cameron1729 commented 9 months ago

After the lib upgrade, there's still a few calls to getArray that need to be changed to getOptionalArray

e.g.,:

And this one confuses me: https://github.com/sarahjcotton/moodle-auth_saml2/blob/MOODLE_39_STABLE/locallib.php#L209 - I don't get why it's called with 3 params.