[con 4.9.12] Vorlage verliert Module

[mattmarr]

Hallo!

Gerade gesehen und nicht schön.

Wenn ich im Contenido 4.9.12 bei einer Vorlage das Layout wechsel, verliert es alle zugewiesenen Modul.
In älteren Version, hier getestet in v4.9.5, ist noch alles Ok.

Kann das wer bestätigen?



Gruß
Matthias

[malsdgtac]

Hallo,
ja das kann ich bestätigen, ist bei mir auch so.

[resp]

Kann ich bestätigen!

Bei einer älteren Installation (4.9. scheint es noch so funktionieren!

[frederic.schneider_4fb]

Ich bin gerade irritiert:

Ihr habt ein Layout. Diesem Layout habt Ihr eine Vorlage zugewiesen und Module ausgewählt. Dann ändert Ihr die Vorlage und wundert Euch, weshalb die bisherige Modulauswahl verschwunden ist? Helft mir mal gerade auf die Sprünge: Das macht doch Sinn?

[mattmarr]

Ihr habt ein Layout. Diesem Layout habt Ihr eine Vorlage zugewiesen und Module ausgewählt. Dann ändert Ihr die Vorlage und wundert Euch, weshalb die bisherige Modulauswahl verschwunden ist? Helft mir mal gerade auf die Sprünge: Das macht doch Sinn?

Ich finde es ganz gut, das bei einem wechsel der Vorlage die Module, insofern die Container ID übereinstimmen, vorhanden bleiben. Bis vor kurzem blieben bei einem wechsel die Modul erhalten.

Wäre klasse, wenn es wieder funktionieren würde.
Spart doch etwas Arbeit, wenn man eine Vorlage als Sample nimmt und diese Kopiert und anschließend nur noch fehlende Modul ergänzt.


Gruß
Matthias

[rethus]

Ich finde das ist ein absolutes Must-Have im Sinne der usability.
Ich kenne es eigentlich auch nur so, das nur die Zuordnungen nicht vorhanden sind, bei denen ein neues Modul hinzugefügt (oder ein Modul ausgewechselt) wurde, und die CMS_XXX Typ -ID nicht in dem vorigen Modul verwendet wurde.

Wenn die neuere Version das nachweislich nicht mehr haben sollte, wäre das ein enormer Rückschritt

[frederic.schneider_4fb]

Hotfix, allerdings mit potentiellen Seiteneffekten (s. u.):
functions.tpl.php, Zeile 83

Code: Alles auswählen

if (is_array($c) && (int)$changelayout !== 1) {
            foreach ($c as $idcontainer => $dummyval) {
                $containerColl2 = new cApiContainerCollection();
                $containerColl2->create($idtpl, $idcontainer, $c[$idcontainer]);
            }
}

Ersetze

Code: Alles auswählen

if (is_array($c) && (int)$changelayout !== 1) {

durch (vorheriges Verhalten)

Code: Alles auswählen

if (is_array($c)) {

Die Änderung stammt aus dem August 2016:
https://issues.contenido.org/browse/CON-2546

Ich mache dazu ein neues Ticket auf, da die damalige Änderung ja auch einen gewissen Sinn hatte. Wir brauchen also eine neue Lösung