Stack filters #362

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

codler commented Feb 24, 2012

Makes it possible to stack filter. This is useful if for example a bundle wants to add something to the "after" filter and not override the applications "after" filter.

Owner

taylorotwell commented Feb 24, 2012

Cool, I'll look it over. Should point out though that there is a way to have a bundle global filter. Just prefix the filter with the bundle's name:

Filter::register('bundle::after', function() {});

I think I forgot to document this.

Contributor

codler commented Feb 26, 2012

bundle::after doesn't seems to work.

Contributor

codler commented Feb 26, 2012

nvm, I needed to set "handles" in bundle.php

Contributor

jasonlewis commented Feb 29, 2012

Filters shouldn't stack. I still think the before/after global events would be useful. That way it can stack. :)

Owner

taylorotwell commented Mar 5, 2012

Going to close this pending more requests. Global filters are already "stackable" using the bundle's filters.

@thomasedwards thomasedwards pushed a commit to thomasedwards/laravel that referenced this pull request Oct 14, 2013

@bencorlett bencorlett Allow any implementation of ArrayableInterface be passed to views. Fixes
 #362.

Signed-off-by: Ben Corlett <bencorlett@me.com>
2b16fc0

@thomasedwards thomasedwards pushed a commit to thomasedwards/laravel that referenced this pull request Oct 14, 2013

@taylorotwell taylorotwell Merge pull request #363 from bencorlett/feature/arrayable-views
Allow any implementation of ArrayableInterface be passed to views. Fixes #362
76a21a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment