Fix | August 2023

[roman-right]

No description provided.

[Mark90]

As promised in #599 I'd leave a message here about the new error I'm seeing with this branch.

In summary what I think is happening is that the .fetch_all_links() works now (it doesn't give an AttributeError anymore as it did with 1.21.0) but it doesn't seem to handle nested linked documents.

My simplified document structure is like this

class Email(Document):
  ... 

class Log(Document):
  emails: list[Link[Email]]

class Ticket(Document):
  logs: list[Link[Log]]

When I have an existing Ticket instance and call .fetch_all_links() on it, it resolves the links up until ticket.logs[]; but all the ticket.logs[].emails[] are still of type beanie.odm.fields.Link.

This is also the case when I do Ticket.get(ticket_uuid, fetch_links=True).

And actually I get the same output with Ticket.get(ticket_uuid, fetch_links=False) where I would have expected ticket.logs[] to be of type Link, but they're already fetched.

[gsakkis]

@roman-right is the (omitted) else part here even valid, e.g. {"$not": {"foo": 3}}? All the examples I have seen with dict value are operator expressions, i.e. the condition here is always true. If that's the case this can be simplified. The added test doesn't cover this case so I can't tell for sure.