Skip to content

Prioritized searching for files in the new folder location during project loading.#578

Merged
Bujee415 merged 1 commit intomasterfrom
improve/loading-moved-project
Jun 11, 2025
Merged

Prioritized searching for files in the new folder location during project loading.#578
Bujee415 merged 1 commit intomasterfrom
improve/loading-moved-project

Conversation

@YukiMatsuzawa
Copy link
Copy Markdown
Contributor

Prioritized searching for files in the new folder location during project loading.

When a project is moved to a new folder, the loading logic now prioritizes searching for files in the new folder location over the original folder. This ensures better handling of relocated projects and avoids unnecessary fallback to the original folder.

…ject loading.

When a project is moved to a new folder, the loading logic now prioritizes searching for files in the new folder location over the original folder. This ensures better handling of relocated projects and avoids unnecessary fallback to the original folder.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves project loading by prioritizing file searches in the new folder location when a project is moved.

  • The first try block now attempts to load using newProjectDir with additional folder fixes and retention of previous RT correction results.
  • The fallback logic has been updated to load from projectDir if needed.
Comments suppressed due to low confidence (1)

src/MSDIAL5/MsdialCore/DataObj/ProjectDataStorage.cs:143

  • Consider adding logging or more specific exception handling in this catch block to aid debugging while still allowing the fallback logic to proceed.
catch {

@Bujee415 Bujee415 merged commit 242afd5 into master Jun 11, 2025
9 checks passed
@Bujee415 Bujee415 deleted the improve/loading-moved-project branch June 11, 2025 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants