Possible solution?

• Q_ASSERT(newDir.exists());
• if (!newDir.exists())
• {
• newDir.mkdir(newDirectory);
• }

Yes, I think that's a better solution.

I'm reassigning this to Julien.

Julien - have a look at the logic here - probably this should prompt the user rather than just creating the old directory. Or it should switch the default directory.

Concerning the MRB scene case, I think it's a mistake to keep around an obsolete scene path. I believe when the temporary path is deleted, the scene URL and nodes scene root dirs should be reset. If they are reset (empty), I believe (not 100% sure) the save data dialog chooses a default directory.

Nonetheless, the user can also delete the scene directory after loading it. If that happens (newDir.exists() == false), then I believe we can create the directory but the user should be warned (yes/no message box).

If the scene root directory does not exist set it to current dir.

http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21288

Alex, Nicole, and I worked on this - Alex is committing a fix.

I agree that the user should be warned of invalid paths, but right now
the operation of saving will create the directories and not crash.
This may not be ideal behavior, but the user will have every
opportunity to change the paths inside the dialog if they want to.

Closing resolved issues that have not been updated in more than 3 months