ZF-2005: Calendar_Extension_WebContent is declared as a direct descendant of Zend_Gdata_App_Extension

Issue Type: Bug Created: 2007-09-27T17:30:13.000+0000 Last Updated: 2011-09-09T17:39:00.000+0000 Status: Resolved Fix version(s): - Next Major Release ()

Reporter: Trevor Johns (tjohns) Assignee: Adam Lundrigan (adamlundrigan) Tags: - Zend_Gdata

Related issues: Attachments: - ZF-2005_v1.patch


Zend_Gdata_Calendar_Extension_WebContent is incorrectly declared as a subclass of Zend_Gdata_App_Extension, rather than the more specific Zend_Gdata_Extension. This should be corrected.


Posted by Trevor Johns (tjohns) on 2007-09-27T17:41:13.000+0000

Created patch for this issue as [^ZF-2005_v1.patch].

I'm debating as to whether this should be merged into the release-1.0 maintenance branch. Right now this bug doesn't break anything, though the patch has a very, very slight possibility of breaking somebody else's code. Somebody would need to be checking if a returned WebContent object is direct descendant of Zend_Gdata_App_Extension, and I can't really imagine any reason why somebody would do such a thing, but the possibility does exist.

Posted by Adam Lundrigan (adamlundrigan) on 2011-05-06T01:55:32.000+0000

As original poster mentions, there is a possibility of a BC break when correcting the name of parent class. Recommend this be fixed in Zend_Gdata for ZF2.

Posted by Adam Lundrigan (adamlundrigan) on 2011-09-09T00:11:28.000+0000

-This issue was fixed in ZF2 repository when Gdata was converted to use namespaces.- Oops. That's not true. Working on a pull request now.

Posted by Adam Lundrigan (adamlundrigan) on 2011-09-09T00:20:31.000+0000

Submitted pull request against zendframework/zf2 branch master

Posted by Adam Lundrigan (adamlundrigan) on 2011-09-09T17:39:00.000+0000

Pull request merged into zendframework/zf2 branch master…

Have you found an issue?

See the Overview section for more details.


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

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