- Issue created by @eduardo morales alberti
- Merge request !5Issue #3521754 by eduardo morales alberti: Deprecated function: sleep():... → (Open) created by eduardo morales alberti
During the work done in
#2972909 →
and specifically the commit
b4c250aed6364d865f0dbdda315379e7999a61f1,
the configuration variable request_token_wait was renamed to requestToken_wait in the code:
- sleep($config->get('request_token_wait'));
+ sleep($config->get('requestToken_wait'));
The corresponding update in marketing_cloud.settings was not made. As a result, the requestToken_wait key does not exist in the settings configuration, causing it to be NULL at runtime.
A similar issue had previously happened with requestToken_url, but it was corrected in
commit d30844e8dcbf4680c2142232dd71afd213f5dce0.
An error is logged in the Drupal watchdog (recent log messages) when the system tries to execute the sleep() function with a NULL value:
Deprecated function: sleep(): Passing null to parameter #1 ($seconds) of type int is deprecated in Drupal\marketing_cloud\MarketingCloudSession->token()
requestToken_wait configuration key to marketing_cloud.settings with an appropriate default value, similar to the approach used for requestToken_url.requestToken_wait default setting.None expected.
None expected. This is purely a configuration fix.
None expected, unless the configuration schema is updated to ensure correct defaults.
Active
1.0
Code