Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

customLink ylink funktioniert nicht mehr mit yform 4.2.0 #1483

Closed
ynamite opened this issue Mar 5, 2024 · 5 comments · Fixed by #1484
Closed

customLink ylink funktioniert nicht mehr mit yform 4.2.0 #1483

ynamite opened this issue Mar 5, 2024 · 5 comments · Fixed by #1484
Milestone
4.2

Comments

@ynamite
Copy link
Contributor

ynamite commented Mar 5, 2024

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
Copy link
Contributor Author

ynamite commented Mar 5, 2024

FriendsOfREDAXO/mform#306

@skerbis
Copy link
Contributor

skerbis commented Mar 5, 2024 •
edited
Loading

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>

ynamite added a commit to ynamite/redaxo_yform that referenced this issue Mar 5, 2024
@ynamite
Update widget.js with rex:YForm_selectData
b14a066 
Fixes yakamara#1483
@ynamite ynamite mentioned this issue Mar 5, 2024
Merged
@dergel dergel added this to the 4.2 milestone Mar 5, 2024
@skerbis
Copy link
Contributor

skerbis commented Mar 5, 2024 •
edited
Loading

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
Copy link
Contributor Author

ynamite commented Mar 5, 2024 •
edited
Loading

@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
Copy link
Member

dergel commented Mar 5, 2024

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

dergel pushed a commit that referenced this issue Mar 7, 2024
@ynamite
Update widget.js with rex:YForm_selectData (#1484)
335dc5c 
* Update widget.js with rex:YForm_selectData
Fixes #1483
* Update widget.js with native rex:YForm_selectData event
* Update widget.js bc added deprecated jQuery implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.2
Development

Successfully merging a pull request may close this issue.

Update widget.js with rex:YForm_selectData ynamite/redaxo_yform
3 participants
@dergel @skerbis @ynamite