Navigation Menu

Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[Contao 3] Adding more than 10 modules in the Layout severely breaks the module rendering #4849

Closed
ben-eSM opened this issue Sep 21, 2012 · 8 comments
Labels
Milestone

Comments

@ben-eSM
Copy link

ben-eSM commented Sep 21, 2012

Hi Leo,

I've encountered an severe issue in Contao 3.
When I add more than 10 modules in the Page layout, one of the modules gets rendered multiple times in the Frontend instead of the selected modules which should be shown.
In my case, the sidebar subnavigation got rendered three times. But it should have shown two different modules instead. The entry in the database is correct though.
I can reproduce the issue in a clean Contao 3 Demo installation as well as in my individual Contao 3 playground installation.
It can be reproduced by adding 3 or 4 more modules in the Demo default layout. Instead of showing all the selected modules, in the Frontend, one will get repeated X times after Contao has rendered 9 modules correctly.

@NinaG
Copy link

NinaG commented Sep 21, 2012

Did you [Save] the settings in the page layout and than check which modules are shown? I remember an issue where the select boxes would not update what they show as long as you don't save.

@ben-eSM
Copy link
Author

ben-eSM commented Sep 21, 2012

The modules got saved correctly. And it will show the correct ones if you re-edit the page layout. But in the Frontend, the module rendering breaks after I've added > 10 modules to the layout.
I've tried with internal cache deactivated and I've deleted the tmp files and everything. Won't help.

@leofeyer
Copy link
Member

leofeyer commented Oct 1, 2012

How do I reproduce this with a fresh and current "release/3.0.0" branch installation and the Music Academy example website?

@NinaG
Copy link

NinaG commented Oct 2, 2012

I'll install the C3 (fresh from Github) and Music Academy and try to trigger it there. Post follows as soon as it "works".

@NinaG
Copy link

NinaG commented Oct 3, 2012

  1. Contao 3 (neuer Download 3.0.0 branch von Github) + Music Academy-Theme installiert
  2. Default-layout geöffnet
  3. Breadcrumb-Modul mehrfach dupliziert (bis es insgesamt 5 davon gibt) und gespeichert.

Somit ist die Menge von 10 Modulen erreicht. Noch sieht es nicht nach dem Bug aus.

  1. User-Login-Modul dupliziert (bis es insgesamt 2 davon gibt) und gespeichert.

Nun tritt der Bug anscheinend auf (in der linken Spalte hat sich das eigentlich als 2. angelegte Login-Modul nun optisch in das Breadcrumb-Modul verwandelt, obwohl es technisch im Layout immer noch das 2. Login-Modul ist). Es zeigt sich auch nicht an der korrekten Stelle (Reihenfolge in der linken Spalte) an, sondern als letztes Element links.

@NinaG
Copy link

NinaG commented Oct 3, 2012

Noch auffälligerer Trigger:

  1. Contao 3 + Music Academy Theme installieren.
  2. Neues Modul vom Typ "Eigener HTML-Code" anlegen und als Modultitel sowie Inhalt nur das Wort "Test" eingeben.
  3. Default-Seitenlayout öffnen.
  4. An der ersten Stelle der eingebundenen Module das Test-Modul zweimal für die Header-Spalte anlegen und speichern.

Voila, der Bug triggert: Das zweite Test-Modul wird nicht mehr ausgegeben, stattdessen erscheint ganz am Ende der linken Spalte das Breadcrumb-Modul ein 2. Mal (obwohl es im Seitenlayout nur einmal vorhanden ist, in der Main-Spalte)

Jetzt kann man den Bug beliebig vervielfachen. Einfach noch ein weiteres Mal das Test-Modul in der Header-Spalte einfügen und beobachten wie Contao ans Ende der linken Spalte noch ein weiteres Breadcrumb-Modul malt.

@ben-eSM
Copy link
Author

ben-eSM commented Oct 4, 2012

Thank you Nina for the detailed description to reproduce the issue. So I don't have to do that anymore. :)

It totally seems to be independent from the type of the module. So it has to do with the number of modules.

leofeyer added a commit that referenced this issue Oct 12, 2012
@leofeyer
Copy link
Member

Fixed in ef6d0d1.

leofeyer added a commit that referenced this issue Oct 31, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants