selection_perm() returns decisions_perm(). There's no need to redefine the same permissions in order for Selection to use them. Selection is a dependency of Decisions.

CommentFileSizeAuthor
#2 selection_perm_etc.patch917 bytesezra-g

Comments

ezra-g’s picture

ezra-g commented

Actually, it looks like we can eliminate selection_access() and selection_form() as well.

ezra-g’s picture

ezra-g commented
Status: Active » Needs review
StatusFileSize
new selection_perm_etc.patch917 bytes

And here's a patch that does that.

ezra-g’s picture

ezra-g commented
Status: Needs review » Fixed

This is now committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.