Comment 1 for bug 900564

Revision history for this message
Brian Waldon (bcwaldon) wrote : Re: Context.read_deleted returns only deleted records

I'm all for showing active and deleted records. i would prefer that rather than using an enum in deleted_visibility, that do this:

1) change context.read_delete to context.can_read_deleted. This indicates that the authenticated user has access to deleted records. We should return either only active or active AND deleted records based on this attribute.

2) Support a boolean 'deleted' filter on our queries that allows users to narrow down results to only deleted or only active. This allows us to keep the behavior where an admin can see only deleted records. We also shouldn't filter on a specific value for deleted by default.