search

yakamara / yform

YForm für REDAXO 5 – Formulare im Frontend und Backend mit Verwaltung von Datenbank-Tabellen.
MIT License
77 stars 55 forks source link

customLink `ylink` funktioniert nicht mehr mit yform 4.2.0 #1483

Closed ynamite closed 7 months ago

ynamite commented 7 months ago

Description / Beschreibung

Das Popup öffnet sich zwar, aber beim Versuch einen Datensatz auszuwählen, erscheint in der Konsole

Uncaught TypeError: Cannot set properties of null (setting 'value')

yform manager triggered den Event rex:YForm_selectData nicht mehr.

Affected versions / Verwendete Versionen yform 4.2.0 mform 7.4.0

ynamite commented 7 months ago

https://github.com/FriendsOfREDAXO/mform/issues/306

skerbis commented 7 months ago

Fehlermeldung in der Konsole bei mir: TypeError: null is not an object (evaluating 'opener.document.getElementById('yform-dataset-view-'+opener_id).value = value')

Auszug des Übernehmen-Button-Code:

<span class="yform-dataset-widget"><a class="btn btn-popup yform-dataset-widget-set" data-id="1254" data-opener_id="1" data-opener_field="title" data-value="Kinder- und Jugendgesundheit … [id=1254]" data-multiple="0">übernehmen</a></span>

skerbis commented 7 months ago

Achtung es hat auch Auswirkung auf CKE5, mform, mblock, redactor, TinyMCE und Co. ... ich finde es nicht gut, dass wir jetzt sämtliche AddOns anpassen müssen. Bitte lasst das Jquery erstmal drin. Das bekommen wir so schnell nicht gefixt. @dergel Das muss vorbereitet sein. - Ich habe dafür zu wenig JS-Fähigkeiten und die Devs der betreffenden AddOns können das auch nicht adhoc anpassen. Gerade bei CKE5 müsste ein komplettes neues Build erzeugt werden.

ynamite commented 7 months ago

@skerbis @dergel man könnte auch mal beide Variante drin lassen. Sollte keine Konflikte geben, wenn beide laufen. jQuery legt den Listener auf das neu erstellte Popup, mit Native JS muss der Listener auf window, kann man also unterschiedlich behandeln.

dergel commented 7 months ago

Für mich ok. Dann halt nur deprecated dran, damit uns das noch bewusst ist. Danke für die Hilfe