You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
in a fresh installation, with default setting we get:
Deprecated: The "timeout" configuration is deprecated and will be removed with the next major update. Set the "timeout" configuration to false and if needed use the set_timeout_limit() function in your own code. in ..vendor/rakibtg/sleekdb/src/Store.php on line 748
but if one use the default settings, there should no deprecation message.
Hi @SegFaulty
Thank you for your suggestion. We intentionally didn't change the old default setting, because that would be a breaking change. The timeout configuration will be removed with the next major update. Until then we need to ensure backwards compatibility.
The problem here is that you are testing for timeout = false in the configuration, which is triggering a deprecation error even if we don't set a timeout value.
IMO, this isn't backwards compatible at all, as a deprecation error is not the previous behaviour.
in a fresh installation, with default setting we get:
Deprecated: The "timeout" configuration is deprecated and will be removed with the next major update. Set the "timeout" configuration to false and if needed use the set_timeout_limit() function in your own code. in ..vendor/rakibtg/sleekdb/src/Store.php on line 748
but if one use the default settings, there should no deprecation message.
now we have to supply:
suggested fix:
The text was updated successfully, but these errors were encountered: