search

FriendsOfREDAXO / yrewrite_scheme

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

kompatibilität mit yrewrite 2.7 #28

Closed isospin closed 4 years ago

isospin commented 4 years ago

in yrewrite 2.7 gibt es das neue feature Startsprache automatisch gemäß Sprache des Client-Browsers, bevorzugte Sprache als Fallback. wenn man diese automatische umleitung aktiviert, ist / nur noch eine weiterleitung, und auch die bevorzugte sprache bekommt als startseite /de/ (o.ä.).

sobald yrewrite_scheme aktiviert ist, wird die url der bevorzugten sprache wieder zu '/'. das führt dazu, das die seite der hauptsprache für user, die diese sprache nicht im browser eingestellt haben nicht aufrufbar ist.

isospin commented 4 years ago

wenn ich die zeile 50 auskommentiere, funktioniert es: https://github.com/FriendsOfREDAXO/yrewrite_scheme/blob/ff0f0244eb00d81cb32cae04b5f44ea9c581c36b/lib/yrewrite_url_schemes.php#L50

        if ($domain->getStartId() == $art->getId()) {
            if ($domain->getStartClang() == $art->getClang()) {
         //       return '/';
            }
            return $this->getClang($art->getClang(), $domain) . $path_suffix;
        }

an der stelle müsste man vermutlich noch abfragen ob die sprachweiterleitung in yrewrite aktiviert wurde...

gharlan commented 4 years ago

an der stelle müsste man vermutlich noch abfragen ob die sprachweiterleitung in yrewrite aktiviert wurde...

Jupp genau: https://github.com/yakamara/redaxo_yrewrite/blob/83fa1c21ca48897c8280795c6d68a65ac8a5d6a8/lib/yrewrite/scheme.php#L82

isospin commented 4 years ago

wenn alles geklappt hat liegt hierzu nun ein pull request vor: https://github.com/FriendsOfREDAXO/yrewrite_scheme/pull/29

skerbis commented 4 years ago

Ist gemerged @isospin, bitte testen. Dann werde ich releasen. Danke

skerbis commented 4 years ago

Leider musste ich den PR reverten, da fehlerhaft

@gharlan @isospin

13.07.2020 13:10:22 | ParseError | syntax error, unexpected '!' | redaxo/src/addons/yrewrite_scheme/lib/yrewrite_url_schemes.php | 49

Habe aber nicht verstanden warum

skerbis commented 4 years ago

ich werde mindestens yrewrite 2.7 voraussetzen.

tyrant88 commented 4 years ago

Du setzt ne beta-Version vorraus?

skerbis commented 4 years ago

ja hier in der GitHub Variante aktuell ja. - Es wird erst verbindlich wenn yrewrite 2.7 im Installer ist.