Datensatz klonen

[dergel]

basierend auf https://github.com/FriendsOfREDAXO/tricks/issues/309 und https://github.com/FriendsOfREDAXO/tricks/pull/311

Danke @christophboecker

[dergel]

closes https://github.com/yakamara/redaxo_yform/issues/49

[christophboecker]

@dergel Super, Jan, das ging ja ratz fatz. Sieht so aus, als ob es funktioniert. Ein Kurztest sagt: dobsche.

Darf ich dennoch was anmerken:

• Ich hätte mich nicht getraut, einen PR zu bauen, der einen YForm-Hack zum Umbau von YForm-Code in YForm einbaut. :-)
• Hast Du das Kleingedruckte des Tricks gelesen? Die EPs laufen nicht unbedingt so ab, wie vieleicht erwartet. Beim Aufbau des Formulars nach Klick auf die Klonen-Aktion wird der EP YFORM_DATA_UPDATE aufgerufen und nicht YFORM_DATA_ADD, während beim Speichern nach Klick auf den Save-Button erst der EP YFORM_DATA_ADD und dann YFORM_DATA_ADDED angezogen werden. Da werden bestimmt noch Fragen nach YFORM_DATA_CLONE und YFORM_DATA_CLONED kommen, die man bei enem Trick nicht stellen würde.
• Die Lösung erwischt zwar auch Datentypen, die auf be_manager_relation aufsetzen, aber nicht “private” Datentypen, die Relationen aufbauen und verwalten. Ob das mit einer tiefer in YFORM verdrahteten Lösung andes wäre? Vermutlich nicht. Nur um es erwähnt zu haben.
• Als Icon würde ich beim Klonen-Icon (.fa-copy oder .rex-icon-duplicate) bleiben, grade um den Unterschied zum normalen Hinzufügen (+) herauszustellen.
• Perspektivisch könnte man irgendwie wohl doch mal rangehen und $afterFieldsExecuted in den eigentlichen Code aufröseln. Also z.B. nicht nachträglich HTML rauswerfen oder ändern, sondern sofort passend weglassen/erzeugen. Falls das überhaupt mit vertretbarem Aufwand in dem komplexen Gesamtzusammenhang machbar ist. Ich bin bisher davor zurückgeschreckt.

[dpf-dd]

Ah, das Problem ist bekannt. Habe auch gerade gemerkt, dass beim Duplizieren der EP YFORM_DATA_ADDED nicht gefeuert wird. Schade, finde das Duplizieren echt nützlich... Mein Skript, was ich über EP einklinke, soll aber erst NACH dem Speichern laufen - deswegen kann ich da jetzt nicht einfach den YFORM_DATA_UPDATE mit benutzen, weil dann greift mein Skript zu früh ein :(

[dergel]

Könnt ihr das noch mal als einzelne Issues aufschreiben. Damit ich die Punkte sinnvoll ansehen und bewerten und hoffentlich lösen kann?

[dpf-dd]

Mach ich gerne. Hättest du es lieber hier bei yForm oder beim Usabilities-AddOn?

[christophboecker]

Habe auch gerade gemerkt, dass beim Duplizieren der EP YFORM_DATA_ADDED nicht gefeuert wird.

Wenn das Formular aufgerufen wird, ist es ja ein vorhandener Datensatz. also YFORM_DATA_UPDATE statt YFORM_DATA_ADD. Beim Speichern, also Aufruf des Formulars nach Klick aus Save, ist es ein neuer Datensatz. Daher werden YFORM_DATA_ADD/ADDED aufgerufen.

Zumindest habe ich das mal so ausbaldovert.

[dpf-dd]

@christophboecker Nein, das kann ich negieren. Ich habe ein Skript in folgendem Aufruf am werkeln: rex_extension::register(['YFORM_DATA_ADDED', 'YFORM_DATA_UPDATED'], function (rex_extension_point $ep) { und wenn ich einen Datensatz dupliziere ´, dann wird mein Skript NICHT getriggert... Ich muss immer erst in irgendeinen Datensatz klicken und nochmal speichern, dann wird der duplizierte Datensatz auch von meinem Skript "erwischt"...

Da mein Skript immer nur NACH dem Speichern werkeln soll, kann ich das für die EPs YFORM_DATA_ADD', 'YFORM_DATA_UPDATE natürlich keine Aussage treffen...

[christophboecker]

Hättest du es lieber hier bei yForm oder beim Usabilities-AddOn?

Sicher, dass wir über dasselbe reden? Dieses Klonen hat null komma nix mit dem Usabilities-AddOn zu tun. Selbiges - und das war ja das Problem - kopiert stumpf den Datensensatz, dann muss man ihn aufrufen und bearbeiten. Was zu Deiner Beschreibung (Ich muss immer erst in irgendeinen Datensatz klicken und nochmal speichern) passt.

[dpf-dd]

Naja, als ich yForm Usabilities noch nicht installiert hatte da konnte ich die Datensätze nicht duplizieren - das war halt einfach nicht da. Hab dann Usabilities installiert und seitdem krieg ich das angezeigt... Hab mir den Zusammenhang jetzt nicht weiter angeguckt, aber jetzt wo du das sagst - klar, ist ein yForm Issue

[christophboecker]

Das worum es hier geht, ist zwar im Repo, aber noch nicht im Relase. Und funktioniert halt ganz anders!

[dpf-dd]

Ah, naja das kann ich ja nicht riechen. Ich nutze ausschließlich die Installer-Versionen... Dann sorry für die Verwirrung. Ich zieh mir heute Abend mal yForm aus GitHub und teste das Duplizieren mal ohne Usabilities... :)

[dpf-dd]

OK. GitHub-Stand soeben gezogen und getestet. Klonen funktioniert und auch mein Skript am EP YFORM_DATA_ADDED wird getriggert weil das Handling tatsächlich anders ist. Ich klicke auf KLONEN und dann öffnet sich die Datensatz-Maske in dem ich den Klon noch mit Speichern bestätigen muss. Mein Skript rödelt dann auch durch, sodass aus meiner Sicht alles läuft wie geplant und gewünscht... :)

Ich ziehe meinen Einwand also zurück, weil von dem Rest von @christophboecker Beitrag hab ich nicht wirklich einen Dunst ^^