mform icon indicating copy to clipboard operation
mform copied to clipboard

Javascript für yform Link-Widget prüfen

Open joachimdoerr opened this issue 1 year ago • 6 comments

Nach einem fetten merge von develop nach master gab es hunderte changes und nochmal so viele mergekonflikte, ich hab getan was ich konnte um das javascript so plausibel wie möglich zu mergen. es wäre aber sicher sehr gut wenn @ynamite wenn du das nochmal prüfen würdest.

joachimdoerr avatar Apr 17 '24 20:04 joachimdoerr

Das sieht auf den ersten Blick und kurzem Vergleich mit der vorherigen Version korrekt aus. Ich werde es heute oder morgen noch testen und melde mich nochmal.

ynamite avatar Apr 25 '24 06:04 ynamite

@ynamite passt das wie es ist? Konntest Du es auch testen?

joachimdoerr avatar Apr 29 '24 09:04 joachimdoerr

@joachimdoerr yep, habe es eben mit der Version 7.4.4 getestet. Die Auswahl eines yform Datensatzes klappt aus mform und yform aus. Allerdings werden vor dem custom_link widget in yform folgende Warnungen ausgegeben:

Warning: Undefined array key 1 in src/addons/mform/lib/Widget/var_custom_link.php on line 112
Warning: Undefined array key 2 in src/addons/mform/lib/Widget/var_custom_link.php on line 113

Die Definition für das yform link widget ist wie folgt test::rex_yf_portfolio_tag::name,Name

ynamite avatar Apr 30 '24 12:04 ynamite

@ynamite bitte teste doch mit der aktuellen MForm 8.0.0-Beta6.

joachimdoerr avatar May 01 '24 11:05 joachimdoerr

Mit der 8.0.0-Beta6 habe ich folgende Warnungen.

Modul-Input custom_link: Warning: Array to string conversion in src/addons/mform/lib/MForm/Parser/MFormParser.php on line 921

Yform custom_link:

Warning: Undefined array key 1 in src/addons/mform/lib/Widget/var_custom_link.php on line 108
Warning: Undefined array key 2 in src/addons/mform/lib/Widget/var_custom_link.php on line 109

Aber auch hiermit klappt bei mir alles

ynamite avatar May 01 '24 11:05 ynamite

ok perfekt danke @ynamite die warnings werd ich mir nochmal anschauen.

joachimdoerr avatar May 02 '24 08:05 joachimdoerr

@ynamite ich hab mir das mal angeschaut und das yform widget wird ganz sicher falsch angesprochen siehe screenshot.

/index.php?page=yform/manager/data_edit&table_name=rex_ycountries&rex_yform_manager_opener[id]=1&rex_yform_manager_opener[field]=de_de&rex_yform_manager_opener[multiple]=0

Bildschirmfoto 2024-05-13 um 23 26 35

joachimdoerr avatar May 13 '24 21:05 joachimdoerr

@joachimdoerr Hm, bei mir klappt es nach wie vor. Bei mir sieht die URL aber so aus: index.php?page=yform/manager/data_edit&table_name=rex_yf_portfolio_tag&rex_yform_manager_opener[id]=cl96b86ce4f54d&rex_yform_manager_opener[field]=name&rex_yform_manager_opener[multiple]=0

ynamite avatar May 14 '24 06:05 ynamite

@ynamite dank dir, bei mir passt jetzt auch alles es gab noch eine Kleinigkeit die jetzt erledigt ist: 6ffc9b084213d9c5cc5ee817a375f2d06f0b9f2a

joachimdoerr avatar May 14 '24 12:05 joachimdoerr