Fix file name of exported parts#33351
Conversation
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThis PR refactors export path storage across the project. The 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Resolves: #21161
When exporting score parts, the resulting filename was expected to have appended some description of the file's contents (e.g. the part name for individual part files, or
-Parts/-Score_and_partsfor multipart files).However, this was only the case the first time an export was made, and later exports contained only the filename.
This was due to a default path being saved in the
ExportDialogModelin order to "remember" the last export directory. This default path contained the full path, including the filename, of the first export made since opening the MSS program, and it caused subsequent exports to use the exact same default without appending the extra filename parts.The fix is to store the path to the default save directory, instead of the full path to the last saved file. This way, only the directory is reused, while the filename is re-calculated each time an export is made.