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

Support shorthand Priority filter in Log\AbstractWriter #6144

Closed

Conversation

cjthompson
Copy link
Contributor

Update the AbstractWriter constructor to accept an int filter as supported by addFilter.

Update the AbstractWriter constructor to accept an int filter as supported by addFilter.

public function testConstructorWithPriorityFilterArray()
{
$options = array('filters' => array(
Copy link
Member

Choose a reason for hiding this comment

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

You can just inline this

@cjthompson
Copy link
Contributor Author

I consolidated the array and non-array tests into a single more concise test.

@weierophinney weierophinney added this to the 2.3.2 milestone Apr 16, 2014
@weierophinney weierophinney self-assigned this Apr 16, 2014
weierophinney added a commit that referenced this pull request Apr 16, 2014
@weierophinney
Copy link
Member

Cherry-picked to master for release with 2.3.2.

weierophinney added a commit to zendframework/zend-log that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-log 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants