search

bbatsche / entrust

Role-based Permissions for Laravel 4
MIT License
3 stars 4 forks source link

Consolidate filter code #6

Closed bbatsche closed 9 years ago

bbatsche commented 9 years ago

Both ability and route needs filter methods take in arrays for their parameters. Make sure they use the added functionality to can() and hasRole() rather than letting duplicate code linger.

bbatsche commented 9 years ago

Consolidating ability() would require either

  1. Further refactoring can() and hasRole() or
  2. Removing some of the flexibility of ability().

Thus, save adjusting ability to a later point release.