False invalid schema status + invalid errors thrown for service with multiple annotations to same target
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
Research direction
Start with the metadata file linked in the issue and search for Target="FAR_MANAGE_PAYMENT_ADVICES_SRV.C_PaymentAdviceType/PaymentAdviceAccount". Trace the pyodata validator handling annotations with that target and reproduce the service initialization using pyodata 1.7.0. Done means the service accepts multiple annotations for one target without reporting an invalid schema or throwing invalid errors.
Written by the indexing model from the issue text.
Description
This issue is reproducible with pyodata 1.7.0 and metadata file in https://github.com/phanak-sap/pyodata-issue-files/tree/master/%23142
Root cause is that multiple annotations having same target, which is OK by odata standard. Search Target="FAR_MANAGE_PAYMENT_ADVICES_SRV.C_PaymentAdviceType/PaymentAdviceAccount" in the metadata file.
Pyodata validator wrongly expects 0..1 relationship only.
Possible solutions:
A) fix the validation for 0..* relationships
B) ignore the validation of annotations completely (since there could be literally anything, including other annotation definition), similar to our JS counterpart library, https://github.com/SAP/odata-library - where the service is initialized without problem.
- Dominant language
- Python
- Stars
- 268
- Forks
- 100
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from SAP/python-pyodata
-
chore
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
SAP/python-pyodata#304 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
SAP/python-pyodata#108 ·
-
chore
SAP/python-pyodata#302 · 1 assignee ·
-
question
SAP/python-pyodata#294 · 1 comment · 1 assignee ·
-
bug
SAP/python-pyodata#282 · 2 comments · 1 assignee ·
All issues in SAP/python-pyodata
Similar issues
-
Add: hunch Open
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
DiamondLightSource/dodal#2211 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
openml/openml-python#1749 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
sipyourdrink-ltd/bernstein#6191 ·