jForms: empty upload controls linked to a dao property should not erase the property's value

[Julien]

...either you would have to upload the file every time you submit the form in update context...

patch: ​https://bitbucket.org/julieni/jelix-trunk-patches/src/128b1c3f30f5/jForms-empty-uploads-should-not-replace-existing-values-in-dao.patch

[laurentj]

Because of the specific behavior of input type="file" in browser, it's obvious for me that an upload control should not be mapped on a dao file.

And if this is the case, I think your patch is not the right way. How can we "erase" the field ? (with a null value or "")

[Julien]

this patch is a first part from a work in progress on my side, which will allow quite good management of files in jForms.

• good mapping to dao properties (good in case you need to overload forms/dao for a specific client who wants 13 upload fields where you only put 12... ;) and you don't want to modify the controller's/classes' code )

• in case of editing a form, auto-fill of upload controls (not the input type="file" itself of course), with preview for browser compatible images and link for others file types.

• of course, empty uploads should not erase existing ones, I don't want to re-upload of my files when just changing the title field (that's what this patch does)

• if the field is not required, add an extra "delete this file" aside the control

why I already posted this part of the patch : because I made an app and when editing forms all file associations were erased ; so it's a quick fix for my first problem. Now you just re-upload when you want to change a file.

I really think that an upload control should be mappable to a dao, as we do for every other control in jforms ; and I'm working on it to have that management easy way.