- 🇬🇧United Kingdom adamps
Thanks for the report. I believe the correct fix is much simpler, because all the code we need is on StringTranslationTrait, which is included by a parent class.
I don't hit the bug myself, and I don't see how you do😃. All the calls to
$this->t
should have initialised$this->stringTranslation
.Anyway it's still worth fixing. Instead of
$this->stringTranslation->formatPlural()
do
$this->formatPlural(
- Assigned to shivam_tiwari
- @shivam_tiwari opened merge request.
- Issue was unassigned.
- Status changed to Needs review
about 2 years ago 12:21pm 3 April 2023 - 🇬🇧United Kingdom adamps
Thanks for the patch. It looks good to me. It would be great if @joel_osc could confirm if the problem has been solved.
- Status changed to Fixed
about 2 years ago 6:50pm 5 April 2023 -
AdamPS →
committed b61b40cb on 3.x authored by
shivam_tiwari →
Issue #3318534 by shivam_tiwari, joel_osc, AdamPS: WSOD on simplenews...
-
AdamPS →
committed b61b40cb on 3.x authored by
shivam_tiwari →
Automatically closed - issue fixed for 2 weeks with no activity.