WBCE CMS – Way Better Content Editing.
Du bist nicht angemeldet.
Sind in der menu_link 2.9.x die zwei Threads (hier und hier) auch eingeflossen?
Edit: Link korrigiert
Beitrag geändert von florian (30.10.2018 13:58:24)
Offline
Nein. Das betrifft ja nicht das menu_link-Modul, sondern die da müsste der Core bzw. sm2 angepasst werden.
Code allein macht nicht glücklich. Jetzt spenden!
Offline
Nein. Das betrifft ja nicht das menu_link-Modul, sondern die da müsste der Core bzw. sm2 angepasst werden.
Oh, ok. Danke für die Info.
Offline
florian schrieb:Nein. Das betrifft ja nicht das menu_link-Modul, sondern die da müsste der Core bzw. sm2 angepasst werden.
Oh, ok. Danke für die Info.
Ja, das stimmt.
Ich könnte es so machen, dass es dann funktioniert, wenn man in die config.php eine Konstante setzt:
define('SM2_CORRECT_MENU_LINKS', false);
Setzt man es dann zu true, werden die menu_link Weiterleitungen problemlos von show_menu2() mit den richtigen Links ersetzt.
Es sind nur einige Änderungen im show_menu2() und das Setzen der Konstante in der config.php
Wenn gewünscht, mache ich einen pullRequest auf GitHub, da ich es lokal bereits so umgesetzt habe.
Gruß,
Christian
Beitrag geändert von stefanek (30.10.2018 13:52:05)
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
@florian
bitte trennen und mit #17 einen neuen Thread erstellen.
@stefanek
Positive verbesserungen finde ich immer gut, nur den Regler in die config.php eher nicht so.
Done
-florian
Beitrag geändert von florian (30.10.2018 13:58:53)
Offline
@stefanek
Positive verbesserungen finde ich immer gut, nur den Regler in die config.php eher nicht so.
Das kann ja eine temporäre Lösung sein.
Die Funktionalität ist erstmal "silent" vorhanden, bis man in einer weiteren Revision einen Schalter in den Settings oder im MenuLink Modul setzt (eher in den Settings, da das MenuLink Modul keine Settings hat).
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Habe den PullRequest gemacht.
Du kannst es also gerne testen.
Es bricht auch nichts zusammen, wenn die Konstante in der config.php nicht vorhanden ist.
Man kann sie dann setzen, wenn man auf diese Funktion zurückgreifen will.
Ich habe sie allerdings in SM2_CORRECT_MENU_LINKS umbenannt.
define('SM2_CORRECT_MENU_LINKS', true);
Gruß,
Christian
Beitrag geändert von stefanek (30.10.2018 13:54:57)
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
florian
Also, für mich zum Verständnis:
1.) SM2 patchen
2.) Konstante in der config.php ergänzen
3.) Menüaufruf in Variable schreiben, dafür im sm2-Aufruf Parameter SM2_BUFFER setzen
4.) da, wo das Menü erscheinen soll, <?php echo pageTreeCorrectedMenuLinks($myMainMenu); ?> einfügen
Korrekt?
Code allein macht nicht glücklich. Jetzt spenden!
Offline
Nein, Punkt 3 und 4 erübrigen sich dann. SM2 ist so angepasst, dass es alle Menüs anpasst, wenn die Flag (Konstante) auf true/1 gesetzt ist.
Also eine "vollautomatische" Lösung, sobald die Konstante auf true gesetzt ist.
Gruß,
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
florian
Cool, funktioniert.
Gibt aber noch ein Warning aus:
Warning: include(legacy.php): failed to open stream: No such file or directory in /.../modules/show_menu2/include.php on line 39
(ich habe nur die include ausgetauscht)
Beitrag geändert von florian (30.10.2018 14:18:51)
Code allein macht nicht glücklich. Jetzt spenden!
Offline
Ja wollte ich auch gerade schreiben, stefanek arbeitet wie es aussieht noch mit WBCE 1.3.1 und nicht mit 1.3.2, habs auch auf Github vermerkt.
Edit: also die include die er aus Grundlage verwendet hat ist aus WBCE 1.3.1 und nicht aus 1.3.2
Beitrag geändert von colinax (30.10.2018 14:22:19)
Offline
Ja, ihr habt recht.
Ich werde jetzt einen neuen WBCE Fork auf meiner GitHub Repo aufsetzen, um solche Vorkommen zu vermeiden.
Jetzt ist es an Colinax es zu bereinigen
Beitrag geändert von stefanek (30.10.2018 14:39:56)
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
OK, habe es noch einmal eingespielt. Müsste jetzt funktionieren.
Gruß,
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Wo Ihr eh gerade sm2 am Wickel habt, könnt Ihr bitte hier mal schauen:
https://forum.wbce.org/viewtopic.php?pid=20226#p20226
Beitrag geändert von florian (30.10.2018 17:29:41)
Code allein macht nicht glücklich. Jetzt spenden!
Offline
stefanek
Ja, kann ich gleich mit einbinden. Hast ja schon gelöst :-)
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Wenn ich den SM2-Patch in eine existierende WBCE 1.3.2 stable installiere, erzeugt die Navigation reproduzierbar einen Fatal Error:
Fatal error: Uncaught Error: Call to undefined function is_countable() in /.../modules/show_menu2/include.php:94
Stack trace:
#0 /.../templates/.../index.php(111): show_menu2(1)
#1 /.../index.php(81): require('/var/www/...')
#2 /.../pages/....php(8): require('/var/www/...')
#3 {main} thrown in /.../modules/show_menu2/include.php on line 94
woran kann das liegen? Gibt es da weitere Abhängigkeiten zu Core-Dateien?
Code allein macht nicht glücklich. Jetzt spenden!
Offline
https://github.com/WBCE/WBCE_CMS/commit … 584cc60665
Beitrag geändert von colinax (16.11.2018 20:31:09)
Offline
?
Code allein macht nicht glücklich. Jetzt spenden!
Offline
Oh sorry das war der falsche Link, hab ihn oben in #17 korrigiert.
Der Auslöser für diese Lösung war dieser Beitrag.
Offline
florian
@florian
Am einfachsten ist, wenn Du es in älteren Versionen verwenden willst, die is_countalbe() Funktion in die framework/functions.php einbinden.
Alternativ, vielleicht noch einfacher: die Zeile
if (is_countalbe($wb->page)) {
in
if (is_array($wb->page)) {
ändern.
Gruß,
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
florian
Ich habe beim Testen gerade noch zwei Probleme gefunden.
1) Die Einstellung SM2_CORRECT_MENU_LINKS, true zeigt keine Wirkung, wenn der SM2-Menüaufruf ohne SM2_BUFFER erfolgt.
2) wenn als Menu Link-Ziel eine entfernte Adresse angegeben ist und dies dann im Nachgang auf eine interne Adresse geändert wird, wird im Frontend weiterhin der Link auf die entfernte Adresse generiert. Das Problem tritt nicht auf, wenn SM2_CORRECT_MENU_LINKS auf false gesetzt wird.
Code allein macht nicht glücklich. Jetzt spenden!
Offline
Sollte beide Issues beheben:
https://github.com/WBCE/WBCE_CMS/pull/372
Gruß,
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Danke, getestet & funktioniert - ich hab's gemerged und auch in 1.3.3 nachgezogen
Code allein macht nicht glücklich. Jetzt spenden!
Offline
stefanek