Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
Krusader
Krusader
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 6
    • Merge Requests 6
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Utilities
  • KrusaderKrusader
  • Issues
  • #5

Closed
Open
Opened Jun 19, 2020 by Timur Guzairov@tguzairov

Krusader doesn't delete all selected folders if some of them aren't empty

  1. Request user confirmation for deleting non-empty folder is set to on.
  2. User selects several folders and some of those folders aren't empty
  3. User chooses 'Delete All' at the 'Folder is not empty' warning dialog.
  4. Krusader deletes only folders which were before the first non-empty one. It happens in ListPanelFunc::confirmDeletion We add urls to confirmedFiles set in the loop, then we delete an url from it if user decides not to delete the file, but if they chooses 'Delete All' we just break from the loop without adding the rest of urls to the set.

File: krusader/Panel/panelfunc.cpp Inside if (emptyDirVerify) {

else if (result == KMessageBox::No) {
  break; // accept all remaining

But after the loop we just assign toDelete = confirmedFiles.values();

Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: utilities/krusader#5