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 12: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 14:58:24)

Online

#2 30.10.2018 12: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.

Offline

#3 30.10.2018 13: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.

Online

#4 30.10.2018 14: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 14: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 14: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 14:58:53)

Online

#6 30.10.2018 14: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 14: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 14: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 15: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?

Offline

#9 30.10.2018 15: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 15: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)

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

Offline

#11 30.10.2018 15: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 15:22:19)

Online

#12 30.10.2018 15: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 15: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 17: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 18: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

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

Offline

Liked by:

stefanek

#15 30.10.2018 19: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

#16 16.11.2018 17: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?

Offline

#17 16.11.2018 17:40:08

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)

Online

#18 16.11.2018 17:47:44

florian
Administrator

Re: Menulink-Ziele direkt ausgeben statt Weiterleitung

?

Offline

#19 16.11.2018 21: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.

Online

Liked by:

florian

#20 16.11.2018 23: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


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

#21 25.11.2018 18: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.

Offline

#22 26.11.2018 14: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


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

#23 26.11.2018 14: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

Offline

Liked by:

stefanek

Board footer

Powered by FluxBB

up