- Issue created by @jjgw
- Status changed to Needs work
over 1 year ago 11:54am 29 June 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary should always describe what should be fixed/changed. Neither the title nor screenshots are sufficient to describe what the issue is, even in the case a patch is provided.
In particular, for a bug report, the issue needs to describe the steps to reproduce the bug and what exactly is the message error or the thrown exception. - 🇺🇸United States angrytoast Princeton, NJ
Updating the issue summary and tags to more accurately reflect the PHP 8.2 compatibility issue.
- Status changed to Needs review
over 1 year ago 4:31pm 15 July 2023 - 🇺🇸United States angrytoast Princeton, NJ
Adding a patch for the dynamic class properties issue
- 🇮🇳India Akram Khan Cuttack, Odisha
while reviewing this issue observed some TypeError issue as well. so i added updated patch for that and it resolve the issue now.
- First commit to issue fork.
- Status changed to RTBC
8 months ago 5:07pm 19 March 2024 - 🇺🇸United States jcandan
@Akram Khan → , your mixed patch is already addressed at 🐛 Argument #1 ($seconds) must be of type string|int, null given RTBC . It seems best to not mix fixes.
Marking this RTBC. Rolled patch #5 into MR !7.
- First commit to issue fork.
-
svendecabooter →
committed 8bbdc984 on 2.0.x authored by
jcandan →
Issue #3367474 by jcandan, angrytoast, svendecabooter: Error when trying...
-
svendecabooter →
committed 8bbdc984 on 2.0.x authored by
jcandan →
- Status changed to Fixed
5 months ago 3:44pm 26 June 2024 - 🇧🇪Belgium svendecabooter Gent
Thanks for the fixes!
I have done some cleanup work to avoid duplicate code.
This has now been committed. Automatically closed - issue fixed for 2 weeks with no activity.