JoomGalleryfriends / JG4-dev

Development repository for JoomGallery v4.x
GNU General Public License v3.0
10 stars 6 forks source link

Fix error when reset config #229

Closed MrMusic closed 1 month ago

MrMusic commented 1 month ago

Fix for issue https://github.com/JoomGalleryfriends/JG4-dev/issues/227

How to test this PR

Open a existing JoomGallery configuration and klick on 'Reset' and confirm the action.

Test result before applying this PR

A PHP error occurs.

Test result after applying this PR

No error is thrown.

eumel1602 commented 1 month ago

BEARBEITET: 19.5. 20 Uhr: Die Fehlermeldung ist nach dem Pr installation bei mir, in einer J 4.4.4. Version weg!

rowi68 commented 1 month ago

Bei mir funktioniert es nur in J 4.4.4 In J 5.1 kommt folgende Fehlermeldung:

Joomla\CMS\Event\Model\AfterCleanCacheEvent::onSetResult(): Argument #1 ($value) must be of type bool, array given, called in ..../libraries/src/Event/AbstractEvent.php on line 227 2024-05-19_19h25_36

eumel1602 commented 1 month ago

Bei mir funktioniert es nur in J 4.4.4 In J 5.1 kommt folgende Fehlermeldung:

Ja, muss ich leider bestätigen. Ich hielt es für einen anderen Fehler die eine Komponete verursacht. Habe alles neu gemacht und nun bei 5.1 den Selben Fehler wie @rowi68 PHP version 8.2.5

MrMusic commented 1 month ago

In J 5.1 kommt folgende Fehlermeldung: Joomla\CMS\Event\Model\AfterCleanCacheEvent::onSetResult(): Argument https://github.com/JoomGalleryfriends/JG4-dev/pull/1 ($value) must be of type bool, array given, called in ..../libraries/src/Event/AbstractEvent.php on line 227

Dieser Fehler wird durch das Plugin 'System - JoomGallery' verursacht und wurde bereits hier gemeldet: https://github.com/JoomGalleryfriends/JG4-dev/issues/226 Das Problem betrifft auch andere Branches unter Joomla 5.1. Wenn du diesen PR unter Joomla 5.1 testen möchtest, musst du momentan dieses System-Plugin deaktivieren.

rowi68 commented 1 month ago

PR erfolgreich getestet. ✅

eumel1602 commented 1 month ago

mit dem "ausschalten" den PR erfolgreich mit J 5.1 getestet! ✅

Elfangor93 commented 1 month ago

Thank you very much.