Refactor permissions check

[ericfranz]

I thought about it more and if you refactor to add a new user_has_rx?(pathname) or user_has_permissions?(pathname) then the there ends up being less code since you can get by without the for loop cause the final line of the method could be:

facl.allow?(principle: ood_user, permission: :r) && facl.allow?(principle: ood_user, permission: :x)

In that case maybe good for clarity in the future and less code.