Comment 5 for bug 1049374

Revision history for this message
Cody A.W. Somerville (cody-somerville) wrote :

I appreciate you tackling this Ian.

I hate to be difficult but I'd probably actually prefer the old behaviour over having a method for each different type of artifact.

1. Considering the most likely use case of this functionality, auditing, I think it makes a lot of sense for there to be a single method so that scripts don't have to be updated as privacy is added to additional types of artefacts.
2. It would also be nice to use the same API as the sharing details view to remove the risk of a bug causing discrepancies between the two.
3. There is significant overhead for each API call.

What do you think?