search

FriendsOfREDAXO / yrewrite_scheme

Stellt verschiedene URL-Schemes für YRewrite mehrsprachig zur Verfügung
MIT License
37 stars 7 forks source link

Sprache berücksichtigen lib/yrewrite_url_schemes.php:63 #23

Closed mg-gundb closed 4 years ago

mg-gundb commented 4 years ago

if ($art->isStartArticle() && ($cats = $art->getCategory()->getChildren(true)) && !rex_article_slice::getFirstSliceForCtype(1, $art->getId(), rex_clang::getCurrentId())) {

rex_clang::getCurrentId() durch $art->getClang() ersetzen

tbaddade commented 4 years ago

Kannst du das Problem genauer erklären?

rex_clang::getCurrentId() durch $art->getClang()

Warum sollte das ersetzt werden? Beides liefert die Sprache.

mg-gundb commented 4 years ago

Hallo Thomas

Entschuldige, ein salopper Eintrag, habe den als Reminder für Thomas Skerbis gesetzt. Der Auslöser kommt von hier: https://friendsofredaxo.slack.com/archives/C1BAXLN2F/p1574946119136600 https://friendsofredaxo.slack.com/archives/C1BAXLN2F/p1574946119136600

Soll ich das in GitHub noch präzisieren?

gruess matthieu

Am 28.11.2019 um 16:10 schrieb Thomas Blum notifications@github.com:

Kannst du das Problem genauer erklären?

rex_clang::getCurrentId() durch $art->getClang()

Warum sollte das ersetzt werden? Beides liefert die Sprache.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FriendsOfREDAXO/yrewrite_scheme/issues/23?email_source=notifications&email_token=AICBNVCHALIKVHSXM2RUGMDQV7NP7A5CNFSM4JSU3HY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFM3WII#issuecomment-559528737, or unsubscribe https://github.com/notifications/unsubscribe-auth/AICBNVA5C6RQEQ4444B2IYDQV7NP7ANCNFSM4JSU3HYQ.

tbaddade commented 4 years ago

Soll ich das in GitHub noch präzisieren?

Wenn @skerbis weiß was hier gemeint ist, wäre alles gut. Ein Anderer wird sich das Issue mit den wenigen Infos vermutlich nicht annehmen können.

Zur Info: Die Nachrichten in Slack verschwinden irgendwann. Wir haben dort nur 10.000 frei. D.h., dein Link wird irgendwann ins Leere laufen.

skerbis commented 4 years ago

@tbaddade alles gut. Aus irgendeinem noch zu erforschenden Grund scheint rex_clang::getCurrentId() dort nicht zu greifen. Ich werde das analysieren. getClang aus der YRewrite-Class soll das Problem gelöst haben.

skerbis commented 4 years ago

Bezieht sich auf: https://github.com/FriendsOfREDAXO/yrewrite_scheme/blob/f8ba241fe7e0cd169e1badf894d91390aff1183f/lib/yrewrite_url_schemes.php#L63