terminal42 / contao-mp_forms

Real step separation in the form generator of the Contao Open Source CMS
25 stars 13 forks source link

Dateiupload und Notification Center #42

Closed FrischBlau closed 2 years ago

FrischBlau commented 4 years ago

Erst einmal danke für die großartige Erweiterung! Es scheint ein Problem in Verbindung mit dem Notificaton Center zu geben: In einem mehrseitigen Formular ist ein Dateiupload-Feld enthalten. Die hochgeladene Datei soll via NC als Anhang einer Benachrichtigung gesendet werden. Also in der Benachrichtigung über "Dateianhänge via Tokens". Der Token ##form_upload## (wobei "upload" der Name des Upload-Feldes ist) hat jedoch keine Auswirkung, die Datei wird nicht an die Benachrichtigung angehängt. Entfernt man den PageBreak, so dass das Formular nur noch einseitig ist, greift der Token.

modulists commented 3 years ago

Auch von mir herzlichen Dank ... für eines unserer Projekte ist die Mehrseitigkeit super!

Leider haben wir das gleiche Problem mit zu transportierenden Bildern ... vielen Dank an Frischblau für die Beschreibung! ;-)

Ist absehbar, ob das Problem leicht zu beheben ist ?

markus361 commented 3 years ago

@FrischBlau @modulists

Ich habe es eben auch getestet und bei mir werden Dateien versendet, auch wenn sie an Bedingungen geknüpft sind.

Ich nutze für den Test: Contao 4.9.9 contao-mp_forms 4.1.0 notification_center 1.5.18 contao-conditionalformfields 2.3.2

FrischBlau commented 3 years ago

@markus361 Welchen Token hast Du verwendet, um die Datei anzuhängen?

markus361 commented 3 years ago

@FrischBlau Bei mir heißt das Feld auch upload, also so:

Unbenannt

FrischBlau commented 3 years ago

@markus361 Danke für das EDIT, es scheint genau daran zu liegen, dass die Dateien auch auf dem Server gespeichert werden müssen. Test mit mehreren Uploads - nur die Dateiuploads mit aktivierter Option "Hochgeladene Dateien speichern" werden übertragen.

modulists commented 3 years ago

Allerherzlichsten Dank !!! Unser Kunde ist glücklich!

Toflar commented 2 years ago

Die Meinung war schon, dass es auch mit temporären Uploads geht. Das Problem war eine Kombination aus dem NC und mp_forms. Der Teil beim NC ist in https://github.com/terminal42/contao-notification_center/releases/tag/1.6.4 gefixt.

mp_forms dann in der 4.3 mit https://github.com/terminal42/contao-mp_forms/commit/df6fcf40c228e83310c7b7abd507159a70682314.