Issues

ZF-2105: Zend_Filter::get() could be better named

Issue Type: Improvement Created: 2007-10-26T15:02:39.000+0000 Last Updated: 2009-07-25T10:09:24.000+0000 Status: Resolved Fix version(s): - 1.9.0 (31/Jul/09)

Reporter: Darby Felton (darby) Assignee: Thomas Weidner (thomas) Tags: - Zend_Filter

Related issues: Attachments:

Description

Zend_Filter::get() may be better named staticFilter() or something similar, the goal being to indicate what the method does.

Comments

Posted by Wil Sinclair (wil) on 2008-03-21T17:05:28.000+0000

This issue should have been fixed for the 1.5 release.

Posted by Wil Sinclair (wil) on 2008-04-18T13:12:03.000+0000

This doesn't appear to have been fixed in 1.5.0. Please update if this is not correct.

Posted by Wil Sinclair (wil) on 2008-04-18T17:11:46.000+0000

Please evaluate and categorize/assign as necessary.

Posted by Andries Seutens (andries) on 2008-04-24T08:05:21.000+0000

+1 for getStaticFilter()

In the MVC layer Matthew has used this naming for the helperBroker too HelperBroker::getStaticHelper('x')

Posted by Wil Sinclair (wil) on 2008-12-30T06:20:29.000+0000

Matthew, please decide what we should do with this. 3 possibilities that I see: close as 'will not fix', mark as 'postponed' and fix version 'next major release', or handle it now by deprecating the get() method and adding the new method. The last possibility has the advantage that users could start migrating their apps now, although we may need to firm up our deprecation policies and processes before taking such actions.

Posted by Thomas Weidner (thomas) on 2009-06-26T12:59:22.000+0000

Under code review since 26.06.2009

Posted by Thomas Weidner (thomas) on 2009-07-25T10:09:18.000+0000

Implemented with r17073.

After discussion with the dev-team we changed the function name to Zend_Filter::filterStatic()

Have you found an issue?

See the Overview section for more details.

Copyright

© 2006-2016 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.

Contacts