rupertgermann / tt_news

TYPO3 Extension - News (tt_news)
21 stars 37 forks source link

Links zu News geändert #129

Open FranzBl opened 5 years ago

FranzBl commented 5 years ago

Hallo Herr Germann,

mit Ihrer Hilfe läuft tt_news auf typo3 9.5.8 jetzt anstandslos. Leider ist mir etwas aufgefallen, was recht unschön ist. Die URLs sämtlicher alten News, die unverändert von der Vorinstallation unter typo3 7.6.32 übernommen wurden (über den Umweg über 8.7.26 natürlich!), haben sich gegenüber früher geändert. Ich weiß nicht, ob das eine programmtechnische Notwendigkeit war. Auf jeden Fall funktionieren jetzt die Links von Facebook zu unserer Webseite, wo ich einige Beiträge geteilt hatte, nicht mehr. Die Links landen jetzt entweder auf unserer 404-Fehlerseite und damit bei der Sitemap, oder sie gehen zur allgemeinen News-Übersicht. Für Facebook-Benutzer, die sich unsere älteren Beiträge ansehen, natürlich ärgerlich. Kann man da was tun (irgendeine Kompatibilitätseinstellung)? Notfalls könnte ich das Upgrade noch mal machen, da ich die alten Versionen noch im Backup habe.

rupertgermann commented 5 years ago

haben sie mal ein Beispiel für "Link vorher" und "Link nachher" ?

FranzBl commented 5 years ago

Gerne.

Hier ein alter Link:

https://www.matrix3000.de/home/news/?tx_ttnews%5Btt_news%5D=954&cHash=56ee716b209184048111d0dcd84003f8https://www.matrix3000.de/home/news/?tx_ttnews%5Btt_news%5D=954&cHash=56ee716b209184048111d0dcd84003f8&fbclid=IwAR1i7cYp1kJLxXucCVKNrfNnu52xqnSGPxQApdtRWc1-sTrFfpu4rLcOPz8

und dazu der neue:

https://www.matrix3000.de/default-71d63b1a4c?tx_ttnews%5Btt_news%5D=954&cHash=c503ddae22ddf56f750d1cb8e064f97f

Hier ein anderes Beispiel:

Alt: https://www.matrix3000.de/home/news/?tx_ttnews%5Btt_news%5D=978&cHash=ce2cb43b29e85e984951bfab12b4a291

Neu: https://www.matrix3000.de/default-71d63b1a4c?tx_ttnews%5Btt_news%5D=978&cHash=a87280b4f018d338b20cbbcf27ac2d4e

In beiden Fällen fällt auf, dass im neuen System der News-Ordner home/news nicht mehr in der URL auffällt, doch auch die Hash-Codes stimmen ja nicht mehr überein.

Von: Rupert Germann notifications@github.com Gesendet: Sonntag, 11. August 2019 15:26 An: rupertgermann/tt_news tt_news@noreply.github.com Cc: FranzBl franz.bludorf@outlook.com; Author author@noreply.github.com Betreff: Re: [rupertgermann/tt_news] Links zu News geändert (#129)

haben sie mal ein Beispiel für "Link vorher" und "Link nachher" ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frupertgermann%2Ftt_news%2Fissues%2F129%3Femail_source%3Dnotifications%26email_token%3DAKUJKFM7PVYAA62KACY4T4DQEAHQBA5CNFSM4IK3SKW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4BA42A%23issuecomment-520228456&data=02%7C01%7C%7C06b6efa2f98843c07cb608d71e5f829d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637011267861805721&sdata=rBlWxi2%2FwRD%2B6WHz2rNKLwpvP6pUx3%2FYg%2BCXJg1OZKg%3D&reserved=0, or mute the threadhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKUJKFIEODSOGLJY7UYBDPLQEAHQBANCNFSM4IK3SKWQ&data=02%7C01%7C%7C06b6efa2f98843c07cb608d71e5f829d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637011267861815730&sdata=vtfj3AnBPBinPSzi7K8%2BQprZZoAIijIM24g%2Fnlhq9t8%3D&reserved=0.

FranzBl commented 5 years ago

Noch zur Ergänzung: Ich habe an den alten News nichts rumgefummelt beim Upgrade, nur jedes Mal vor dem Upgrade schön brav tt_news deaktiviert und nach dem Umstellen der neuen DB-Tabellen sofort wieder, in der dann passenden Version, neu installiert, damit der DB-Upgrade mir die News nicht löscht. Die Daten wurden so ganz einfach so lange weiter mitgeschleppt, bis Version 9.5.8 stand. A propos: Ich hatte deshalb so lange gewartet mit dem Upgrade von der alten Version, weil es bis Juni diesen Jahres tt_news nicht mal für Version 8 gab. Und das neue News System von Georg Ringer ist noch zu sehr voller Kinderkrankheiten. Er kriegt Imagemagick nicht gebacken und bringt zu den News anstelle der Thumbnails seitenfüllende Bilder, die dann die Nachbarnews überdecken. Auch hat das neue System manche schönen Features nicht mehr drin, wie z. B. das Anzeigen zufälliger Nachrichten auf der Startseite, was tt_news klaglos erledigt. Also nicht locker lassen, dranbleiben…

Viele Grüße

Franz Bludorf

Von: Rupert Germann notifications@github.com Gesendet: Sonntag, 11. August 2019 15:26 An: rupertgermann/tt_news tt_news@noreply.github.com Cc: FranzBl franz.bludorf@outlook.com; Author author@noreply.github.com Betreff: Re: [rupertgermann/tt_news] Links zu News geändert (#129)

haben sie mal ein Beispiel für "Link vorher" und "Link nachher" ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frupertgermann%2Ftt_news%2Fissues%2F129%3Femail_source%3Dnotifications%26email_token%3DAKUJKFM7PVYAA62KACY4T4DQEAHQBA5CNFSM4IK3SKW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4BA42A%23issuecomment-520228456&data=02%7C01%7C%7C06b6efa2f98843c07cb608d71e5f829d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637011267861805721&sdata=rBlWxi2%2FwRD%2B6WHz2rNKLwpvP6pUx3%2FYg%2BCXJg1OZKg%3D&reserved=0, or mute the threadhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKUJKFIEODSOGLJY7UYBDPLQEAHQBANCNFSM4IK3SKWQ&data=02%7C01%7C%7C06b6efa2f98843c07cb608d71e5f829d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637011267861815730&sdata=vtfj3AnBPBinPSzi7K8%2BQprZZoAIijIM24g%2Fnlhq9t8%3D&reserved=0.

Patta commented 5 years ago

Could be related to #118

dklimpel commented 4 years ago

Ich habe für mich einen Workaround gefunden. Vorher habe ich Realurl verwendet. Die neuen routeEnhancers benötigen jedoch die ID in der URL. Daher habe ich mir ein Redirect für alle meine alten News erstellt, sodass das Problem für ich gefixt ist. Die Pfadangabe pflegen wie in #133 beschrieben, wollte ich nicht. Mit dem folgenden SQL-Statement lassen sich die Redirects alle auf ein Mal anlegen

INSERT INTO sys_redirect (`source_host`, `source_path`, `force_https`, `keep_query_parameters`, `target`, `target_statuscode`)
SELECT "*", concat("/aktuelles/news/", tx_realurl_uniqalias.value_alias, ".html"), 1, 1, concat("t3://record?identifier=tx_ttnews&uid=", tx_realurl_uniqalias.value_id), 301
FROM tx_realurl_uniqalias
WHERE tablename = "tt_news" and expire = 0;