Address inefficiencies and possible misfunction over delegated mailboxes in other user namespace

Authored by vanmeeuwen on Jul 21 2017, 12:16 PM.

Diff Detail

rP pykolab
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vanmeeuwen created this revision.Jul 21 2017, 12:16 PM
machniak requested changes to this revision.Jul 21 2017, 12:40 PM
machniak added a subscriber: machniak.
machniak added inline comments.

We need only two annotations. Wouldn't be possible to extend get_metadata() to fetch only these?


I think we could just store the folder type (with subtype) here, giving the private annotation a prio over shared. Making all checks below much more simpler.

This revision now requires changes to proceed.Jul 21 2017, 12:40 PM
vanmeeuwen requested review of this revision.Jul 21 2017, 12:55 PM
vanmeeuwen edited edge metadata.
vanmeeuwen added inline comments.

Yes it could, but that's an efficiency parameter that is otherwise unrelated to the particular ticket this attempts to resolve.


Changing the definition of what is stored, and how it is stored will be a cause of great confusion later down the line.

Note that we don't mutate data here, and we don't mutate the data structure either -- we're just filtering out irrelevant entries.

vanmeeuwen accepted this revision.Jul 22 2017, 1:06 PM
This revision is now accepted and ready to land.Jul 22 2017, 1:06 PM
This revision was automatically updated to reflect the committed changes.