ZF-7233: Zend_InfoCard_Exception_Abstract declared twice in Zend/InfoCard/Exception.php

Issue Type: Patch Created: 2009-07-09T08:16:57.000+0000 Last Updated: 2012-11-20T20:53:16.000+0000 Status: Closed Fix version(s): Reporter: Apaella (apaella) Assignee: None Tags: - Zend_InfoCard

Related issues: Attachments:


Zend_InfoCard_Exception_Abstract declared twice: {quote} if (class_exists("Zend_Exception")) { abstract class Zend_InfoCard_Exception_Abstract extends Zend_Exception { } } else { abstract class Zend_InfoCard_Exception_Abstract extends Exception { } } {quote}

Proposed substitution: {quote} if (!class_exists("Zend_Exception")) { require_once 'Zend/Exception.php'; }

abstract class Zend_InfoCard_Exception_Abstract extends Zend_Exception { } {quote}


Posted by John Coggeshall (coogle) on 2009-07-10T00:03:59.000+0000

This is intentional, as the Zend_InfoCard component is shipped separately from Zend Framework as a whole and thus the Zend_Exception class is not always available.

Posted by Ben Scholzen (dasprid) on 2009-07-29T12:38:07.000+0000

Well, but this is currently against the coding standard, and Zend_Infocard could, just like the other standalone package Zend_Amf, ship Zend_Exception with it.

Posted by Thomas Weidner (thomas) on 2009-07-29T12:40:49.000+0000

Also Zend_Gdata is shipped as standalone package with Zend_Exception and does not use such a switch.

Posted by Tobias Gies (tobias) on 2009-07-29T12:40:56.000+0000

I agree with Ben on this. Shipping Zend_Exception with Zend_InfoCard does neither hurt performance nor add significant size to the download, and keeping consistence with the other standalone component can't be too bad either. Violating the CS for something as trivial as an Exception class ain't worth it imho.

Posted by Rob Allen (rob) on 2012-11-20T20:53:16.000+0000

Bulk change of all issues last updated before 1st January 2010 as "Won't Fix".

Feel free to re-open and provide a patch if you want to fix this issue.

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.