WBCE CMS Forum

colinax

Developer

Menulink-Ziele direkt ausgeben statt Weiterleitung

Sind in der menu_link 2.9.x die zwei Threads (hier und hier) auch eingeflossen?

Edit: Link korrigiert

Last edited by florian (30.10.2018 14:58:24)

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

Nein. Das betrifft ja nicht das menu_link-Modul, sondern die da müsste der Core bzw. sm2 angepasst werden.

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

colinax

Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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.

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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

Last edited by stefanek (30.10.2018 14:52:05)

---

“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale

colinax

Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

@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

Last edited by florian (30.10.2018 14:58:53)

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

@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

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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

Last edited by stefanek (30.10.2018 14:54:57)

---

“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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?

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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)

Last edited by florian (30.10.2018 15:18:51)

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

colinax

Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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

Last edited by colinax (30.10.2018 15:22:19)

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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   

Last edited by stefanek (30.10.2018 15:39:56)

---

“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

OK, habe es noch einmal eingespielt. Müsste jetzt funktionieren.

Gruß,
Christian

---

“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

Wo Ihr eh gerade sm2 am Wickel habt, könnt Ihr bitte hier mal schauen:
https://forum.wbce.org/viewtopic.php?pid=20226#p20226

Last edited by florian (30.10.2018 18:29:41)

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

Ja, kann ich gleich mit einbinden. Hast ja schon gelöst :-)

---

“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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?

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

colinax

Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

https://github.com/WBCE/WBCE_CMS/commit … 584cc60665

Last edited by colinax (16.11.2018 21:31:09)

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

?

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

colinax

Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

Oh sorry das war der falsche Link, hab ihn oben in #17 korrigiert.

Der Auslöser für diese Lösung war dieser Beitrag.

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

@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

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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.

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.

stefanek

Core-Team

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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

florian

Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

Danke, getestet & funktioniert - ich hab's gemerged und auch in 1.3.3 nachgezogen

---

Sorgen sind wie Nudeln: man macht sich meist zu viele.