jneilliii / OctoPrint-UltimakerFormatPackage

GNU Affero General Public License v3.0
30 stars 7 forks source link

Uploaded UFP file not deleted from Plugin-Folder #13

Closed OllisGit closed 4 years ago

OllisGit commented 4 years ago

Hi @jneilliii , I am starting analysing the "remove thumbnail-issue". So, I tested a "normal" case:

fine....but after deleting the gcode from the GUI, all files where delete, except the orig. ufp file.

Tested with: 1.4.0rc5, Python 2.7

Could you reproduce this behaviour?

BTW. Why do you store the gocde and ufp file in your plugin folder...it waste harddisc-space?

After upload: image

After delete: image

jneilliii commented 4 years ago

Just an oversight @OllisGit. It was my assumption that the DiskFileWrapper based on the docs moved the file instead of copy the file.

jneilliii commented 4 years ago

Should be fixed with commit https://github.com/jneilliii/OctoPrint-UltimakerFormatPackage/commit/a66166bd089fb8709df804d6c8a82d6dc8f77f3c and added to next release version #12

OllisGit commented 4 years ago

Morning @jneilliii ,

I tested the 0.1.4 branch.

I tested also the FileManager plugin for moving gcode to a sub-folder. Unfortunately only the remove-event is passing the UFP-Plugin, the "FileAdded" event doesn't occourse....I think to solve the move image-issue (a.k.a. additional metadata), we need to analyse/improve the Octoprint-Core code or patching all plugins which were using file move.

jneilliii commented 4 years ago

Again, I thought the removal of the source file was handled properly by the DiskWrapper here, but I guess not. I'll have to add the manual removal. I didn't even think about the FileManager doing this too since I don't use it.

jneilliii commented 4 years ago

Just pushed another commit that also removes the gcode file from the plugin's data folder.