search

tt-acm / EnergyAnalysisForDynamo

Dynamo <> Autodesk Green Building Studio interoperability
Other
29 stars 18 forks source link

Create Project node should return the project ID if the project already exists #37

Closed mostaphaRoudsari closed 10 years ago

mostaphaRoudsari commented 10 years ago

Currently it returns null.

mostaphaRoudsari commented 10 years ago

I left this for the next release. I'm not sure how we can throw an exception and still return a value. I assume there is a way to turn Dynamo node orange without throwing an exception-

bhowes-tt commented 10 years ago

@eertugrul as discussed, please let the Create Project node return the Id of a project if it already exists. If more than one project with the same name already exist, throw an exception telling the user that multiple projects with that name exist.

eertugrul commented 10 years ago

@bhowes-tt @mostaphaRoudsari I have revised CreateProject Node per our discussion, If the project is existing, return projectid of existing project in GBS. If multiple projects exist with same name throw and exception. I renamed the node as "CreateProject" instead "CreateNewProject"! Add meaningful tool tip and error message for the user.

mostaphaRoudsari commented 10 years ago

Thanks @eertugrul! Can't wait to get rid of all these ids and just work with project names.