Suppress SimpleXmlElement warning (ZF-9039)

git-svn-id: http://framework.zend.com/svn/framework/standard/trunk@21359 44c647ce-9c0f-0410-b52a-842ac1e357ba

library/Zend/XmlRpc/Response.php

@@ -177,8 +177,11 @@ class Zend_XmlRpc_Response
         }
 
         try {
+            $useInternalXmlErrors = libxml_use_internal_errors(true);
             $xml = new SimpleXMLElement($response);
+            libxml_use_internal_errors($useInternalXmlErrors);
         } catch (Exception $e) {
+            libxml_use_internal_errors($useInternalXmlErrors);
             // Not valid XML
             $this->_fault = new Zend_XmlRpc_Fault(651);
             $this->_fault->setEncoding($this->getEncoding());

tests/Zend/XmlRpc/ResponseTest.php

@@ -44,6 +44,11 @@ class Zend_XmlRpc_ResponseTest extends PHPUnit_Framework_TestCase
     protected $_response;
 
     /**
+     * @var bool
+     */
+    protected $_errorOccured = false;
+
+    /**
      * Setup environment
      */
     public function setUp()
@@ -137,6 +142,19 @@ class Zend_XmlRpc_ResponseTest extends PHPUnit_Framework_TestCase
     }
 
     /**
+     * @group ZF-9039
+     */
+    public function testExceptionIsThrownWhenInvalidXmlIsReturnedByServer()
+    {
+        set_error_handler(array($this, 'handleError'));
+        $invalidResponse = 'foo';
+        $response = new Zend_XmlRpc_Response();
+        $this->assertFalse($this->_errorOccured);
+        $this->assertFalse($response->loadXml($invalidResponse));
+        $this->assertFalse($this->_errorOccured);
+    }
+
+    /**
      * @group ZF-5404
      */
     public function testNilResponseFromXmlRpcServer()
@@ -232,4 +250,9 @@ EOD;
         } catch (Exception $e) {
         }
     }
+
+    public function handleError($error)
+    {
+        $this->_errorOccured = true;
+    }
 }