phpac - glist.php

Anando Eger a.eger at aneg-dv.de
Mo Mär 1 13:06:05 CET 2004


Liebe Listenleserinnen und -leser, lieber Herr Eversberg,

für glist.php hätte ich noch zwei Verbesserungsvorschläge:

Zeile 28:  for($i=1;($m = getval($i))==0;$i++) ;

ändern in 

for($i=1;($m = getval($i))==0;$i++) if ($i>999) break;

das verhindert eine Endlosschleife, wenn im Formular vorher
kein Eintrag markiert wurde und dann "Liste holen" betätigt
wurde.

Ab Zeile 99:
----------------------------------------------------------
else
$job = array(
"var '#$op1'",
"find",
"xport p $Dispar",
"wri '<table>'",
"dow",
"dow set",
"wri '</table>'",
"end"
);
----------------------------------------------------------
ändern in
----------------------------------------------------------
elseif ($op1)
$job = array(
"var '#$op1'",
"find",
"xport p $Dispar",
"wri '<table>'",
// "dow",   !!! entfernt, da sonst erster Satz doppelt angezeigt wird 
"dow set",
"wri '</table>'",
"wri '<br><center><a onclick=javascript:history.back() href=#>zurück</a></center>' n",
"end"
);
else 
$job = array(
"xport p $Dispar",
"wri '<center><p>Bitte wählen Sie mindestens einen Eintrag aus der Liste aus.</p><br>' n",
"wri '<br><a onclick=javascript:history.back() href=#>zurück</a></center>' n",
"end"
);
----------------------------------------------------------
Dann wird auch ein passender Text angezeigt. Das "href=#" kann man auch weglassen,
dann ist der Link als solcher jedoch nicht gleich erkennbar (vielleicht fällt jemandem
eine bessere Lösung ein? ).

Das oben deaktivierte "dow" müsste ja eigentlich schon lange aufgefallen sein, oder
kan das Verhalten der Befehlsfolge "dow\dow set" mit der Art der Installation
variieren?

Mit freundlichen Grüßen
Anando Eger

-----------------------------------------------------------------------------
Anando Eger Datenverarbeitung
Herr Dipl.-Ing. Anando Eger
Oppelhain, Siedlungsweg 1
03238 Rückersdorf

Tel: +49 (0)35325 16000    http://www.aneg-dv.de
Fax: +49 (0)35325 16002   mailto:a.eger at aneg-dv.de
-----------------------------------------------------------------------------





Mehr Informationen über die Mailingliste Allegro