Zend Framework

Inconsistency in setPluginLoader functionality in Zend_View_Abstract

Details

  • Type: Improvement Improvement
  • Status: Postponed Postponed
  • Priority: Blocker Blocker
  • Resolution: Unresolved
  • Affects Version/s: 1.6.0, 1.6.1, 1.6.2, 1.7.0, 1.7.1, 1.7.2, 1.7.3, 1.7.4, 1.7.5, 1.7.6, 1.7.7, 1.7.8, 1.8.0, 1.8.1, 1.8.2, 1.8.3, 1.8.4, 1.9.0, 1.9.1, 1.9.2, 1.9.3, 1.9.4, 1.9.5
  • Fix Version/s: None
  • Component/s: Zend_View
  • Labels:
    None

Description

Zend_View_Abstract is the only class that restricts usage to the defined Zend_Loader_PluginLoader. Other components like Zend_Form, Zend_Filter_Inflector, Zend_Controller_Action_HelperBroker allow usage of another defined pluginloader via the Zend_Loader_PluginLoader_Interface

Activity

Hide
Jan Pieper added a comment -

Zend_View_Abstract::setPluginLoader() needs Zend_Loader_PluginLoader instead of Zend_Loader_PluginLoader_Interface because Zend_View_Abstract::getHelperPaths() is using a getPaths()-method that is not declared in the interface, so we cannot change this.

A way to solve this is to add getPaths() to Zend_Loader_PluginLoader_Interface, but this will break backwards compatibility.

Show
Jan Pieper added a comment - Zend_View_Abstract::setPluginLoader() needs Zend_Loader_PluginLoader instead of Zend_Loader_PluginLoader_Interface because Zend_View_Abstract::getHelperPaths() is using a getPaths()-method that is not declared in the interface, so we cannot change this. A way to solve this is to add getPaths() to Zend_Loader_PluginLoader_Interface, but this will break backwards compatibility.
Hide
Matthew Weier O'Phinney added a comment -

Postponing to 2.0.

Zend_View_Abstract already correctly requires Zend_Loader_PluginLoader (and not the interface), so getPaths() must be defined in any plugin loader attached to the view helper.

That said, this should change in the future, but, also as noted, it cannot be done cleanly right now without a BC change.

Show
Matthew Weier O'Phinney added a comment - Postponing to 2.0. Zend_View_Abstract already correctly requires Zend_Loader_PluginLoader (and not the interface), so getPaths() must be defined in any plugin loader attached to the view helper. That said, this should change in the future, but, also as noted, it cannot be done cleanly right now without a BC change.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated: