[simple_news_urls] News direkt über Alias aufrufen

[Spooky]

Habe nun die Implementation aus Nachrichten URL anpassen: www.domain.de/news-alias als Extension im ER2 und über Composer veröffentlicht. Mit dieser Extension können News nun direkt über deren Alias aufgerufen werden, also zB example.org/news-entry.html statt example.org/reader-page/news-entry.html. Auch alle von Contao erzeugten News-URLs benutzen die kurze Form. Darüberhinaus befindet sich mit dieser Extension ein Redirect auf die kurze URL im Newsreader, falls der Newseintrag über die lange URL aufgerufen wird, um duplicate content zu vermeiden.

Dies funktioniert auch mit beliebigen URL Suffixen, auch ganz ohne.

• ER2: [simple_news_urls]
• Packagist: fritzmg/contao-simple-news-urls
• GitHub: fritzmg/contao-simple-news-urls

Hinweis: durch die Verwendung des generateFrontendUrl Hooks könnte es sein, dass sich die Performance der Website verschlechtert. Diese Funktion wird innerhalb von Contao natürlich potentiell sehr oft aufgerufen und mit dieser Extension wird auch jedesmal eine zusätzliche Datenbankabfrage gemacht, falls die erzeugte URL Parameter hat (um eben zu überprüfen, ob es sich bei der URL um eine News-URL handelt).

[Kahmoon]

Also quasi wie urlcleaner nur ohne Config File?

Geht das dann auch mit Strukturen wie:

domain.de/service/aktuelles/alias.html

statt

domain.de/service/aktuelles/reader-page/items/alias.html

[Spooky]

Also quasi wie urlcleaner nur ohne Config File?

Naja nicht ganz, diese Funktionalität geht mit dem urlcleaner nicht, außer ich übersehe etwas.

Geht das dann auch mit Strukturen wie:

domain.de/service/aktuelles/alias.html

statt

domain.de/service/aktuelles/reader-page/items/alias.html

Nein, wenn dann nur domain.de/alias.html - das was du haben möchtest geht wiederum am besten mit dem urlcleaner

[Kahmoon]

OK. Gecheckt :-)

Danke

[tab]

Habe nun die Implementation aus Nachrichten URL anpassen: www.domain.de/news-alias als Extension im ER2 und über Composer veröffentlicht. Mit dieser Extension können News nun direkt über deren Alias aufgerufen werden, also zB example.org/news-entry.html statt example.org/reader-page/news-entry.html. Auch alle von Contao erzeugten News-URLs benutzen die kurze Form. Darüberhinaus befindet sich mit dieser Extension ein Redirect auf die kurze URL im Newsreader, falls der Newseintrag über die lange URL aufgerufen wird, um duplicate content zu vermeiden.

Da sollte man dann wohl allerdings die Redakteure briefen, dass die nicht versehentlich ein News-Alias generieren, das mit einer statischen Seite kollidiert.

[Spooky]

Da sollte man dann wohl allerdings die Redakteure briefen, dass die nicht versehentlich ein News-Alias generieren, das mit einer statischen Seite kollidiert.

Yep . Das könnte ich jedoch in die Extension einbauen. Also eine Überprüfung beim anlegen einer News, ob der Alias bereits als Seitenalias existiert und umgekehrt auch beim anlegen einer Seite, ob der Alias bereits als News Alias existiert.

[tab]

Das wäre toll.

[mandrake]

Dies wäre eine essenzielle Funtion, die für den produktiven Betrieb der Erweiterung benötigt wird. Daher wäre es super, wenn Du das bei Gelegenheit implementieren könntest.

[Spooky]

Version 1.1.0 hat nun den Alias Check. Darüberhinaus sollte die Extension nun auch bei Contao Installationen mit mehreren Domains und/oder Sprachen funktionieren.

[c.adam]

Hallo zusammen,

die Erweiterung ist auch für uns sehr interessant. Wir haben sie einmal bei unsere Seite installiert und getestet. Leider kommen beim Aufruf von Seiten über den News-Alias nur 404-Fehler.

URLs umschreiben ist mit den korrekten Optionen aktiv. Was kann da schief laufen?

[Spooky]

Hm, ist die Seite wo online, damit man sich das mal zumindest im Frontend ansehen kann? Ansonsten bräuchte ich wohl zum debuggen FTP Zugriff.

[c.adam]

Hallo,

hier der Link:

http://www.adam-transporte.de/de/kun...rmationen.html

Die Seite verwendet ein an uns angepasstes Template der News-Übersicht. Die Links funktionieren aktuell nicht. Wenn wir Ihre Extension deaktivieren, dann kommen wieder die normalen funktionsfähigen Links.

Beim Standard-Template für die News-Übersicht erhalte wir das identische Problem. Am Template kann es also nicht liegen.

[Spooky]

Ist da Version 1.1.0 der Extension im Einsatz? Wenn ja dann teste ich nochmal die Funktionalität bei aktivierter Option "Sprache zur URL hinzufügen", evt. habe ich da etwas übersehen.

[Spooky]

Also ich habe es heute nochmal überprüft: die URLs zu den News sollten eigentlich so aussehen: http://www.adam-transporte.de/de/buerozeiten.html
Derzeit sehen sie auf der Website jedoch so aus: http://www.adam-transporte.de/buerozeiten.html - also ohne Language Fragment in der URL. Das sollte aber eigentlich nicht passieren, selbst mit Version 1.0.0, theoretisch.

[c.adam]

Wir nutzen die Erweiterung [i18nl10n] für die Mehrsprachigkeit. Gibt es da vielleicht eine Inkompatibilität mit [simple_news_urls] ?

Ansonsten sehe ich gerade keinen Ansatzpunkt.

[Spooky]

Das kann sein. Diese Erweiterung habe ich selbst noch nie verwendet. Ist bei Verwendung dieser Erweiterung im Backend der Punkt "Sprache zur URL hinzufügen" aktiviert, oder darf man das im Verbindung mit dieser Erweiterung nicht aktivieren?

[c.adam]

Hier mal ein Screenshot der Einstellungen.

Unbenannt.JPG

Wir verwenden beide Extensions in der aktuellen Version.

Ich habe jetzt einmal [simple_news_urls] wieder deaktiviert. Dann werden sofort die korrekten Links angezeigt. Aber man kann die simple_news_urls aber halt auch nicht mehr nutzen

[Spooky]

Ok ja, da dürfte aktuell meine Extension nicht mit [i18nl10n] kompatibel sein. Wird sich aber leicht ändern lassen, wahrscheinlich kann ich heute Abend dazu ein Update liefern. Ist auch mal als Issue hier festgehalten: https://github.com/fritzmg/contao-si...-urls/issues/4

[Spooky]

Ich habe jetzt einmal [simple_news_urls] wieder deaktiviert. Dann werden sofort die korrekten Links angezeigt. Aber man kann die simple_news_urls aber halt auch nicht mehr nutzen

Ja, wie gesagt, [simple_news_urls] ist derzeit wahrscheinlich schlicht inkompatibel mit [i18nl10n]. Das zu beheben wird aber nicht lange dauern. Wenn es einen neuen Release gibt, poste ich es auch hier.

[c.adam]

Super Sache!!!

Ansonsten ist Deine Extension nämlich eine tolle Funktionserweiterung. Vielen Dank für den Support!

[Spooky]

Ersetze mal die Datei simple_news_urls/classes/SimpleNewsUrls.php hiermit: https://github.com/fritzmg/contao-si...leNewsUrls.php
Ist nur ein grober Hack. Aber wenn es funktioniert, veröffentliche ich das mal als neue Version.

[c.adam]

Habe die Datei eingespielt und einige Tests gemacht. Das Problem ist mit dieser Anpassung gelöst.

Vielen Dank im Voraus für die Veröffentlichung des Patches.

[Spooky]

Version 1.1.2 mit dieser Änderung ist nun released.

[c.adam]

...und bei uns eingespielt.

Daumen hoch!

[Spooky]

Ab Version 2.0.0 sollte die Extension nun mit anderen Extensions kompatibel sein, die normalerweise den Newsreader von Contao ersetzen.

[matthias_g]

Wow! Das ist genau das, was ich die letzten Tage gesucht habe! Danke dafür*…

Allerdings scheint es, als habe ich Probleme bei der Konfiguration?
Muss noch irgendwas besonderes beachtet werden, .htaccess Datei anpassen oder so?


Liebe Grüße und besten Dank

[Spooky]

Eigentlich nicht, welche Probleme hast du?

[matthias_g]

Also ich werde zwar auf die News gelinkt … allerdings bekomme ich folgenden Fehler?

Bildschirmfoto 2016-08-05 um 08.55.33.png

[Spooky]

Hmm ok. Da müsstest du mir evt. Zugangsdaten zukommen lassen, damit ich mir ansehen kann, warum das passiert (FTP + Contao Backend).

[matthias_g]

Hmm ok. Da müsstest du mir evt. Zugangsdaten zukommen lassen, damit ich mir ansehen kann, warum das passiert (FTP + Contao Backend).

Super nett von dir! Ich schicke dir noch eine PN!

Danke!

[cgreiss]

Hallo,

ich habe die simple_news_urls erfolgreich in Benutzung, genau was ich gesucht hatte!

Nun nutze ich bei vielen Newsmeldungen das Contao Content Element Download oder Downloads. Und die Downloads funktionieren nicht mehr auf den News. Nach Deinstallation der Erweiterung geht alles wieder wie gewohnt.

Mit der Erweiterung wird nur die selbe Seite (=Newsmeldung) neu geladen, aber der Downloadvorgang startet nicht.

Viele Grüße

chris

[Spooky]

Link zur Seite?

[cgreiss]

sorry, hier der link:

http://2016.md-institute.com/de/pressemitteilungen.html

Bei allen Pressemiteilungen dort gibt es einen Download, der mit simple_news_url nicht geht, ohne schon.

Die Seite geht bald online, daher weiß ich nicht, wie lange das Phänomen unter dem Link noch zu sehen ist, ich muß morgen wieder deinstallieren, damit die downloads funktionieren.

Aber ich denke, der fehler (wenns denn einer ist) ist leicht zu reproduzieren.

LG
Chris

[Spooky]

Ok, habe das Problem schon gefunden, werde heute noch ein Update rausbringen

[Spooky]

Version 2.0.4 wurde so eben veröffentlicht, das sollte das Problem beheben.

[cgreiss]

Hallo Spooky,

habe grade geupdatet, nun klappts auch mit den Downloads!

Super, und so schnell! Tausend Dank!

LG
Chris

[*igi*]

Hallo zusammen,

ich habe die Erweiterung auch mal installiert und durfte feststellen, dass diese genau das macht was ich gesucht habe. Recht herzlichen Dank an dieser Stelle
Zwei Fragen bzw. Punkte habe ich gerade noch:

a) Gibt es hinsichtlich der erwähnten Probleme mit der Performance Erfahrungswerte, wie sich dies in der Praxis auswirkt?
Wird die gesamte Seite hierdurch spürbar langsamer bzw. ist dies bei Tools wie z.B. Google Pagespeed bemerkbar?

b) Ich habe festgestellt, dass das Plugin bestehende Aliase der Seitenstruktur überschriebt, sofern diese gleich sind.
Beispiel: Seite www.domain.de/demoseite wurde über die Seitenstruktur angelegt. Wird jetzt bei einem Nachrichtenelement ebenfalls der Alias "demoseite" verwendet, ist die ursprüngliche Seite "demoseite" nicht mehr erreichbar. Es wird stattdessen der Nachrichtenartikel angezeigt.

Dies ließe sich z.B. durch eine Abfrage wie die folgende beheben (Klasse SimpleNewsUrls.php, Funktion getPageIdFromUrl()):

// check if an alias was extracted
        if( $alias )
        {
            // new: check if page alias exist
            if( ( $objPage = \PageModel::findByAlias( $alias ) ) !== null )
            {
                return $arrFragments;
            }
... 

Hier wird zunächst geprüft, ob eine Seite zu dem jeweiligen Alias existiert.
Was jedoch eine weitere Datenbankabfrage bedeuten würde.
Ist es denbar / sinnvoll einen solchen Schutz vor Überschreibungen einzufügen (ist jetzt nur eine Idee / ein Beispiel, muss natürlich nicht genau so aussehen)?
Könnte man evtl. generell dafür sorgen, dass dieser Mechanismus nur für Nachrichtenartikel ausgeführt wird und so auch die restliche Webseite nicht tangiert?

[Spooky]

a) Gibt es hinsichtlich der erwähnten Probleme mit der Performance Erfahrungswerte, wie sich dies in der Praxis auswirkt?
Wird die gesamte Seite hierdurch spürbar langsamer bzw. ist dies bei Tools wie z.B. Google Pagespeed bemerkbar?

Google Page Speed würde wenn dann eine erhöhte Response Time beanstanden.

b) Ich habe festgestellt, dass das Plugin bestehende Aliase der Seitenstruktur überschriebt, sofern diese gleich sind.
Beispiel: Seite www.domain.de/demoseite wurde über die Seitenstruktur angelegt. Wird jetzt bei einem Nachrichtenelement ebenfalls der Alias "demoseite" verwendet, ist die ursprüngliche Seite "demoseite" nicht mehr erreichbar. Es wird stattdessen der Nachrichtenartikel angezeigt.

Dies ließe sich z.B. durch eine Abfrage wie die folgende beheben (Klasse SimpleNewsUrls.php, Funktion getPageIdFromUrl()):

// check if an alias was extracted
        if( $alias )
        {
            // new: check if page alias exist
            if( ( $objPage = \PageModel::findByAlias( $alias ) ) !== null )
            {
                return $arrFragments;
            }
... 

Auf diese Weise ist aber dann der News Artikel nicht mehr erreichbar.

Ist es denbar / sinnvoll einen solchen Schutz vor Überschreibungen einzufügen (ist jetzt nur eine Idee / ein Beispiel, muss natürlich nicht genau so aussehen)?
Könnte man evtl. generell dafür sorgen, dass dieser Mechanismus nur für Nachrichtenartikel ausgeführt wird und so auch die restliche Webseite nicht tangiert?

Diesen Schutz gibt es eigentlich bereits: https://github.com/fritzmg/contao-si...ws.php#L32-L45

Falls also der Alias einer URL der selbe wie von einer Seite innerhalb der selben Domain und Sprache wäre, würde die ID an den Alias angehängt werden. Aber evt. funktioniert dies nicht richtig?

[*igi*]

Ich habe mir das nochmal angesehen.
Den save_callback hatte ich tatsächlich übersehen.

In der checkAlias Funktion springt er bei mir an folgender Stelle raus:

// check if page is on the same domain and language
if( $objPage->domain == $objTarget->domain && ( \Config::get('addLanguageToUrl') && $objPage->rootLanguage == $objTarget->rootLanguage ) )
{
    // append id
    $varValue.= '-' . $dc->id;
} 

Zum einen ist $objPage->domain nicht gesetzt (ich arbeite momentan über localhost, liegt evtl daran?) und zum anderen hatte ich den Wert "Die Sprache zur URL hinzufügen" (in den BE Settings) nicht aktiviert.
Wenn ich diese Abfrage auskommentiere, funktioniert das mit dem Anhängen der ID auch korrekt.

Habe ich in meiner Konfiguration etwas falsch gemacht oder kann es sein, dass das letzte "&&" in der Abfrage eigentlich ein "||" sein sollte? Dies würde auch die extra Klammern erklären.

Also:

// check if page is on the same domain and language
if( $objPage->domain == $objTarget->domain && ( \Config::get('addLanguageToUrl') || $objPage->rootLanguage == $objTarget->rootLanguage ) )
{
    // append id
    $varValue.= '-' . $dc->id;
} 

[Spooky]

Die Bedingung für das anhängen der ID ist, dass sich die Seite mit dem selben Alias, und die Zielseite des Newsarchivs der News in der selben Domain und in der selben Sprache befinden müssen. Die Bedingung sollte daher stimmen.

Du hast nur ein paar der Werte ermittelt. Um herauszufinden, warum es bei dir nicht funktioniert hat, musst du die Ausgabewerte folgender Variablen ermitteln:

$objPage->domain
$objTarget->domain
\Config::get('addLanguageToUrl')
$objPage->rootLanguage
$objTarget->rootLanguage