search

iceman-fx / gridblock

Redaxo 5-Addon zum gruppieren/kombinieren mehrerer beliebiger Inhaltsmodule (Blöcke) innerhalb eines selbst definierbaren Spaltenrasters (GRID).
16 stars 3 forks source link

Contentsettings - Checkbox-Bug #45

Closed iceman-fx closed 2 years ago

iceman-fx commented 2 years ago

ich habe einen Bug bei der Checkbox gefunden. Wenn eine vorhergehende Checkbox aktiv ist, wird eine andere, neue, noch nicht gesetzte bei mir autom. aktiv.

Ich kann es sogar reproduzieren. In Settings-Tab1 habe ich eine Checkbox und in Settings-Tab4 eine. Alle haben unterschiedliche Namen etc, Setze ich die CB in Tab1 nun auf On und speichere den Block, dann ist die CB in Tab4 auch gesetzt, obwohl diese vorher nie angewählt war.

Ich könnte mir vorstellen, dass in Deiner Routine eine Variable nicht vorher genullt wird, wenn der neue Durchlauf für das nächste Element beginnt.

iceman-fx commented 2 years ago

In Zeile 481 müsste vor das break noch ein unset() rein:

unset($sChecked);
break;

https://github.com/iceman-fx/gridblock/blob/e83e87cfa81359b5fc8108284c5fd58d90eb6b8f/plugins/contentsettings/lib/class.GridblockContentSettings.php#L481

novinet-dsteffen commented 2 years ago

Erledigt