ZF-2893: Detach Zend_InfoCard from Zend_Loader

Issue Type:
Improvement
Created:
2008-03-16T13:28:12.000+0000
Last Updated:
2009-07-25T22:47:44.000+0000
Status:
Resolved
Fix version(s):
  • 1.6.0 (02/Sep/08)
Reporter:
Martin Hujer (mhujer)
Assignee:
Thomas Weidner (thomas)
Tags:
  • Zend_InfoCard
Related issues:
Attachments:

Description

All ZF components should be loosely coupled and be able to be used alone without other components if possible.

Zend_InfoCard (line 144):


 Zend_Loader::loadClass('Zend_InfoCard_Adapter_Default');

A simple require_once would be faster and would decouple it from Zend_Loader, which makes it possible to use Zend_InfoCard without Zend_Loader.

Comments

Posted by Wil Sinclair (wil) on 2008-03-25T20:46:38.000+0000

Please categorize/fix as needed.

Posted by Matthew Weier O'Phinney (matthew) on 2008-04-22T11:20:34.000+0000

Assigning to Darby to evaluate and schedule.

Posted by Martin Hujer (mhujer) on 2008-08-06T16:31:49.000+0000

Attaching very short path:


Index: Zend/InfoCard.php
===================================================================
--- Zend/InfoCard.php   (revision 10697)
+++ Zend/InfoCard.php   (working copy)
@@ -138,8 +138,8 @@
      */
     public function getAdapter()
     {
-        if(is_null($this->_adapter)) {
-            Zend_Loader::loadClass('Zend_InfoCard_Adapter_Default');
+        if(is_null($this->_adapter)) {
+            require_once 'Zend/InfoCard/Adapter/Default.php';
             $this->setAdapter(new Zend_InfoCard_Adapter_Default());
         }

Posted by Thomas Weidner (thomas) on 2008-08-24T12:29:03.000+0000

Fixed with r11010

Posted by Wil Sinclair (wil) on 2008-09-02T10:39:41.000+0000

Updating for the 1.6.0 release.