Update all EmbeddedDocument at once

[teamhide]

Here is my models.

class Attachment(EmbeddedDocument):
    path = StringField(required=True)
    is_deleted = BooleanField(default=False)

class Test(Document):
    test_id = SequenceField()
    attachments = ListField(EmbeddedDocumentField(Attachment), required=False)

Until now, to set is_deleted=True all embedded document, I used below code.

test = Test.objects(test_id=test_id)
for attachment in test.attachments:
    delete_attachments(test_id=attachment.test_id)

def delete_attachments(test_id: int) -> None:
    Test.objects(
        test_id=test_id,
    ).update(set__attachments__S__is_deleted=True)

Is it a only way to update all document?

Is there any update_all method here?

Thanks.

[bagerard]

Hi @teamhide. From what I see in the doc, if you are using mongo>3.5, there is an operator that got introduced (link) and that you could use using __raw__ as in:
Test.objects(test_id=test_id).update(__raw__={'$set': {'attachments.$[].is_deleted': True}})

Otherwise, I think your best option is to do the update on the Document and call .save()

test = Test.objects(test_id=test_id)
for attachment in test.attachments:
    attachment.is_deleted = True
test.save()

Let me know if you still have problems with this

[erdenezul]

Can we implement this feature like update(set__attachments__S__[]__is_deleted=True) ?

[teamhide]

@bagerard Both way works perfectly. Anyway, which way is more faster? (I think raw query is faster than for loop)

[teamhide]

@erdenezul It throw syntax error.

[erdenezul]

@teamhide yes raw query is faster than for loop because it sends one query instead of many.

[erdenezul]

@teamhide I mean we develop new feature instead of using raw

[teamhide]

@erdenezul I see. I have one more question.

If I use raw query Test.objects(test_id=test_id).update(__raw__={'$set': {'attachments.$[].is_deleted': True}}), How . can I trigger condition likepath == '1.jpg'?

I tried to query with Q like this,

Test.objects(Q(test_id=test_id) & Q(attachments__path='1.jpg')).update(__raw__={'$set': {'attachments.$[].is_deleted': True}})

But it update all embedded documents.

Maybe it can be implement with second answer(using for loop). Or add condition into update()?

[erdenezul]

I think answer is the second. add condition into update

[erdenezul]

$[<identifier>] | Acts as a placeholder to update all elements that match the arrayFilters condition for the documents that match the query condition.

https://docs.mongodb.com/manual/reference/operator/update-array/

[bagerard]

@erdenezul Adding support for _S_[]_ would be nice but we'll need another convention because that's not valid syntax, any idea?