- Issue created by @realityloop
- Merge request !168Issue #3502661 by realityloop: handle null integer → (Merged) created by realityloop
- 🇦🇺Australia realityloop
The error occurs because the
collect
method expects an integer for the$limit
argument but receivesnull
. To fix this allows$limit
to be either an integer or null, defaulting to 0 if not provided. - 🇳🇱Netherlands bojan_dev
This can only occur if the config "simple_oauth.settings" or the setting inside "token_cron_batch_size" isn't available. I wouldn't make the limit argument nullable, I rather see it addressed in the if statement on
simple_oauth.module:30
. - First commit to issue fork.
- 🇳🇱Netherlands idebr
- simple_oauth_cron() now checks for falsy values for token_cron_batch_size and defaults to 0
- token_cron_batch_size is now a required field on the config form, since it is not nullable
- Added the 0 limit to the description of the form item on the config form and schema
-
bojan_dev →
committed ee0b0253 on 6.0.x authored by
realityloop →
Issue #3502661 by realityloop: handle null integer
-
bojan_dev →
committed ee0b0253 on 6.0.x authored by
realityloop →
Automatically closed - issue fixed for 2 weeks with no activity.