BHoM / Rhinoceros_Toolkit

Set of functionalities for communication with Rhinoceros
GNU Lesser General Public License v3.0
7 stars 4 forks source link

Wrong null check in FromRhino #184

Closed alelom closed 3 years ago

alelom commented 3 years ago

Description:

This change during the current Milestone has broken the component FromRhino: https://github.com/BHoM/Rhinoceros_Toolkit/pull/178/files#diff-3172cad297267b3731759fe48c33df6704aca3363264c6538b2167bace68d56eR669-R681

The null check needs to be changed.

Steps to reproduce:

Open the script: https://burohappold.sharepoint.com/:f:/s/BHoM/EoVl3E4-5gRAr4kqCF6zyWcB-te-XrdJBYh7IqEc8oOqkw?e=gDkqE7

If you open the script using BHoM v.3.3, you see the component FromRhino behaves correctly:

![image](https://user-images.githubusercontent.com/6352844/102337879-f7c74300-3f8a-11eb-9c98-23ce1b251f2e.png)

If you open the script using latest master, the component returns the error "No conversion could be found between [object] and Rhino geometry."

![image](https://user-images.githubusercontent.com/6352844/102338346-a53a5680-3f8b-11eb-986e-7ceff0972fed.png)

Expected behaviour:

No error should be returned, conversion should succeed.

Test file(s):

See above.