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.

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

File attribute - HTML input mode swipes the current value when updating a record #5599

Closed
ahukkanen opened this issue Jun 22, 2017 · 2 comments · Fixed by #7843
Closed

File attribute - HTML input mode swipes the current value when updating a record #5599

ahukkanen opened this issue Jun 22, 2017 · 2 comments · Fixed by #7843
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Bug Existing functionality not performing as expected.

Comments

@ahukkanen
Copy link
Contributor

I find this issue with Express (not sure if it applies everywhere for file attributes):

I have an existing record with a file assigned to it through the file attribute (using HTML input method). When I update one of these records, the updating process swipes out the current value if the field was left empty.

This makes it very hard to keep these records up to date as you would have to upload the files again every time you do an update.

@aembler aembler added Status:Available Reviewed issue, it’s real, we’d review a pull request. priority:like to have Type:Bug Existing functionality not performing as expected. labels Jun 23, 2017
@aembler aembler added this to the 8.2.1 milestone Jun 23, 2017
@aembler aembler modified the milestones: 8.2.1, Future Aug 3, 2017
@ounziw
Copy link
Contributor

ounziw commented Nov 21, 2017

I'm not tested 8.2.1 or 8.3rc.

I published a composer image upload function skipping file manger for 5.7.5.x
https://gist.github.com/ounziw/56e2b4f2602cafa29f84020768e44b37
Here, I define a hidden form for "value", so that it stores an ID for existing file.

I hope this code will help fixing the code for 8.2.1

@mlocati
Copy link
Contributor

mlocati commented May 26, 2019

See #7843

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Bug Existing functionality not performing as expected.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants