search

andydandy74 / DynamoAutomation

Batch-processing of Revit models using the Dynamo visual programming environment
MIT License
62 stars 23 forks source link

Journal.ByWorkspacePath can't handle whitespace in file paths #7

Open andydandy74 opened 9 years ago

andydandy74 commented 9 years ago

@ikeough - found the first bug by pure luck. I rearranged the folder structure of the test files and renamed some of the subfolders. Some of the folder names now contained whitespaces. The path in the resulting journal file will look like this:

C:\temp\DynamoAutomation\slave/graphs\Revit2016\CountWalls.dyn

So it turned slave graphs into slave/graphs. Revit will not be able to find a Revit model with such a path nor will the Dynamo addin find a graph with such a path.

andydandy74 commented 9 years ago

Also - I wonder if this may happen with other special characters as well?

ikeough commented 9 years ago

@andydandy74 We should re-open this. It's replacing spaces with / because I made it do that when debugging why it didn't work for some paths. We should set it back, I just haven't had time.

andydandy74 commented 9 years ago

Okay @ikeough - I reverted it. Can you remember a specific case where leaving the original whitespace didn't work?

andydandy74 commented 7 years ago

Seems that this is just a standard limitation of Revit journal playback.

andydandy74 commented 6 years ago

Let's have a look at this again... https://twitter.com/PragmaticPrxs/status/963420422741069824

andydandy74 commented 6 years ago

https://social.msdn.microsoft.com/Forums/vstudio/en-US/d345fb15-7676-438d-9530-bfc9de640f59/how-do-i-pass-whitespace-to-a-process-command-line?forum=csharpgeneral

steinah6 commented 4 years ago

What's the current status of this? I'd love to use your tools but renaming all of our office directories is not an option right now.