UI_Table_Form, SubmitJS bei onClick??

Gesperrt
Oldperl
Beiträge: 4316
Registriert: Do 30. Jun 2005, 22:56
Wohnort: Eltmann, Unterfranken, Bayern
Hat sich bedankt: 6 Mal
Danksagung erhalten: 4 Mal
Kontaktdaten:

UI_Table_Form, SubmitJS bei onClick??

Beitrag von Oldperl »

Bei der Programmierung eines Plugins in der 4.5.3er Version fiel mir folgendes, auch in der 4.6.2er noch vorhandene, Problemchen auf.

Hmm, ich seh das mal als BUG an.

In der Klasse UI_Table_Form in der Datei class.ui.php besteht die Möglichkeit ein eigenes Javascript zu Übergeben (function setSubmitJS), welches beim Abschicken verarbeitet werden soll.

In der Funktion render steht nun aber ab Zeile 427 folgendes

Code: Alles auswählen

if ($this->submitjs != "")
		{
			$fextra = 'onclick="'.$this->submitjs.'"';
		} else {
			$fextra = "";
		}
Dabei wird nun aber auch bei jeglichem Klick auf ein Formelement das verbundene Script aufgerufen und nicht nur beim Abschicken des Formulars. Ist mir eigentlich nur dadurch aufgefallen, weil ich Werte per Script erzeuge und diese dann per Submit übergeben wollte, nur sprang mein Script bei jedem Klick in die Form an und schrieb munter drauf los. :roll:

Eine Änderung der render-Funktion dort in

Code: Alles auswählen

if ($this->submitjs != "")
		{
			$fextra = 'onsubmit="'.$this->submitjs.'"';
		} else {
			$fextra = "";
		}
würde dies beheben.

Eine andere Alternative wäre dort garnichts zu hinterlegen und dem Entwickler selbst entscheiden zu lassen welches Event er benutzen will.

Mit Gruß aus dem verregneten Franken

Ortwin
ConLite 3.0.0-dev, alternatives und stabiles Update von Contenido 4.8.x unter PHP 8.x - Download und Repo auf Gitport.de
phpBO Search Advanced - das Suchwort-Plugin für CONTENIDO 4.9
Mein Entwickler-Blog
emergence
Beiträge: 10653
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence »

ich hab das mal verschoben...
*** make your own tools (wishlist :: thx)
HerrB
Beiträge: 6935
Registriert: Do 22. Mai 2003, 12:44
Wohnort: Berlin
Kontaktdaten:

Beitrag von HerrB »

In bestehenden C-Versionen schon gefixt, closed.

Gruß
HerrB
Bitte keine unaufgeforderten PMs oder E-Mails -> use da Forum!

Newsletter: V4.4.x | V4.6.0-15 (Module, Backend) | V4.6.22+
Standardartikelliste: V4.4.x | V4.6.x
http://www.contenido.org/forum/search.php | http://faq.contenido.org | http://www.communido.net
Gesperrt