Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Add interface FilterEnabledInterface #4764

Conversation

mouhamed
Copy link
Contributor

@mouhamed mouhamed commented Jul 1, 2013

Add FilterEnabledInterface like StrategyEnabledInterface.
Continue discussion PR : #4752

@mouhamed mouhamed mentioned this pull request Jul 1, 2013
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 50c20a1 on mouhamed:hydrator-filter-enabled-interface into ecbb3dd on zendframework:develop.

* return false;
* }
* return true;
* }, FilterComposite::CONDITION_AND
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move this argument to the next line, and put the first argument ("servicelocator") on a line of its own (i.e., all arguments on their own lines).

weierophinney added a commit that referenced this pull request Jul 2, 2013
weierophinney added a commit that referenced this pull request Jul 2, 2013
- one argument per line
weierophinney added a commit that referenced this pull request Jul 2, 2013
@weierophinney weierophinney merged commit 50c20a1 into zendframework:develop Jul 2, 2013
@ghost ghost assigned weierophinney Jul 2, 2013
@mouhamed mouhamed deleted the hydrator-filter-enabled-interface branch July 2, 2013 19:12
@mouhamed mouhamed restored the hydrator-filter-enabled-interface branch July 2, 2013 19:13
@mouhamed mouhamed deleted the hydrator-filter-enabled-interface branch July 2, 2013 19:13
weierophinney added a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015
…rator-filter-enabled-interface

Add interface `FilterEnabledInterface`
weierophinney added a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants