Add 'has' filter to CaseCollection #328
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
#325
Short description
CaseCollection
constructor and the methodCaseCollection.filter()
have a new optional parameter calledhas
. Thefilter()
method generally returns a newCaseCollection
instance; ifhas
is specified, it is passed on to the constructor, which stores it in the member slot_has
.DocumentCollection
involve the_next_batch()
/_next_batch_async()
methods. These methods are overridden byCaseCollection
._next_batch()
/_next_batch_async()
forCaseCollection
checks if the member slot_has
is set. If it is, wefmu.case.uuid
for the existing filters for theCaseCollection
instance._has
with the set of values forfmu.case.uuid
. Note: this new query applies to all object types, and not justcase
objects.fmu.case.uuid
for this new query.CaseCollection
instance to something like{"ids": {"values": <uuids>}}
_has
member attribute toNone
._has
attribute,_next_batch()
/_next_batch_async()
passes control to the parent class implementation.Pre-review checklist
print()
statements, commented-out code, or other remnants from the development. 👀Pre-merge checklist