Skip to content
This repository has been archived by the owner on Apr 6, 2020. It is now read-only.

Contao 3.3.3 und theme-plus 4.4 Cache problem #83

Closed
bruc13 opened this issue Jun 30, 2014 · 11 comments
Closed

Contao 3.3.3 und theme-plus 4.4 Cache problem #83

bruc13 opened this issue Jun 30, 2014 · 11 comments

Comments

@bruc13
Copy link

bruc13 commented Jun 30, 2014

Hi,
ich nutze Beide Cachemethoden und seit dem Update zur Contao 3.3. wird die Home Seite, die auch ohne Alias (parameter) aufgerufen werden kann - z.B. https://github.com, ohne CSS und JS Ausgegeben.
Habt ihr eine Idee warum?

@tristanlins tristanlins self-assigned this Jul 1, 2014
@tristanlins
Copy link
Contributor

Bei mir ist die Startseite / Home-Seite immer ohne Alias (= index) aufrufbar und habe noch nie Probleme gehabt. Aber bei Contao 3.3 gibt es wohl hier und da noch ein paar Probleme, die ich noch nicht alle analysiert habe.

@tristanlins tristanlins added bug and removed invalid labels Jul 1, 2014
@bruc13
Copy link
Author

bruc13 commented Jul 1, 2014

Deine Erweiterung ist sehr nützlich, aber auch kompliziert. Hast du eine Idee, woran das liegen kann, dass die collection Dateien beim Cache nicht ausgegeben werden. Bei Contao 3.2 hat es noch funktioniert, was wurde da am Cache geändert?
Habe es noch mal blank ohne irgendwelche Extensions getestet (Nur Composer und Theme Plus)

ubuntu32

Habe inzwischen herausgefunden, dass beim 1 Laden der Seite (wo noch keine Cache Datei angelegt ist) wird disse richtig generiert und unter html abgelegt. Ausgegeben wird aber nicht die gecachte, und dann fehlen die Ressourcen. Beim 2 Laden wird die gecachte geladen, mann kann sie sogar verändern, damit man das sieht. Die Index Seite nimmt aber nie die gecachte Datei.
Vielleicht hilft das. Wenn du eine Idee hast, wie man das lösen kann teste ich es gern.

@dmolineus
Copy link
Member

Existiert das Problem weiterhin?

@davidmaack
Copy link

Ja, das Problem besteht weiterhin :).

Beim Output werden die Methode replaceDynamicScriptTags() zweimal aufgerufen. Einmal in der Methode addToCache() und direkt danach in der eigentlichen output-Methode. Theme-Plus baut hier per Hook die Assets zusammenbaut und damit Contao die Dateien nicht noch einmal verwendet, werden die Arrays TL_JAVASCRIPT, TL_CSS usw. geleert. Beim zweiten Durchlauf des Hooks fehlen dann halt ein paar Dateien :)

Als ersten Workaround habe ich mich in den Hook getCacheKey gehängt und sicher die Werte für TL_JAVACSRIPT und TL_CSS weg (mehr brauchte ich gerade nicht). Beim zweiten Durchlauf von replaceDynamicScriptTags() konnte ich die Werte somit wieder herstellen. Wirklich elegant ist das nicht, hat aber funktioniert :D

@fritzmg
Copy link

fritzmg commented Apr 6, 2016

Habe dasselbe Problem auf http://stats.spikx.net/mwll.html bspw. Dort ist der Seitencache aktiviert und jQuery fehlt beim ersten Ladevorgang im kombinierten JavaScript file (vermutlich dann wenn die Seite für den Cache generiert wird). Beim zweiten Ladevorgang ist jQuery dann wieder dabei.

Scheint aber tatsächlich erst in neueren Contao Versionen zu passieren. Eine andere Seite von mir, die noch unter Contao 3.2.21 läuft, hat auch den Seitencache aktiviert, dort gibt es aber nie Probleme mit fehlenden JavaScripts.

Wäre nett wenn das gefixt wird, ansonsten ist Theme+ für aktuelle Contao Versionen ja nicht mehr benutzbar, wenn man den Seitencache braucht.

@markusmilkereit
Copy link

Im referenzierten Beitrag habe ich eine Lösung vorgeschlagen, die das Problem aus der Welt schafft:

Ich habe jetzt meinen Code an einen Hook früher in der Kette gehängt: statt an replaceDynamicScriptTags wird meine Login jetzt im Hook outputFrontendTemplate bearbeitet - damit funktioniert alles wie erwartet.

@dmolineus
Copy link
Member

Kann das Problem unter 3.5.18 nicht mehr nachstellen. Tritt es bei euch noch auf?

@dmolineus dmolineus added this to the Version 4.x milestone Nov 8, 2016
@davidmaack
Copy link

Also bei einer 3.5.17 ist es die Tage bei uns noch aufgetreten. Da sich am Core an den entsprechenden Stellen aber nichts verändert hat, würde es mich wundern, wenn das Problem weg wäre.

@dmolineus
Copy link
Member

Hab mir das Problem nochmal angeschaut und konnte es nun auch nachstellen. c40af4c sollte das Problem beheben.

Bitte testet dies mal im Fix-Branch https://github.com/netzmacht/contao-theme-plus/tree/fix/page-cache-issue und gebt hier Rückmeldung.

@fritzmg
Copy link

fritzmg commented Nov 18, 2016

Seems to work :)

@dmolineus
Copy link
Member

Seems to work :)

Gerade 4.4.7 veröffentlicht, dass den Bugfix beinhaltet.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants