Fehler in showFrontendDropdownMenu.inc.php

barnie

Anfänger

Beiträge: 43

CMS Version: 4.0.7 FINAL

Wohnort: Bayern ganz oben

1

Samstag, 18. Juni 2011, 14:24

Hallo Moderatoren und Entwickler,

es gibt noch einen Fehler in der Funktion showFrontendDropdownMenu.

Wenn man Formatierungen bei "first, middle, last" angibt werden diese nicht richtig umgesetzt.

Das betrifft speziell die 2. (schliessende) Formatierung.

Auszug Orginal Code (aktuelles Update)

Quellcode

 if ($level_2_count > '0') { 

print "\n ".$blockEx['1']."\n\n"; 
print $close_list_level_1; 

} else { 

print $close_list_level_1; 

} 

} else { 

print $close_list_level_1; 

}

So sollte es richtig funktionieren (Auszug).

Quellcode

 if ($level_2_count > '0') { 

print "\n ".$blockEx['1']."\n\n"; 
print $close_list_level_1; 

} else { 

print $close_list_level_2; 

} 

} else { 

print $close_list_level_1; 

}

Ich habe die korrigierte Version angehängt (mit Kommentaren).

Gruß Barnie :rolleyes:

»barnie« hat folgende Datei angehängt:

showFrontendDropdownMenu.inc.zip (2,67 kB - 5 mal heruntergeladen - zuletzt: 21. Juni 2011, 23:07)

Machmal sieht man den Wald vor lauter Bäumen nicht.

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »barnie« (18. Juni 2011, 15:58) aus folgendem Grund: Ergänzung

Zum Seitenanfang

cms2day

Profi

Beiträge: 1 282

CMS Version: 4.x/5.x

Beruf: Webentwickler

Paypal:

2

Montag, 20. Juni 2011, 12:42

Also im aktuellen hochgeladenen ZIP finde ich keinen Fehler.
Seit dem Release der RC am 15.05. wurde schon mehrmals Dateien geupdatet.
Bitte mal nachschauen, das sollte längst problemlos funktionieren...

Zu deinen Kommentaren:

	PHP-Quelltext
1	// $level_1_check anstatt mysql_num_rows($level_1)

etc - das kommt auf's selbe raus, da beides funktioniert...

Zum Seitenanfang

barnie

Anfänger

Beiträge: 43

CMS Version: 4.0.7 FINAL

Wohnort: Bayern ganz oben

3

Montag, 20. Juni 2011, 21:41

Hallo cms2day,

ich habe mal in der Funktion die Nummern (Level - Nummer) mit ausgeben lassen. Dabei kommt folgendes Ergebnis raus.

Quellcode

<div id="menu"> 
<ul> 
<li><a ...></a>                    // Level 1-1 
<ul> 
<li><a ...></a>                    // Level 2-1 
<ul> 
<li><a ...></a>level2-1</li>  // level 3-1 
</ul> 
level3-1</li>                        // ende level 2-1 
<li><a ...></a>                   // level 2-2 
<ul> 
<li><a ...></a>                   // level 3-1 
<ul> 
<li><a ...></a>level4-1</li> // level 4-1 
</ul> 
level4-1</li>                       // ende level 3-1 
</ul> 
level3-1</li>                       // ende level 2-2 
</ul> 
level1-1</li>                       // ende level 1-1 
<li><a ...></a>level1-2</li> // level 1-2 
<li><a ...></a>                  // level 1-3 
<ul> 
<li><a ...></a>level2-1</li> // level 2-1 
</ul> 
level1-3</li>                       // ende level 1-3 
<li><a ...></a>level1-4</li> // level 1-4 
</ul> 
</div>

Wie man am Beispiel erkennen kann werden die Level nicht richtig abgeschlossen.

Gruß Barnie

Machmal sieht man den Wald vor lauter Bäumen nicht.

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »barnie« (20. Juni 2011, 21:46)

Zum Seitenanfang

barnie

Anfänger

Beiträge: 43

CMS Version: 4.0.7 FINAL

Wohnort: Bayern ganz oben

4

Montag, 20. Juni 2011, 21:56

Hallo cms2day

der Fehler fällt eigentlich nicht auf, da </li> (level2) = </li> (level3, ...).

Wird aber die Funktion erweitert oder zusätzliche Formatierungen

angegeben, kommt nicht das gewünschte Ergebniss heraus.

Gruß Barnie

Machmal sieht man den Wald vor lauter Bäumen nicht.

Zum Seitenanfang

emet

Admin & Support

Beiträge: 2 530

Wohnort: Rhein/Main

Paypal:

5

Dienstag, 21. Juni 2011, 15:02

es ist egal wie - es funzt beides, wenn ich die klassen formatiere

im Beispiel hab ich nur kurz Klassen vergeben:

border maroon unten für more
und für first, middle, last rechts grün, blau, rot

Bitte nicht so genau sein, da es teilweise überlagert wird (ich habe nicht alles überarbeitet)

Man sieht es aber genau am 1 Menü - alles bis auf eins maroon border-bottom

und ganz rechts grün, blau, rot beim der letzten Menuebene

Der Versuch wurde mit der bisherigen und deiner Datei durch geführt

Nimm mal die Originaldatei und lade dir das Neue template-cms2day-393 (da ist das normale dropdown menu drin) runter ... dann fügst du in die main.css einfach mal diesen Code hinzu

Quellcode

/* - MENU: MORE - FIRST - MIDDLE - LAST - */

#menu li.more {	
	border-bottom: 3px solid maroon; }

#menu li.first {	
	border-right: 2px solid green; }

#menu li.middle {	
	border-right: 2px solid navy; }

#menu li.last {	
	border-right: 2px solid red; }

Du wirst sehen, es geht

siehe auch screenshot ...

.

»emet« hat folgendes Bild angehängt:

.
HTML, was ist das? Ein neues Männermagazin? Css.., was es so alles gibt!

News: http://www.cms2day.de/news/1/
Features: http://www.cms2day.de/cms-funktionen/

Zufrieden mit cms2day und unserer Arbeit/Support - Spenden (-Button links) sind herzlich Willkommen - Dankeschön.
.

Zum Seitenanfang

barnie

Anfänger

Beiträge: 43

CMS Version: 4.0.7 FINAL

Wohnort: Bayern ganz oben

6

Dienstag, 21. Juni 2011, 17:03

Hallo emet,

habe gerade nochmal die RC 3.9.3 inklusive Template cms2day_393 heruntergeladen und installiert (main.css nach deinen Vorgaben angepasst).

Die Funktion "showFrontendDropdownMenu" scheint auf den ersten Blick richtig zu funktionieren.

Ändert man den Aufruf in der index.html vom Template (habe als Bsp. mal Text verwendet)

	PHP-Quelltext
1	<?php showFrontendDropdownMenu('0','<ul>\|</ul>','<li class="middle">\|</li>','<li class="first">\|</li>','<li class="last">\|AA</li>'); ?>

zusätzlich in main.css folgende Formatierung ergänzt.

	Quellcode
1 2	#menu ul { color: black; }

Dann kommt folgendes Ergebniss zustande !

Es fehlt einmal der Text "AA" in Level 3 (unter Level3-3). Dieser wird bei Level3-2 am Ende ausgegeben, wo er nicht hingehört.

Gruß Barnie :rolleyes:

Machmal sieht man den Wald vor lauter Bäumen nicht.

Zum Seitenanfang

cms2day

Profi

Beiträge: 1 282

CMS Version: 4.x/5.x

Beruf: Webentwickler

Paypal:

7

Dienstag, 21. Juni 2011, 23:10

Okay, habs nun auch gefunden - wird umgehend korregiert und neu hochgeladen, danke!

:thumbup:

Zum Seitenanfang

barnie

Anfänger

Beiträge: 43

CMS Version: 4.0.7 FINAL

Wohnort: Bayern ganz oben

8

Mittwoch, 22. Juni 2011, 00:01

freut mich das der Fehler korrigiert wird

Danke

@cms2day

Zitat von »cms2day«
Zu deinen Kommentaren:
PHP-Quelltext
1
// $level_1_check anstatt mysql_num_rows($level_1)
etc - das kommt auf's selbe raus, da beides funktioniert...

$level_1_check liefert den Inhalt der Variablen (schneller)

mysql_num_rows($level_1) ruft auf dem Server die Funktion auf und liefert das gleiche Ergebnis

Das Ergebnis ist das gleiche, der Unterschied liegt in der Performance.

Gruß Barnie

Machmal sieht man den Wald vor lauter Bäumen nicht.

Zum Seitenanfang

emet

Admin & Support

Beiträge: 2 530

Wohnort: Rhein/Main

Paypal:

9

Mittwoch, 22. Juni 2011, 11:22

Danke nochmal für deinen Hinweis. Wenn alles funzt vermutet man ja keinen Fehler. Aber manchmal ist es auch anders. Es gibt ja dann noch Richtig und Richtiger :-)

.
HTML, was ist das? Ein neues Männermagazin? Css.., was es so alles gibt!

News: http://www.cms2day.de/news/1/
Features: http://www.cms2day.de/cms-funktionen/

Zufrieden mit cms2day und unserer Arbeit/Support - Spenden (-Button links) sind herzlich Willkommen - Dankeschön.
.