Workaround is to delete the entry itself and add it again without the file.

Hi Petr

Thanks for the reply. Came to the same conclusion

In the majority of cases this may not be such a problem, and may teach people to get it right in the first place. That said, when you have multiple file uploads in one database entry and the files concerned are large this workaround can cost hours not minutes.

Perhaps this is a feature request rather than a bug? I.e. add a way to delete an uploaded file in the database activity UI.

I think if the GUI allows to upload a file, it should also allow to remove it, otherwise this feature seems incomplete to me.

Is this issue not going to be addressed (Fix Version/s)?

Just updating this issue as it seems to still be a problem in Moodle 2.0.

Reassigning to moodle.com for consideration.

Sorry this issue is still not fixed. It remains a problem in 2.0.3.

We are still having this issue as well and cant find a way around it other than replacing the file

I definitely need this. If I delete the whole entry, I will need to re-enter comments (from other people as well as my own).

This seems to still be an issue. I created a database containing a file, a picture and a text field (which includes an uploaded video). I can't see any way to delete the files.

We are working with 2.3 and it still remains unresolved. Please make an effort on this.

Adding 2.4 as affects version and assigning to Adrian as database activity module component lead.

While working on this issue I noticed that I was removing the only calls to some functions in mod/data/data.js. I've added a deprecated tag to these functions and if the integrators agree I'll create an issue in MDL-35024 to remove them.

Hi Adrian,

I think this is a nice improvement. I also think deprecating those functions is correct - only because it is possible that someone has written some custom database field types that use them.

There were a couple of minor coding style issues:

mod/data/field/file/field.class.php line 83 - missing capitalisation in comment
mod/data/field/file/field.class.php line 94 - very long line
mod/data/field/picture/field.class.php line 91 - missing capitalisation in comment
mod/data/field/picture/field.class.php line 108 - very long line
mod/data/field/picture/field.class.php line 119 - multiple empty lines

[N] Syntax
[Y] Output
[N] Whitespace
[-] Language
[-] Databases
[Y] Testing
[-] Security
[-] Documentation
[Y] Git
[Y] Sanity check

Thanks, Damyon

Thanks Daymon,

I fixed up the coding style issues.

I'm not sure whether this should be back ported. And if it isn't, whether I should come up with a different solution for the earlier branches.

Thanks Adrian, your fixes are all good.

My vote is that this is an improvement and shouldn't be backported - lets see what the integrators think.

Cheers, Damyon

Thanks Damyon,

I just realised that I've been spelling your name wrong (super sorry!).

Submitting to integration.

Hi guys,

I've looked over this now.
Code looks fine, I was a little thrown off by the use of form_filemanager outside of a form but having looked at that component despite its location and name it holds no dependency to formslib.

In regards to backporting after really considering it I agree that this is an improvement, and although it solves the issue it also changes UI. Really this is a master only change.
Should anyone feel very strongly about this feel free to create a tracker issue to get this backported. We can discuss it more there.
For the time being this is landing master only.

Many thanks
Sam

Test result: Success!

Tested in Master only. Works nicely.

And your fantastic code has met core, hope they become good friends for a long period.

Closing, thanks!

Removing docs_required label as FAQ 'How can I delete a file attached to a database entry?' removed from http://docs.moodle.org/25/en/Database_activity_FAQ

As this issue fix is a usability improvement, I don't think it needs documenting specially.