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

[2.1] Added Zend\Stdlib\StringUtils::hasPcreUnicodeSupport() #3495

Merged
merged 1 commit into from
Jan 21, 2013
Merged

[2.1] Added Zend\Stdlib\StringUtils::hasPcreUnicodeSupport() #3495

merged 1 commit into from
Jan 21, 2013

Conversation

marc-mabe
Copy link
Member

The method is a copy of Zend\Filter\AbstractFilter::hasPcreUnicodeSupport() but within Zend\Stdlib\StringUtils it's more reusable an can be used without Zend\Filter.

Zend\Stdlib isn't marked as a dependency of Zend\Filter but this method is using Zend\Stdlib\ErrorHandler. If we mark it as a dependency we could redirect and deprecate the method else the use of ErrorHandler should be removed.

weierophinney added a commit that referenced this pull request Jan 21, 2013
- Updated Filter classes to use StringUtils::hasPcreUnicodeSupport
- AbstractFilter::hasPcreUnicodeSupport now proxies to
  StringUtils::hasPcreUnicodeSupport
  - docblock marks the method as deprecated
- Added zendframework/zend-stdlib to composer.json (should have been
  there regardless)
@weierophinney weierophinney merged commit 4913c64 into zendframework:develop Jan 21, 2013
@weierophinney
Copy link
Member

Merged, and made the changes suggested to the Filter component.

@ghost ghost assigned weierophinney Jan 21, 2013
weierophinney added a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-filter that referenced this pull request May 15, 2015
…port

- Updated Filter classes to use StringUtils::hasPcreUnicodeSupport
- AbstractFilter::hasPcreUnicodeSupport now proxies to
  StringUtils::hasPcreUnicodeSupport
  - docblock marks the method as deprecated
- Added zendframework/zend-stdlib to composer.json (should have been
  there regardless)
weierophinney added a commit to zendframework/zend-filter 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.

2 participants