Access right mixin

Closes #588 (closed)

Codecov Report

Merging #1131 (055d49b7) into master (1c0b423c) will increase coverage by 0.09%. The diff coverage is 97.02%.

@@            Coverage Diff             @@
##           master    #1131      +/-   ##
==========================================
+ Coverage   87.88%   87.98%   +0.09%     
==========================================
  Files         246      248       +2     
  Lines       16241    16428     +187     
==========================================
+ Hits        14274    14454     +180     
- Misses       1967     1974       +7     

Impacted Files	Coverage Δ
arkindex/project/tests/__init__.py	83.33% <ø> (ø)
arkindex/users/models.py	10.28% <0.00%> (-0.30%)
arkindex/users/utils.py	96.00% <96.00%> (ø)
arkindex/project/mixins.py	92.59% <97.56%> (+2.16%)
arkindex/project/tests/test_acl_mixin.py	100.00% <100.00%> (ø)
arkindex/users/api.py	88.77% <100.00%> (ø)
arkindex/users/serializers.py	98.78% <100.00%> (ø)
arkindex/users/tests/test_membership.py	100.00% <100.00%> (ø)
arkindex/documents/tests/test_classes.py	99.22% <0.00%> (-0.78%)
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1c0b423...055d49b. Read the comment docs.

added 15 commits

• c82c4d91...78145b4c - 8 commits from branch master
• 8e7ad137 - Update models
• 1c70e7f3 - Generic ACL mixin
• 533c3b53 - Mixin tests
• c1d9642b - Use a generic Role access level
• b160893b - Rebase
• 2e8e20f2 - Correctly handle public option
• 92b7e4a6 - Add mixin and public attr tests

Compare with previous version

marked this merge request as ready

approved this merge request

added 1 commit

• b3d272de - Suggestions

Compare with previous version

resolved all threads

resolved all threads

added 3 commits

• 906b62ff - Allow custom order_by
• 25bdf36d - Correct handling of special authentications in the mixin
• fc646117 - Add tests

Compare with previous version

resolved all threads

added 15 commits

• e5c9e105 - 1 commit from branch master
• d59da64b - Update models
• 6428be31 - Generic ACL mixin
• 844d4fd1 - Mixin tests
• 2444ae5f - Use a generic Role access level
• e68cbf3c - Rebase
• fa9bb736 - Correctly handle public option
• 357822b9 - Add mixin and public attr tests
• c92df885 - Handle anonymous user
• 93346b9e - Handle public instances in the ACL mixin
• d6953dfc - Update tests
• 8c596f5d - Suggestions
• 861e4cb7 - Allow custom order_by
• 007a6783 - Correct handling of special authentications in the mixin
• 4d11bd2b - Add tests

Compare with previous version

marked this merge request as draft

resolved all threads

added 1 commit

• da402551 - Add a test case

Compare with previous version

resolved all threads

added 2 commits

• 189d571f - Implement the force approach
• 9aba6d99 - Update SQL tests

Compare with previous version

marked this merge request as ready

resolved all threads

added 4 commits

• 7b212e7f - Implement the force approach
• af1bc8ae - Revert "Implement the force approach"
• ed70fa71 - Split filter_rights function
• 329ad98d - Update tests

Compare with previous version

resolved all threads

added 23 commits

• 329ad98d...174e435f - 4 commits from branch master
• b98fa971 - Update models
• bb24e041 - Generic ACL mixin
• 0455c73e - Mixin tests
• 9857bdfd - Use a generic Role access level
• f12aac1b - Rebase
• d96959d4 - Correctly handle public option
• adac5da6 - Add mixin and public attr tests
• adbf5946 - Handle anonymous user
• 0f6c78a8 - Handle public instances in the ACL mixin
• 9c11c777 - Update tests
• f4497147 - Suggestions
• a4807bc9 - Allow custom order_by
• dfd12889 - Correct handling of special authentications in the mixin
• e700a1bd - Add tests
• 740d8999 - Add a test case
• 76f39ef2 - Implement the force approach
• 92f4d352 - Revert "Implement the force approach"
• 9a50a538 - Split filter_rights function
• 7cabbaed - Update tests

Compare with previous version

resolved all threads

added 2 commits

• 278ce198 - Split filter_rights function
• 1b65632e - Update tests

Compare with previous version

resolved all threads

added 2 commits

• 0c034eff - Split filter_rights function
• 3238f0c0 - Update tests

Compare with previous version

@babadie this MR should be ok right now

resolved all threads

added 1 commit

• 9512f8cf - Remove a useless check

Compare with previous version

resolved all threads

added 1 commit

• 7444d87c - Remove unused code in the mixin

Compare with previous version

mentioned in issue #593 (closed)

resolved all threads

added 1 commit

• 211aef3d - Fixes

Compare with previous version

resolved all threads

added 1 commit

• 7b70e103 - Fixes

Compare with previous version

resolved all threads

added 1 commit

• 547f8a51 - Suggestion

Compare with previous version

resolved all threads

added 24 commits

• 547f8a51...1c0b423c - 2 commits from branch master
• 1a69dfbc - Update models
• 637ee2b0 - Generic ACL mixin
• 9319a16a - Mixin tests
• a539cbc3 - Use a generic Role access level
• 367a533c - Rebase
• 9f35137b - Correctly handle public option
• c49d2d21 - Add mixin and public attr tests
• 8bcf5870 - Handle anonymous user
• c29ddc2b - Handle public instances in the ACL mixin
• 77773799 - Update tests
• dc963ce1 - Suggestions
• ba07a91f - Allow custom order_by
• 947b33a5 - Correct handling of special authentications in the mixin
• 15654183 - Add tests
• b1920f18 - Add a test case
• 8f6902fa - Implement the force approach
• bf75a543 - Revert "Implement the force approach"
• db9cb158 - Split filter_rights function
• dda69469 - Update tests
• c6fcdcef - Remove unused code in the mixin
• 49e7a532 - Fixes
• 055d49b7 - Suggestion

Compare with previous version

enabled an automatic merge when the pipeline for 055d49b7 succeeds

mentioned in commit 782f8d32

merged

mentioned in issue #588 (closed)

changed the description