Product downloads controller reviewed

Over the last couple of days I reviewed the product downloads controller in order to give it a more reliable shape. This includes a number of changes to the ProductDownload class, none of which should affect behavior for other uses.

It’s this thing, part of the product editor:

Changed and new features:

Deleting the file deletes the file only, not the entire product download record. As one would expect when clicking ‘Delete this file’.

Removing the file associated to the product still removes both, file and record.

I’ll have a final look today and will let you know. Looking good so far.

What is missing for the second precision? Just the frontend widget? I believe tb needs a datetime option instead of just date. This time I actually wouldn’t mind if we added a new feature. I needed to use the datetime option several times in my modules, so instead ended up adding it to the module via a template override.