Argument #2 ($array) must be of type array, null given

FriendsOfREDAXO / geolocation

Tile-Proxy/Cache und Kartendarstellung mit Leaflet für REDAXO 5.13+

MIT License

26 stars 3 forks source link

Argument #2 ($array) must be of type array, null given #125

Closed dpf-dd closed 1 year ago

dpf-dd commented 1 year ago

Feedback eines Beta-Testers: Ich habe soeben die 2.0beta installiert (neuestes Redaxo auf PHP 8.0) und ohne das ich großartig etwas an der nackten Installation verändert habe erhalte ich im Register "Kartensatz" beim Speichern des Datensatzes eine Fehlermeldung => siehe Screenshot.

Gruß, Stefan

dpf-dd commented 1 year ago

Zur Vollständigkeit hier noch der letzte Eintrag vom Tracestack:

Der Fehler ist reproduzierbar, wenn man in den Einstellungen den Radio-Button nicht gesetzt hat:

Es scheint also nur eine fehlender Validierung zu sein bzw. ließe sich das mit einer Validierung einfach beheben!? Wobei das eigentlich auch dumm ist, weil ich keinen Overlay möchte - also wähle ich da auch nichts aus. Es ist deswegen für mich ein kleiner Bug (der mich in diesem Usecase aber glücklicherweise nicht stört)...

christophboecker commented 1 year ago

@dpf-dd war das eine Neuinstallation oder über ein Update?

christophboecker commented 1 year ago

Ah, ok, Ich hab mal ein System frisch aufgezogen und kriege den Fehler reproduziert. Damit kann ich arbeiten. Danke, ich schau mal

christophboecker commented 1 year ago

@dpf-dd Stefan, könntest Du mal testen und in lib/yform/value/layerselect.php die zwei Zeilen 94 und 95 ändern?

$selectedLayers = isset($value['choice']) ? $value['choice'] : [];
$value = isset($value['value']) ? $value['value'] : [];

Bei mir läuft es mit der Änderung durch. Wenn bei Dir auch, mache ich nen PR dazu auf.

dpf-dd commented 1 year ago

@christophboecker Jo, getestet. Läuft durch. Alles prima jetzt! 👍

christophboecker commented 1 year ago

Danke fürs finden und testen.

dpf-dd commented 1 year ago

Danke für die schnelle Reaktion und Lösung. So macht das Spaß :)