buildingSMART / buildingSMART/bSDD

/api/Class/v1 bug on URI of external Properties

Open
#142 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
193
Forks
45
PR merge metrics
No merged PRs in 30d

Description

**Describe the bug**
When Requesting a Class with` IncludeClassProperties = True `the URI of ClassProperties that reference external Properties is wrong

**To Reproduce**
Steps to reproduce the behavior:
1. request
```cmd
curl -X "GET" ^
"https://api.bsdd.buildingsmart.org/api/Class/v1?Uri=https%3A%2F%2Fidentifier.buildingsmart.org%2Furi%2Fhw%2Fsom%2F0.2.2%2Fclass%2FAlarm&IncludeClassProperties=true" ^
-H "accept: text/plain"
```
2. look at result["classProperties"][8]["uri"]

```python
>>> from bsdd import Client
>>> client = Client()
>>> class_uri = 'https://identifier.buildingsmart.org/uri/hw/som/0.2.2/class/Alarm'
>>> result = client.get_class(
... class_uri,
... include_class_properties=True,
... include_class_relations=False,
... include_reverse_relations=False,
... )
>>> print(result["classProperties"][8]["uri"])
https://identifier.buildingsmart.org/uri/hw/som/0.2.2/class/Alarm/prop/Pset_AlarmTypeCommon/uri/buildingsmart/ifc/4.3/prop/AlarmCondition
```

**Expected behavior**
the URI should be 'https://identifier.buildingsmart.org/uri/hw/som/0.2.2/class/Alarm/prop/Pset_AlarmTypeCommon/AlarmCondition'
instead, parts of the URI of the IFC Property are added

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.