plone / guillotina

Python AsyncIO data API to manage billions of resources
https://guillotina.readthedocs.io/en/latest/
Other
187 stars 50 forks source link

Fix not checking if the ID is valid when duplicating a resource #1183

Closed masipcat closed 1 year ago

masipcat commented 1 year ago

https://github.com/vinissimus/product/issues/5812

codecov-commenter commented 1 year ago

Codecov Report

Merging #1183 (64e3676) into master (6ee373b) will decrease coverage by 0.2%. The diff coverage is 100.0%.

Additional details and impacted files [![Impacted file tree graph](https://codecov.io/gh/plone/guillotina/pull/1183/graphs/tree.svg?width=650&height=150&src=pr&token=MIUJDWnGXD&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone)](https://codecov.io/gh/plone/guillotina/pull/1183?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone) ```diff @@ Coverage Diff @@ ## master #1183 +/- ## ======================================== - Coverage 94.7% 94.6% -0.1% ======================================== Files 358 377 +19 Lines 30700 32685 +1985 ======================================== + Hits 29069 30908 +1839 - Misses 1631 1777 +146 ``` | [Impacted Files](https://codecov.io/gh/plone/guillotina/pull/1183?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone) | Coverage Δ | | |---|---|---| | [guillotina/content.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9jb250ZW50LnB5) | `93.1% <100.0%> (ø)` | | | [guillotina/tests/test\_api.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS90ZXN0cy90ZXN0X2FwaS5weQ==) | `99.7% <100.0%> (+0.1%)` | :arrow_up: | | [guillotina/tests/conftest.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS90ZXN0cy9jb25mdGVzdC5weQ==) | `100.0% <0.0%> (ø)` | | | [guillotina/response.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9yZXNwb25zZS5weQ==) | `88.0% <0.0%> (ø)` | | | [guillotina/event.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9ldmVudC5weQ==) | `100.0% <0.0%> (ø)` | | | [guillotina/tests/fixtures.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS90ZXN0cy9maXh0dXJlcy5weQ==) | `92.9% <0.0%> (ø)` | | | [guillotina/permissions.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9wZXJtaXNzaW9ucy5weQ==) | `100.0% <0.0%> (ø)` | | | [guillotina/async\_util.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9hc3luY191dGlsLnB5) | `87.5% <0.0%> (ø)` | | | [guillotina/exceptions.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9leGNlcHRpb25zLnB5) | `89.1% <0.0%> (ø)` | | | [guillotina/registry.py](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone#diff-Z3VpbGxvdGluYS9yZWdpc3RyeS5weQ==) | `88.2% <0.0%> (ø)` | | | ... and [11 more](https://codecov.io/gh/plone/guillotina/pull/1183/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=plone) | |