Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Offline-Status defekt, wenn Artikelinhalt bereits vorm EP SLICE_SHOW generiert wird #86

Closed
schuer opened this issue Dec 16, 2019 · 3 comments

Comments

@schuer
Copy link
Member

schuer commented Dec 16, 2019

Problembeschreibung:

Wenn du im Backend rex_article_content->getArticle() verwendest, wird dir REDAXO alle Slices liefern, auch die Offline-Slices. Schlimmer noch: Es werden dabei bereits die Cache-Files für den Artikel generiert, die dann ebenso die Offline-Slices abbilden und sie damit sogar im Frontend ausbringen. Du hebelst damit also den blÖcks-Mechanismus vollständig aus. Denn blÖcks greift erst/nur beim EP SLICE_SHOW ein.

Code-Beispiel, das den Fehler zeigt:

Innerhalb der boot.php des Project-AddOns:

<?php
$article = new rex_article_content(42);
$content = $article->getArticle();

Fix?

Ich habe jetzt nicht alles bis ins Detail zerpflückt, aber mir scheint, es gibt auf Seite von blÖcks gar keine andere Lösung dafür. Wollte man ermöglichen, dass sowas wie rex_article_content->getArticle() richtig mit Offline-Slices umgeht, müsste man es im Core bzw. im Structure-AddOn implementieren, nicht in blÖcks.

Vielleicht hat jemand eine Idee?

@tbaddade
Copy link
Member

Wollte man ermöglichen, dass sowas wie rex_article_content->getArticle() richtig mit Offline-Slices umgeht, müsste man es im Core bzw. im Structure-AddOn implementieren, nicht in blÖcks.

Sehe ich auch so. Ich würde sogar soweit gehen, dass man auch für andere Tabellen gleich das Feld status mit aufnimmt. Für Module, Templates, Medien, Medienkategorien, Metainfo etc. Die Tabellen sozusagen insgesamt ein wenig mehr angleichen.

@alxndr-w
Copy link
Member

alxndr-w commented Mar 2, 2020

Ich mache hier mal zu, da es bloecks/status nicht mehr geben wird, sondern im Core zu Hause ist.

@alxndr-w alxndr-w closed this as completed Mar 2, 2020
@tbaddade
Copy link
Member

tbaddade commented Mar 2, 2020

Kann geschlossen bleiben, da der Core die Slices nicht in den Cache schreibt.

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

No branches or pull requests

3 participants