Deleting sketch caused Processing to delete my entire Downloads folder

[kaylinli]

Description

I downloaded a pde file from a friend and opened the pde file from the Downloads folder. Later, I tried to delete the sketch through the Processing app, and it ended up deleting everything in my Downloads folder, which was a lot of files. These files couldn't be found in the recycling bin either, so they were directly deleted by Processing.

Expected Behavior

I was expecting Processing to only delete my pde files.

Steps to Reproduce

1. Move a pde file to the Downloads folder
2. Open pde file using Processing
3. Delete sketch using drop down menu in Processing

Your Environment

• Processing version:
• Operating System and OS version: Windows 10

Possible Causes / Solutions

Not sure, it seems like every time Processing deletes a sketch it deletes all the files in the working directory, but since I opened my pde file in the Downloads folder it made the Downloads folder the working directory and deleted everything in there.

[benfry]

Yikes, thanks for letting me know. I'll get a fix in ASAP.

The problem is that the file wasn't in a sketch folder, so it seems to have assumed that the “Downloads” folder was the sketch folder. This is new as of beta 6, where we're trying to decouple the requirement for the sketch folder to be named the same thing as the .pde file.

That's tricky to sort out, but we'll think of something. And at a minimum we should only be moving files to the Trash/Recycle Bin so that if there's a glitch like this, it's recoverable.

[benfry]

Fixed for beta 7, including:

• Better detection of what's likely a sketch folder and what is not. For this scenario, the PDE would have seen lots of other files in Downloads, and prompted the user to find out whether they intended “Downloads” to be the sketch folder, of if the selected .pde file should be moved to its own folder.
• Clarified the delete message to include the name of the folder that's about to be removed (see below).
• Even if deleting, it will (attempt to) move to the trash instead of deleting the files in an unrecoverable way.
• If opening a bare .pde file like this, it will reset to the default Mode, because using any other Mode requires a sketch.properties file.

Thanks for the report, and sorry for the trouble.

[github-actions]

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.