inspiredminds / contao-sibling-navigation

Provides a previous/next navigation for news & events.
GNU Lesser General Public License v3.0
6 stars 0 forks source link

Navigation verlinkt zu (noch) nicht veröffentlichten Nachrichten #10

Closed johannes-bittner closed 3 years ago

johannes-bittner commented 4 years ago

Offenbar prüft die Navigation nicht, ob Nachrichten auch schon sichtbar/publiziert sind - ich habe hier den Fall, dass die Publikation einer Nachricht auf den kommenden Tag terminiert ist (= in einer gewöhnlichen Liste bis dahin nicht angezeigt), der Link der sibling-navigation aber bereits im Frontend ausgegeben wird.
grafik Lässt sich das irgendwie abfangen?

fritzmg commented 4 years ago

Ja, das wurde tatsächlich für Nachrichten bisher nicht berücksichtigt. Bitte die aktuelle dev-master Version testen.

johannes-bittner commented 4 years ago

Gerade getestet - Contao-Fehlermeldung im Preview, im Debug-Modus erhalte ich die Weiterleitungs-Fehlermeldung:
Diese Seite funktioniert nicht ... hat Sie zu oft weitergeleitet. Löschen Sie Ihre Cookies. ERR_TOO_MANY_REDIRECTS

... allerdings nur auf der Reader-Seite - ?

fritzmg commented 4 years ago

Hm 🤔 ich denke nicht, dass das etwas mit den Änderungen zu tun hat. Funktioniert es wieder, wenn auf Version 1.2.0 zurück aktualisiert wird?

johannes-bittner commented 4 years ago

... ja, mit 1.2 gehts, mit dev-master definitiv nicht. Aber der Weiterleitungsfehler ist immer noch da ... 🙈 ... sobald das Leser-Modul auf der Seite angezeigt wird. Das macht es dann natürlich etwas schwierig, eine Fehlermeldung zu erhalten ...

johannes-bittner commented 4 years ago

Hm ... wenn ich https://github.com/fritzmg/contao-simple-news-urls deinstalliere, klappts auch mit der Fehlermeldung 😉 (aber schade eigentlich ...)

Unknown method isPreviewMode

Exception: Unknown method isPreviewMode

at vendor/contao/core-bundle/src/Resources/contao/library/Contao/Model.php:1003 at Contao\Model::__callStatic('isPreviewMode', array()) (vendor/inspiredminds/contao-sibling-navigation/src/Module/SiblingNavigationNews.php:109) at InspiredMinds\ContaoSiblingNavigation\Module\SiblingNavigationNews->compile() (vendor/contao/core-bundle/src/Resources/contao/modules/Module.php:209) at Contao\Module->generate() (vendor/inspiredminds/contao-sibling-navigation/src/Module/SiblingNavigationNews.php:85) at InspiredMinds\ContaoSiblingNavigation\Module\SiblingNavigationNews->generate() (vendor/contao/core-bundle/src/Resources/contao/elements/ContentModule.php:76) at Contao\ContentModule->generate() (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:580) at Contao\Controller::getContentElement(object(ContentModel), 'main') (vendor/contao/core-bundle/src/Resources/contao/modules/ModuleArticle.php:187) at Contao\ModuleArticle->compile() (vendor/contao/core-bundle/src/Resources/contao/modules/Module.php:209) at Contao\Module->generate() (vendor/contao/core-bundle/src/Resources/contao/modules/ModuleArticle.php:74) at Contao\ModuleArticle->generate(false) (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:520) at Contao\Controller::getArticle(object(ArticleModel), false, false, 'main') (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:371) at Contao\Controller::getFrontendModule('0', 'main') (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:168) at Contao\PageRegular->prepare(object(PageModel)) (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:48) at Contao\PageRegular->getResponse(object(PageModel), true) (vendor/contao/core-bundle/src/Resources/contao/controllers/FrontendIndex.php:339) at Contao\FrontendIndex->renderPage(object(PageModel)) (vendor/symfony/http-kernel/HttpKernel.php:158) at Symfony\Component\HttpKernel\HttpKernel->handleRaw(object(Request), 1) (vendor/symfony/http-kernel/HttpKernel.php:80) at Symfony\Component\HttpKernel\HttpKernel->handle(object(Request), 1, true) (vendor/symfony/http-kernel/Kernel.php:201) at Symfony\Component\HttpKernel\Kernel->handle(object(Request)) (web/preview.php:31)

fritzmg commented 4 years ago

Unknown method isPreviewMode

Dieser Fehler ist nun in der neuesten dev-master Version behoben. Bitte nochmal testen.

Hm ... wenn ich https://github.com/fritzmg/contao-simple-news-urls deinstalliere, klappts auch mit der Fehlermeldung 😉

Ich habe auch diese Extension nochmal getestet, konnte aber keinen Fehler feststellen. Die Extension funktioniert auch noch in Contao 4.9.5. Vermutlich sind hier andere Faktoren im Spiel, die einen unendlichen Redirect auslösen (entweder eine weitere Extension oder eine Server Konfiguration).

johannes-bittner commented 3 years ago

Super, funktioniert jetzt einwandfrei.

Zu contao-simple-news-urls habe ich in einer frischen 4.9.5 herausgefunden, dass es wohl beim Zusammenspiel mit fehlenden Suffixen hakt. Wenn ich /app/config/config.yml anlege mit dem Inhalt:

contao: url_suffix: ''

... dann funktioniert auf einer News-Reader-Seite die Preview-Funktion nicht mehr (normaler Aufruf ohne Preview klappt).

fritzmg commented 3 years ago

Super, funktioniert jetzt einwandfrei.

Passt, habe ich als Version 1.2.1 veröffentlicht.

Zu contao-simple-news-urls habe ich in einer frischen 4.9.5 herausgefunden, dass es wohl beim Zusammenspiel mit fehlenden Suffixen hakt. Wenn ich /app/config/config.yml anlege mit dem Inhalt:

contao: url_suffix: ''

... dann funktioniert auf einer News-Reader-Seite die Preview-Funktion nicht mehr (normaler Aufruf ohne Preview klappt).

Hm, bitte dazu im jeweiligen Repository ein Issue anlegen.