Modul-Code wird mehrfach ausgeführt

[jdlx]

Estigy möchte aber ja nicht den Inhalt dauerhaft cachen, sondern nur während eines Requests.

Über OUTPUT_FILTER gehen? Würde ich vermutl. in dem Fall.. bin aber nicht wirklich sicher ob ich die Intentionen des op richtig verstehe.

[Gregor.Harlan]

Keine Ahnung, was du jetzt genau meinst. Aber ich stelle es mir wenn so vor (und Estigy vermutlich auch), dass die rex_article-Klasse (die in R5 rex_article_content heißt) intern irgendwo die bereits ausgeführten Artikel ablegt, und bei erneuter Abfrage direkt zurückliefert. Aber alles halt nur während eines Requests.

[jdlx]

Wenn man das von vornherein so konstruiert hat: klar, eine Möglichkeit.. in Bezug auf rex4 is halt nicht. Wenn ich den op richtig verstanden habe, will er aus dem fertig geparsten Artikel SEO kram/keywords rausziehen, und oben in die metas eintragen -> OPF.. allerdings sollte er da dann ein eigenes caching dahinterlegen.

[jdlx]

Eine komplett andere Alternative: die Rexseo custom keywords benutzen und automatisiert mittels seiner Funktion befüllen..

[Gregor.Harlan]

Achso, dir ging es um eine akute Lösung für sein Problem

Da hätte ich auch noch eine dritte Alternative:
Aktuell scheint er die Keywordberechnung ja direkt in der config.inc.php eines Addons zu machen. Zitat:

Zur Erklärung: Ich habe ein SEO-AddOn geschrieben, welches zB die Meta-Keywords direkt aus dem Artikel-Inhalt gewichtet berechnet. Dazu muss ich den Artikel-Content bekommen, bevor das Template gerendert wird, da ja die daraus berechneten Keywords dann in die Meta-Tags sollen.

Ich würde im Template oben einmal den Content holen (getArticle()), dann die die Berechnung aus dem Addon anstoßen, und den Content übergeben. Danach hätte er dann sowohl die keywords, als auch den einmalig generierten Content im Template zur Verfügung.

[Thomas.Blum]

Estigy möchte aber ja nicht den Inhalt dauerhaft cachen, sondern nur während eines Requests.

Inhalt in eine Variable speichern.

Steht schon alles auf Seite 1. Auch die Lösung für sein Problem.


vg Thomas

[jdlx]

Yo, das ist die einfachste Lösung..

[Estigy]

Wenn man das von vornherein so konstruiert hat: klar, eine Möglichkeit.. in Bezug auf rex4 is halt nicht.

Dass es derzeit nicht so ist, ist eh klar. Und genau diesen Umstand kritisiere ich bzw. sehe als Bug im System. Wenn man Content dynamisch generiert und dabei auch PHP ausgeführt werden darf, muss man halt als Programmierer des Systems dafür sorgen, dass das nur ein einziges mal passiert. Ich möchte aber wirklich niemandem persönlich zu nahe treten.

Über OUTPUT_FILTER gehen? Würde ich vermutl. in dem Fall..

Ja, daran habe ich eh auch schon gedacht. Das einzige Problem an der Sache ist, dass ich dann berücksichtigen muss, wo genau im gesamten Content der Hauptartikel steht. Wenn ich das direkt über getArticle() mache, bekomme ich ausschließlich den Artikel, um den es mir geht, und muss nicht später dann den gesamten Content parsen. (Und das auch noch vielleicht abhängig davon, welches der Templates verwendet wird.) Die Chancen dafür, dass so eine Funktionalität dann für die Allgemeinheit verfügbar gemacht werden könnte (lies: so einfach wartbar ist, dass es für den einfachen User leicht nutzbar ist) oder einfach wiederverwendbar ist, sehe ich praktisch bei Null.

Nachmal: Mir geht es weniger um die SEO-Sache selbst. Das war für mich nur der Anlass, wie ich auf das Problem gestoßen bin. Wenn ein Auto immer eingeht, wenn jemand seine 20 km entfernte Freundin besucht, kann man zwar darüber diskutieren, ob die beiden nicht einfach zusammenziehen sollten - besser wäre es aber, das Auto käme in die Werkstatt.

Estigy möchte aber ja nicht den Inhalt dauerhaft cachen, sondern nur während eines Requests. Darüber habe ich für R5 auch schon mal kurz nachgedacht, bin aber noch zu keinem Ergebnis kommen.

Ganz genau, so ist es.
Zur Umsetzung: Einfach ein internes (statisches) Array anlegen als Daten-Cache:

Code: Alles auswählen

protected static $generatedArticles = array();

Wenn Daten da, diese ausgeben, ansonsten neu generieren und danach ausgeben.

Code: Alles auswählen

// Code nur generieren, wenn wir ihn noch nicht haben
if (!isset(self::$generatedArticles[$id][$sprache])) {
    $generatedCode = [...]; // Artikelcontent generieren
    // Ab in den Cache damit
    self::$generatedArticles[$id][$sprache] = $generatedCode,
}
return self::$generatedArticles[$id][$sprache];

Im schlimmsten Fall machst du eine Proxyklasse, die sich darum kümmern kann. (Ich habe schon überlegt, genau dieses als eigenes kleines AddOn zu machen.)

Denn dass Leute damit Probleme haben, ist ja öfters hier Thema im Forum.

Sehe ich auch so - gerade deshalb wundert es mich, dass die Leute lieber über die Sinnhaftigkeit meiner SEO-Sache diskutieren als über das Problem an sich...

Ich hoffe jedenfalls, dass sich für die nächste Redaxo-Version eine Lösung dafür finden lässt. Ich würde auch gerne etwas Code beisteuern, wenn nötig.

LG, E.

[jdlx]

Über OUTPUT_FILTER gehen? Würde ich vermutl. in dem Fall..

Ja, daran habe ich eh auch schon gedacht. Das einzige Problem an der Sache ist, dass ich dann berücksichtigen muss, wo genau im gesamten Content der Hauptartikel steht.

HTML comments als Delimiter verwenden..