Zend Framework

The ZF is not able to use user-defined autoloaders for user-defined classes

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.5.1
  • Fix Version/s: 1.8.2
  • Component/s: Zend_Loader
  • Labels:
    None

Description

Some ZF classes load user-defined classes (such as Zend_Db_Table_Abstract::fetchRow()) with Zend_Locale::loadClass(). However, the loadClass() method will not work if the user-defined class is not in the PHP's include_path and/or has an application-specific prefix. User-defined autoloaders would work, but Zend_Locale::loadClass() doesn't use them, even if it couldn't load the requested class.

Zend_Locale::loadClass() should use a third optional parameter, $tryUserAutoloaders = false, for this method to try loading the requested class with user-defined autoloaders if it itself couldn't load the requested class.

I'm going to fix this because I need it for my code to work.

  1. patch_to_bug_3151.dif
    22/Apr/08 10:17 AM
    3 kB
    Gustavo Narea
  2. patch_to_bug_3151.dif
    19/Apr/08 11:02 AM
    2 kB
    Gustavo Narea
  3. patch_to_bug_3151-3rd_edition.dif
    22/Apr/08 11:41 AM
    5 kB
    Gustavo Narea
  4. patch-zf-v4.dif
    02/May/08 12:23 PM
    7 kB
    Gustavo Narea

Issue Links

Activity

Hide
Gustavo Narea added a comment -

The path to fix the bug.

Show
Gustavo Narea added a comment - The path to fix the bug.
Hide
Gustavo Narea added a comment -

The previous patch was incomplete. This also takes into account other instances where the third optional parameter of loadClass() should be used.

Show
Gustavo Narea added a comment - The previous patch was incomplete. This also takes into account other instances where the third optional parameter of loadClass() should be used.
Hide
Gustavo Narea added a comment -

Hopefully, this is the final edition of the patch (I've found more code to fix).

Show
Gustavo Narea added a comment - Hopefully, this is the final edition of the patch (I've found more code to fix).
Hide
Gustavo Narea added a comment -

I found some more code I had to fix, hence the fourth version of my patch.

Show
Gustavo Narea added a comment - I found some more code I had to fix, hence the fourth version of my patch.
Hide
Matthew Weier O'Phinney added a comment -

Postponed until 2.0.

Show
Matthew Weier O'Phinney added a comment - Postponed until 2.0.
Hide
Matthew Weier O'Phinney added a comment -

This should now be resolved in trunk and the 1.8 release branch.

Show
Matthew Weier O'Phinney added a comment - This should now be resolved in trunk and the 1.8 release branch.

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
Not Specified
Original Estimate - Not Specified
Remaining:
0m
Remaining Estimate - 0 minutes
Logged:
3h
Time Spent - 3 hours