PowerToys icon indicating copy to clipboard operation
PowerToys copied to clipboard

New+ bug fix for 35179 where if target folder with same name already exists New+ creates folder instance within the existing one

Open cgaarden opened this issue 1 year ago • 0 comments

Summary of the Pull Request

Fix for if target folder with same name already exists New+ creates folder instance within the existing one

PR Checklist

  • [x] Closes: #35179
  • [x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
  • [not included] Tests: Added/updated and all pass
  • [n/a] Localization: All end user facing strings can be localized
  • [n/a] Dev docs: Added/updated
  • [n/a] New binaries: Added on the required places
  • [n/a] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

This PR only fixes the case where if a folder with the name already exist then a new folder would be created within the existing one Code added a check to see if target is a directory don't append source filename

Validation Steps Performed

Manually tested locally created 5 folders Manually tested locally created 5 files

cgaarden avatar Oct 17 '24 01:10 cgaarden