Moving a subfolder doesn't delete *.sbd directory. Steps to reproduce: (1) Create a subfolder. (2) Move it to another place by drag and drop. The arrow which indicates subfolder remains. *.sbd direcotry isn't deleted.
over to naving.
Assignee: sspitzer → naving
Yes the .sbd directory will be there, but it will be empty.
.sbd is deleted with this patch. But the icon for the subfolder remains.
The previous patch doesn't check whether there are other subfolders or not.
Attachment #75382 - Attachment is obsolete: true
Does this patch work yet? Can it be checked in?
Comment on attachment 75385 [details] [diff] [review] patch, v2 This patch still works.
Navin, how can we request a review to possibly get this patch checked in?
patch v2 deletes *.sbd even if it contains subfolders.
Attachment #75385 - Attachment is obsolete: true
Is it possible to get this in for 1.4 beta?
CC-ing bienvenu, he might be interested.
Comment on attachment 119790 [details] [diff] [review] patch v3 you should add a comment that i.Exists() checks to see if the .sbd directory is empty or not (that's what it does, right?) Also, you don't need the extra braces around the call to parentPath.Delete. You've made sure that this works in the case of a non-empty parent directory, right? It looks fine, otherwise, thx for working on this.
Assignee: naving → sspitzer
Added a comment and deleted the braces.
Attachment #119790 - Attachment is obsolete: true
Comment on attachment 75385 [details] [diff] [review] patch, v2 clearing obsolete review request
There is a patch, why has this bug stalled?
taking - I'll try this out.
Assignee: sspitzer → bienvenu
fix checked in, r/sr=me, thanks Kazuhiko
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.