WBCE Home | WBCE Hilfe | WBCE Addon Repository | Impressum | Datenschutz

WBCE CMS Forum

WBCE CMS – Way Better Content Editing.

You are not logged in.

#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

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

Offline

#2 30.10.2018 11:59:26

florian
Projektmanagement und Support WBCE CMS

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.

Online

#3 30.10.2018 12:45:29

colinax
Developer

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

florian wrote:

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 wrote:
florian wrote:

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 13:52:05)


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

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

Last edited by 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 wrote:

@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


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

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

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


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

Offline

Liked by:

florian

#8 30.10.2018 14:06:22

florian
Projektmanagement und Support WBCE CMS

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?

Online

#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


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

Offline

Liked by:

florian

#10 30.10.2018 14:18:24

florian
Projektmanagement und Support WBCE CMS

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 14:18:51)

Online

#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

Last edited by 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

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


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

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


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

Offline

#14 30.10.2018 17:29:05

florian
Projektmanagement und Support WBCE CMS

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 17:29:41)

Online

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 :-)


For any activity, creative or business oriented alike, always ask yourself what the ROI of your contribution will be?
And by the way: Appreciation is a valid form of exchange for help.

Offline

Board footer

Powered by FluxBB

up