Verschachtelte ifsets/ifnotsets und neu: ifvalue

[MarkusR]

Da der Original-Parser von ConPresso so seine Probleme mit verschachtelten ifsets und ifnotsets hat, habe ich mal eine Version geschrieben, die diese brav umgekehrt hierarchisch abarbeitet.

Bei ein paar kleinen Tests funktionierte es wie gewünscht. Falls das jemand mal testen will, um hier ein Feedback abzugeben, kann er den Parser hier downloaden:

http://conpresso4.de/_reg_downloads/ind ... =Downloads

Mein Ziel ist es diese Version noch mit den von mir gewünschten werteabhängigen Bedingungen zu versehen (also sowas wie: <!-- ifvalue: Preis>100; -->), aber das ist noch nicht mit drin...

edit: Titel geändert wegen neuem Feature

[balu]

Hi Markus,

interessante Lösung. Ich hatte noch nicht genug Zeit, da reinzuschauen, aber auf Anhieb fallen mir folgende Fragen ein:

1. Was passiert, wenn jemand falsch schachtelt: <!-- ifSet ... --> <!-- /ifNotSet ... -->

2. Was passiert, wenn jemand ein ifSet nicht schliesst? Eventuell $taglevel auf > 0 prüfen und keine Aktion durchführen oder einen Hinweis ausgeben?

3. Hab ich vergessen, weil so viele Anrufe zwischendurch kamen :-/ - fällt mir später vielleicht wieder ein.

Balu

[MarkusR]

zu 1.) der kriegt bei mir eine kostenpflichtige Schulung

zu 2.) das gleiche wie beim Original... nichts!

Ich habe auch schon über irgendeine Prüfung nachgedacht, aber angesichts der endlos vielen Fehler der User, auf die ich niemals kommen würde kann auch keine Prüfung alle Möglichkeiten erfassen.
Aber eine einfach Prüfung auf korrekte Reihenfolge kriege ich vielleicht noch hin...

[Mr. Magpie]

Hallo Markus,

das klingt sehr vielversprechend!! Das bringt uns und Conpresso ein großes Stück weiter - wenn's die Leute vernünftig einsetzen! Aber das wird man sehen...

Einen Riesendank aus Aichtal (b. Stuttgart) von uns!

2010-01-17 UPDATE: Ich habe Markus überzeugen können, den "Enhanced Parser" weiter zu entwickeln und jedem Interessierten kostenlos zur Verfügung zu stellen. Das Ergebnis kann sich sehen lassen.

Den Download des RC2 findet ihr hier: http://community.conpresso.de/viewtopic ... 5411#25411

Eine Anleitung und Beschreibung hier: http://manual.conpresso4.de/_rubric/ind ... rser_index

[MarkusR]

Ich habe nochmal eine neue Version hochgeladen.

Diese prüft entsprechend der Hierarchie alle Verschachtelungen und gibt entsprechende Meldungen aus.

Außerdem ist das if-tag ifvalue jetzt mit dabei!

Anwendungsbeispiele:

Bei einem Preis über 100 Euro soll ein text ausgegeben werden.
<!-- ifvalue: Preis>100 -->Das ist aber teuer!<!-- /ifvalue -->
Das lässt sich auch verschachteln um z.B. Preisspannen auszuwerten.

Auf einer Downloadseite können vom Redakteur bis zu 3 Downloads angegeben werden. Da diese aber als Word, Excel, PDF, ZIP und DXF Format vorliegen können und immer das entsprechende Icon erhalten sollen, müßten 15(!) File-Platzhalter eingebaut werden.
Mit ifvalue gibt man einfach den Dateityp an und lässt das entsprechende Icon setzen, also z.B.<!-- ifvalue: Dateityp=PDF-Datei --><img src="pdf.gif"><!-- /ifvalue><!-- ifvalue: Dateityp=Word-Dokument --><img src="doc.gif"><!-- /ifvalue>...[etc.]...<!-- File: Datei; -->
Man braucht also immer nur 3 Dateifelder bei der Artikelerfassung. Den Dateityp kann man problemlos mit einem Dropdown aus mod_text auswählen.

Werte können mit =, >, <, !=, <= und >= geprüft werden. Die zu prüfenden Felder müssen Textfelder sein (alle anderen machen aus meiner Sicht keinen Sinn).


...und jetzt wo ich das schreibe denke ich man könnte auch gut auf Datumswerte mit einem ifdate Prüfungen machen und möglicherweise noch ein Berechnungs-Tag in der Form Brutto: <!-- expr: Netto*1,19; 2; --> Euro erfinden...

[Mr. Magpie]

Markus, du solltest deinen Titel ändern: von "Handbuchversteher" zu "Benutzerversteher"

[MarkusR]

Ich bin doch selbst Benutzer... (aber manchmal verstehe ich mich selbst nicht )

Jetzt wäre natürlich ein Feedback der Tester ganz hilfreich...

[Eichi]

Wenn ich mal entsprechendes umsetzen möchte/muss, werd ich es probieren.

Leider fällt mir vor Schreck garnichts ein, wo ich den Parser einsetzen könnte.

[Mr. Magpie]

sobald ich heute Abend wieder etwas Luft habe, werd' ich's selbstverständlich austesten - mindestens eine Anwendung habe ich ja schon, wie du weißt

[Mr. Magpie]

... von wegen heut' Abend ... hat jetzt doch 'ne geschlagene Woche gedauert bis ich's austesten durfte - erst war der Cheffe dagegen (keine Zeit! wie immer) dann der Kunde ("too risky und überhaupt, geht doch auch so..."), aber bei einer ersten Testinstallation konnte ich keine Probleme feststellen, im Gegenteil: funzt 1A!!

[baluo]

Markus, du solltest deinen Titel ändern: von "Handbuchversteher" zu "Benutzerversteher"

oups,
dieses "danke" von mir war ein beitrag zu tief ... galt markus --- aber auch danke an magpie fuer's testen
gruss, gerhard

[Marky]

ich hätte da aber noch was ...

Also erst ist mir diese Meldung aufgefallen

Code: Alles auswählen

Warning: max() [function.max]: Array must contain atleast one element in /www/htdocs/0815/conpresso/_include/parser.php on line 241

und dann meine alten Sünden, mit den zuviel geschachtelten if-set's, damit das überhaupt früher funktioniert hat

[MarkusR]

Kann es sein, daß eines der Templates gar kein if-Tag hat???

Und daß ein altes Template durchaus alte Probleme hat ist ja nun nicht das Problem des Parsers, oder? (ich habe auch ein paar Webseiten, wo ich brutalst verschachtelt habe, um das gewünschte Ergebnis zu bekommen... da kommt der Parser erst mal nicht zum Einsatz...)

Für das erste Problem habe ich mal den Teil

Code: Alles auswählen

	$maxlevel = max($iftaglevel);
	for ($if=$maxlevel;$if>0;$if--)  {
		foreach ($iftags AS $key=>$value)  {
			if ($iftaglevel[$key]==$if)  $thisiftags[$key] = $value;
		}
		if (count($thisiftags)>0)
			$template_datei = preg_replace(array_keys($thisiftags), array_values($thisiftags), $template_datei);
	}

durch

Code: Alles auswählen

if (count($iftaglevel)>0)  {
	$maxlevel = max($iftaglevel);
	for ($if=$maxlevel;$if>0;$if--)  {
		foreach ($iftags AS $key=>$value)  {
			if ($iftaglevel[$key]==$if)  $thisiftags[$key] = $value;
		}
		if (count($thisiftags)>0)
			$template_datei = preg_replace(array_keys($thisiftags), array_values($thisiftags), $template_datei);
	}
}

ergänzt.

[Marky]

Hi,

Und daß ein altes Template durchaus alte Probleme hat ist ja nun nicht das Problem des Parsers, oder?

nein, natürlich nicht. War ja auch nicht als Kritik gemeint. Da muss ich schon selbst ran.
Das Korrektur werde ich ausprobieren ... und "Bescheid" sagen ...

[Marky]

Hi,

Bescheid!
Hattest Du es direkt schon geändert ?! Musste ja gar nix mehr machen
Besten Dank!

ja ... bei den Templates muss ich mal auf "IFSET-Validität" prüfen