ModuleCatalogEdit.php datepicker Bug

[Andreas]

Ich weiß jetzt nicht, ob das ein Bug ist, weil ich den Catalog in 2.10.2 benutze. In der ModuleCatalogEdit.php Zeile 776

$datepicker = '<img src="plugins/datepicker/icon.gif" width="20" height="20" id="toggle_' . $objWidget->id . '" style="vertical-align:-6px;">';
$GLOBALS['TL_HEAD'][]=
'window.addEvent(\'domready\', function() { 

Fehlt das Element SCRIPT

$datepicker = '<img src="plugins/datepicker/icon.gif" width="20" height="20" id="toggle_' . $objWidget->id . '" style="vertical-align:-6px;">';
$GLOBALS['TL_HEAD'][]=
'<script type="text/javascript">
/* <![CDATA[ */
window.addEvent(\'domready\', function() { 

Dementsprechen auch das Closing-Tag.

ps Das IMG ist nicht XHTML-valide.

Gruß Andreas

[MacKP]

Hallo Andreas,
könntest du bitte ein Ticket dazu machen wenn es noch keins gibt?

http://www.contao-forge.org/projects/catalog/issues

Viele Grüße

[Andreas]

Jetzt bin ich aber durcheinander, ich habe das letzte Catalog-Ticket hier erstellt http://code.google.com/p/typolight-catalog/issues/list

Was ist denn jetzt richtig?

Zum Ticket: Da stimmt sonst noch einiges nicht mit dem Catalog-Edit-Modul was den Datepicker betrifft. Das müsste wohl mal komplett überarbeitet werden. Z.B. werden, wenn man mehrere Datepicker hat für jeden eine Instanz erzeugt, es benötigt aber nur einer Instaziierung. Dazu müssten dann allerdings die entspr. Inputfelder leichter ansprechbar sein, z.B. mit einer eigenen CSS-Klasse. Dann scheint der FE-Output des Moduls nicht konfigurierbar zu sein, es werden Tabellen-Zeilen ausgegeben. Dadurch kann man die Toggler nicht richtig positionieren, weil die außerhalb der Tabelle liegen. Dann sind die Einstellungen der Datepicker hardcodiert, bei mir funktionierte es deshalb nicht mit dem Format "d.m.Y".

Das Ticket wird also nicht viel bringen. Wenn man das Modul überarbeitet, wird man den oben beschriebenen Fehler auch sofort sehen, da der JS-Code im FE sichtbar ausgegeben wird.

[MacKP]

Ja, sind irgendwann umgezogen.. aber die Entwickler schauen auch noch bei Google. Ich geb aber immer den aktuellen raus, damit das bald nur noch darüber läuft.

Hmm... dann mach doch ein Ticket für das Ding allgemein. Damit das nicht untergeht ;-)

Viele Grüße