consistent wording: dt_lua_image_t.delete() implements function otherwise referred to as "remove"
It would be easier for beginners (like me) to see consistent wording for functionality.
I suggest adding dt_lua_image_t.remove() or renaming dt_lua_image_t.delete() to dt_lua_image_t.remove() before more lua code is written.
#2 Updated by Richard Wonka almost 3 years ago
"Impossible" seems a big word. Would "unpleasantly tedious" describe the situation? :-)
I think this issue will create a lot of work in FAQs, forums, README's and unhappy users.
The effort/overhead of changing it now might well be less than the sum of future unpleasantness.
Also: how about adding a "remove()" that does the same thing? Simpler to implement and won't break the current API...
#3 Updated by Tobias Ellinghaus almost 3 years ago
No, "impossible" is the right word. If we changed "delete" to actually delete the files from disk now then all the scripts out there that use that function to remove images from the db (which is what it currently does) would suddenly start deleting files. Which is completely out of question. Like, not even worth discussing.