PowerToys
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
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] JSON for signing for new binaries
- [n/a] WXS for installer for new binaries and localization folder
- [n/a] YML for CI pipeline for new test projects
- [n/a] YML for signed pipeline
- [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