Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Upon authentication approval, the user receives the following error after approving the authentication:
Notice: Undefined index: disable_auth_level_selection in oauth_common_form_authorize_validate() (line 239 of /opt/development/jobeirne/jammiejam/html/sites/all/modules/oauth/oauth_common.pages.inc).
This is because user contexts are saved without `disable_auth_level_selection` as a key in the authentication_options array if that particular checkbox is left empty in the context edit form.
Comment | File | Size | Author |
---|---|---|---|
#6 | oauth-error_on_authentication_approval-1412258-6.patch | 5.1 KB | h3rj4n |
#1 | auth-options-1412258-1.patch | 5.38 KB | jobeirne |
Comments
Comment #1
jobeirne CreditAttribution: jobeirne commentedAttached is a patch for the context edit form. Instead of completely ignoring an authentication_options value if it is null in the form, it is saved in the array as null. This maintains compatibility with the plethora of `empty` checks that happen. Also included is a cosmetic cleanup of the submit function.
Comment #2
jobeirne CreditAttribution: jobeirne commentedTested and committed.
Comment #3
kylebrowning CreditAttribution: kylebrowning commentedThis is still broken in 6.x
Comment #4
rjbrown99 CreditAttribution: rjbrown99 commentedUpdating status per the issue queue handbook guidelines.
Comment #5
h3rj4n CreditAttribution: h3rj4n commentedtrailing white space error
I can't apply the patch on dev.
Tried it on a fresh pull of dev.
Comment #6
h3rj4n CreditAttribution: h3rj4n commentedThis is the patch that works on D6. Haven't checked the results xD Only ported it.