Code: Alles auswählen
$article->getArticle()
Code: Alles auswählen
$a = rex_article();
$a->setArticleId(DIE_ARTIKEL_ID);
echo $a->getArticle();Wie willst du denn an eine Ausgabe kommen, wenn du keinen Code ausführst?Estigy hat geschrieben:Wie schon geschrieben: Die Tatsache, dass der Modul-Code mehrfach evaluiert wird, halte ich wirklich für einen grundsätzlichen Bug im System, nicht für einen Anwenderfehler mit einfachem Workaround.
) Code: Alles auswählen
<html>
<head>
<?php echo MeineKlasse::getMetaTags(); ?>
</head>
<body>
REX_ARTICLE[]
</body>
<html>Code: Alles auswählen
<html>
<head>
<?php
$content = $this->getArticle();
$meta = new MeineKlasse();
$meta->setContent($content);
echo $meta->getMetaTags();
// oder
echo MeineKlasse::getMetaTags($content);
?>
</head>
<body>
<?php echo $content; ?>
</body>
<html>Code: Alles auswählen
if ($this->isGenerated) {
return $this->generatedCode;
} else {
// [...] generiere alles wie bisher
}Ich muss gestehen, dass ich noch nie dieses Problem hatte. Mein Template enthält meistens ein einfaches REX_ARTICLE[] für die Ausgabe, maximal vielleicht noch so viele, wie ich CTypes habe.Estigy hat geschrieben:Etwas verstört bin ich dennoch, dass hier keiner ein Problem damit hat, dass der Modul-Code überhaupt mehrfach ausgeführt werden kann. Stattdessen wird nur über mögliche Workarounds gesprochen.
Der Weg, den du gehst, ist halt auch sehr ungewöhnlich. Normalerweise wird die Artikelausgabe exakt einmal pro Seitenaufruf gebraucht.Estigy hat geschrieben:In meinen Augen ein klarer Bug. Wenn ich 7x getArticle() ausführe, möchte ich als Anwender 7x den Artikel-Code zurückbekommen - und natürlich nicht 7 Mails senden und 7 Datenbank-Einträge erstellen; das sollte doch wohl jedem klar sein.
Eigentlich hätte ich ein "Stimmt, wir werden das beheben" erwartet.
Endlich rückst du mal damit raus, was dein Problem ist. Wenn du anhand von Artikelinhalten solche Sachen, wie deine automatische Metakeywordgeneration machen möchtest, dann geht man da am besten über die Extensionpoints, die beim Anlegen und Editieren von Slices aufgerufen werden.Estigy hat geschrieben: Ich spreche eh vom Frontend.
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. (Ja, ich weiß, es gibt Meta-Keywords auch direkt beim Artikel, aber ich habe Kunden, die ihre >500 Artikel nicht einzeln mit Metadaten verschlagworten wollen...
Naja... Kommt drauf an, wie man es sieht.Endlich rückst du mal damit raus, was dein Problem ist.
Alleine die Tatsache, dass es hier im Forum einige Threads zu mehrfach versendeten E-Mails oder dergleichen gibt, sollte deine Aussage doch widerlegen, oder?Normalerweise wird die Artikelausgabe exakt einmal pro Seitenaufruf gebraucht.
Du hast meinen Beitrag missverstanden. Mit "Aufstand" meine ich dein Addon, bzw. die Energie die du da rein steckst. Ich wollte dich auch nicht heruntermachen, ich finde den Kontext deines Problems nur etwas amüsant. Nix für ungut.Estigy hat geschrieben:Hm... Das Melden eines Bugs ist also "ein Aufstand".
Ich sehe hier kein Bug nur einen Anwender-, Programmierer-Fehler.Estigy hat geschrieben:Hm... Anstatt über den Bug selbst zu diskutieren, werde ich nun wegen des Anwendungsfalles heruntergemacht.
Darauf sollte man (wie Du auch bemerkt hast) deutlicher in der Doku hinweisen. €€€€ Code: Alles auswählen
Wiederverwendbarkeit: Der User, der mein AddOn verwendet, soll natürlich nicht alle seine Templates umschreiben müssen.Statische Seite vs. dynamische?Estigy hat geschrieben:Außer natürlich, jemand kann mir schlüssig erklären, warum es tatsächlich manchmal nötig sein sollte, den Code für jeden Aufruf einzeln abzuarbeiten.
Genau diesen Punkt sehe ich leider ganz grundsätzlich anders. Die Methode heißt "getArticle()" und nicht "generateArticleContent()". Wenn ich ersteres aufrufe, möchte ich den Artikel-Inhalt haben, nicht generieren lassen. Für mich sind die Erzeugung des Contents (samt Abarbeitung der darin enthaltenen Module) und die Rückgabe des Ergebnisses zwei komplett getrennte Aufgabengebiete. - Ersteres darf natürlich nur ein einziges Mal passieren, zweiteres kann beliebig oft geschehen. Eine saubere API würde hier trennen (und dem User keine Vorschriften machen, wie oft ein Artikel-Content später abgerufen werden darf).Redaxo macht genau das was ihm mitgeteilt wird.
Das ist ja gerade der Unterschied zwischen dem, was ich will, und dem, was Redaxo tut: Ich will ihn ja gerade nicht "mehrfach ausführen", sondern nur "mehrfach erhalten". Dafür bietet Redaxo aber leider keine Funktionalität. Ich muss mich selbst drum kümmern mitsamt dem dazu nötigen Arbeitsaufwand für die Dokumentation und den Support. Eine Lösung innerhalb von Redaxo würde hier Abhilfe schaffen: Da offensichtlich eh kaum jemand getArticle() mehrfach aufruft, um den Content tatsächlich mehrfach generieren zu lassen, könnte man getArticle() weiterhin benutzen, ohne Rückwärtsinkompatibilitäten befürchten zu müssen. Bei der internen Verarbeitung könnte man zwischen Gernerierung (nur beim ersten Aufruf) und Rückgabe sauber trennen.Wenn Du den Content mehrfach ausführen willst, dann wird dies auch erledigt.
Das ist natürlich immer ein ArgumentUnd wenn der Kunde damit ein Problem hat, dann biete Ihm eine entsprechende Template-Optimierung an.
Nochmal: wie willst du das machen wenn der Inhalt dynamisch ist? Ein dynamischer Artikel kann in zich Varianten vorliegen je nach Parameter. Du siehst das Thema einfach nur aus deiner speziellen Sicht..Estigy hat geschrieben:..möchte ich den Artikel-Inhalt haben, nicht generieren lassen.
![[ externes Bild ]](http://maumha.de/muh/schaf.gif){kind=link}