WBCE CMS Forum

WBCE CMS – Way Better Content Editing.

Du bist nicht angemeldet.

#1 30.10.2018 11:54:47

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

Beitrag geändert von florian (30.10.2018 13:58:24)

Offline

#2 30.10.2018 11:59:26

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.


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#3 30.10.2018 12:45:29

colinax
Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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.

Offline

#4 30.10.2018 13:19:24

stefanek
Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

colinax schrieb:
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

#5 30.10.2018 13:25:49

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

Beitrag geändert von florian (30.10.2018 13:58:53)

Offline

#6 30.10.2018 13:36:23

stefanek
Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

colinax schrieb:

@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

#7 30.10.2018 13:54:36

stefanek
Developer

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

Beitrag geändert von stefanek (30.10.2018 13:54:57)


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

Offline

Liked by:

florian

#8 30.10.2018 14:06:22

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?


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#9 30.10.2018 14:09:55

stefanek
Developer

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

Offline

Liked by:

florian

#10 30.10.2018 14:18:24

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)

Beitrag geändert von florian (30.10.2018 14:18:51)


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#11 30.10.2018 14:21:17

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

Beitrag geändert von colinax (30.10.2018 14:22:19)

Offline

#12 30.10.2018 14:39:39

stefanek
Developer

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  big_smile  tongue

Beitrag geändert von stefanek (30.10.2018 14:39:56)


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

Offline

#13 30.10.2018 16:33:04

stefanek
Developer

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

Offline

#14 30.10.2018 17:29:05

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

Beitrag geändert von florian (30.10.2018 17:29:41)


Code allein macht nicht glücklich. Jetzt spenden!

Offline

Liked by:

stefanek

#15 30.10.2018 18:23:31

stefanek
Developer

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

Offline

#16 16.11.2018 16:04:02

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?


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#17 16.11.2018 16:40:08

colinax
Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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

Beitrag geändert von colinax (16.11.2018 20:31:09)

Offline

#18 16.11.2018 16:47:44

florian
Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

?


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#19 16.11.2018 20:33:18

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.

Offline

Liked by:

florian

#20 16.11.2018 22:22:37

stefanek
Developer

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

Offline

Liked by:

florian

#21 25.11.2018 17:29:18

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.


Code allein macht nicht glücklich. Jetzt spenden!

Offline

#22 26.11.2018 13:21:32

stefanek
Developer

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

Offline

#23 26.11.2018 13:33:00

florian
Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

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


Code allein macht nicht glücklich. Jetzt spenden!

Offline

Liked by:

stefanek

Fußzeile des Forums

up