Zend Framework

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.9.2
• Fix Version/s: 1.9.5
• Component/s: Zend_Db
• Labels:

  None

This is actually for Zend_Db_Statement_Oracle, but that doesn't show up in the list of Components.

I'm looking at version 1.9.2 (Minimal). In Zend/Db/Statement/Oracle.php in the _execute() function there is some unreachable code. The same conditional is checked twice, and there's a return false after the first one, so the throwing of the exception is unreachable in the seconde. The coding style is also different for the two conditionals, one with and one without a space between the ! and the $this->_stmt.

$connection = $this->_adapter->getConnection();
if (!$this->_stmt) {
    return false;
}

if (! $this->_stmt) {
    /**
     * @see Zend_Db_Adapter_Oracle_Exception
     */
    require_once 'Zend/Db/Statement/Oracle/Exception.php';
    throw new Zend_Db_Statement_Oracle_Exception(oci_error($connection));
}

• Assignee:

  Ralph Schindler

  Reporter:

  Brian Tipton

• Created:

  18/Sep/09 7:58 AM

  Updated:

  16/Oct/09 10:45 PM

  Resolved:

  16/Oct/09 10:45 PM