Promote DOM invalid state errors during property access

Closes GH-6780

diff --git a/ext/dom/attr.c b/ext/dom/attr.c
index e348f8fc5abafe930a0c8498ba0a82ccb2b97ebd..383f39095182c976588ece6602fdf965809124d2 100644 (file)
--- a/ext/dom/attr.c
+++ b/ext/dom/attr.c
@@ -81,7 +81,7 @@ int dom_attr_name_read(dom_object *obj, zval *retval)
        attrp = (xmlAttrPtr) dom_object_get_node(obj);
 
        if (attrp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -117,7 +117,7 @@ int dom_attr_value_read(dom_object *obj, zval *retval)
        xmlChar *content;
 
        if (attrp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -138,7 +138,7 @@ int dom_attr_value_write(dom_object *obj, zval *newval)
        xmlAttrPtr attrp = (xmlAttrPtr) dom_object_get_node(obj);
 
        if (attrp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -171,7 +171,7 @@ int dom_attr_owner_element_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/characterdata.c b/ext/dom/characterdata.c
index cd332c8db4cd9b721fce765c11e05156517fdccb..fa8f3dfc7170315e2158c189eec1f95bbf62de98 100644 (file)
--- a/ext/dom/characterdata.c
+++ b/ext/dom/characterdata.c
@@ -41,7 +41,7 @@ int dom_characterdata_data_read(dom_object *obj, zval *retval)
        xmlChar *content;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -61,7 +61,7 @@ int dom_characterdata_data_write(dom_object *obj, zval *newval)
        zend_string *str;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -90,7 +90,7 @@ int dom_characterdata_length_read(dom_object *obj, zval *retval)
        long length = 0;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/document.c b/ext/dom/document.c
index 76fffa4168808d9d0bd65d358ff2f873535ef3e3..fd6563803e13bb1e82322aeca514c523890d9ba8 100644 (file)
--- a/ext/dom/document.c
+++ b/ext/dom/document.c
@@ -55,7 +55,7 @@ int dom_document_doctype_read(dom_object *obj, zval *retval)
        xmlDtdPtr dtdptr;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -95,7 +95,7 @@ int dom_document_document_element_read(dom_object *obj, zval *retval)
        xmlNode *root;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -121,7 +121,7 @@ int dom_document_encoding_read(dom_object *obj, zval *retval)
        char *encoding;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -143,7 +143,7 @@ zend_result dom_document_encoding_write(dom_object *obj, zval *newval)
        xmlCharEncodingHandlerPtr handler;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -183,7 +183,7 @@ int dom_document_standalone_read(dom_object *obj, zval *retval)
        docp = (xmlDocPtr) dom_object_get_node(obj);
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -197,7 +197,7 @@ int dom_document_standalone_write(dom_object *obj, zval *newval)
        zend_long standalone;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -220,7 +220,7 @@ int dom_document_version_read(dom_object *obj, zval *retval)
        char *version;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -241,7 +241,7 @@ int dom_document_version_write(dom_object *obj, zval *newval)
        zend_string *str;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -452,7 +452,7 @@ int dom_document_document_uri_read(dom_object *obj, zval *retval)
        char *url;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -472,7 +472,7 @@ int dom_document_document_uri_write(dom_object *obj, zval *newval)
        zend_string *str;
 
        if (docp == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/documenttype.c b/ext/dom/documenttype.c
index e910cf40097c67aee63768547f441f7c42ed1630..0046ac7f45388b2d9f73355f53710074e687755b 100644 (file)
--- a/ext/dom/documenttype.c
+++ b/ext/dom/documenttype.c
@@ -33,7 +33,7 @@ int dom_documenttype_name_read(dom_object *obj, zval *retval)
        xmlDtdPtr dtdptr = (xmlDtdPtr) dom_object_get_node(obj);
 
        if (dtdptr == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -56,7 +56,7 @@ int dom_documenttype_entities_read(dom_object *obj, zval *retval)
        dom_object *intern;
 
        if (doctypep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -84,7 +84,7 @@ int dom_documenttype_notations_read(dom_object *obj, zval *retval)
        dom_object *intern;
 
        if (doctypep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -110,7 +110,7 @@ int dom_documenttype_public_id_read(dom_object *obj, zval *retval)
        xmlDtdPtr dtdptr = (xmlDtdPtr) dom_object_get_node(obj);
 
        if (dtdptr == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -135,7 +135,7 @@ int dom_documenttype_system_id_read(dom_object *obj, zval *retval)
        xmlDtdPtr dtdptr = (xmlDtdPtr) dom_object_get_node(obj);
 
        if (dtdptr == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -160,7 +160,7 @@ int dom_documenttype_internal_subset_read(dom_object *obj, zval *retval)
        xmlDtdPtr intsubset;
 
        if (dtdptr == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/element.c b/ext/dom/element.c
index 8661641a9871ba23e3aa3806e99e974795d910f6..1d93aa3371f3701ec5f937e8da8abbff5baaa816 100644 (file)
--- a/ext/dom/element.c
+++ b/ext/dom/element.c
@@ -117,7 +117,7 @@ int dom_element_tag_name_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/entity.c b/ext/dom/entity.c
index 7ad43a1ee33f7d72433a58c4f03bf4293d780d16..53f2b20bd1f92194e44265c158bf44a1a873ad63 100644 (file)
--- a/ext/dom/entity.c
+++ b/ext/dom/entity.c
@@ -41,7 +41,7 @@ int dom_entity_public_id_read(dom_object *obj, zval *retval)
        xmlEntity *nodep = (xmlEntity *) dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -66,7 +66,7 @@ int dom_entity_system_id_read(dom_object *obj, zval *retval)
        xmlEntity *nodep = (xmlEntity *) dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -92,7 +92,7 @@ int dom_entity_notation_name_read(dom_object *obj, zval *retval)
        char *content;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/node.c b/ext/dom/node.c
index 34dc5c76d2ceb9bea264fba8344f4b2a28b830d9..253073598c32c75c7aeb6fcdf26c1ef77a071386 100644 (file)
--- a/ext/dom/node.c
+++ b/ext/dom/node.c
@@ -45,7 +45,7 @@ int dom_node_node_name_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -127,7 +127,7 @@ int dom_node_node_value_read(dom_object *obj, zval *retval)
        char *str = NULL;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -166,7 +166,7 @@ int dom_node_node_value_write(dom_object *obj, zval *newval)
        zend_string *str;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -212,7 +212,7 @@ int dom_node_node_type_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -240,7 +240,7 @@ int dom_node_parent_node_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -267,7 +267,7 @@ int dom_node_child_nodes_read(dom_object *obj, zval *retval)
        dom_object *intern;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -291,7 +291,7 @@ int dom_node_first_child_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -322,7 +322,7 @@ int dom_node_last_child_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -353,7 +353,7 @@ int dom_node_previous_sibling_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -381,7 +381,7 @@ int dom_node_next_sibling_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -409,7 +409,7 @@ int dom_node_previous_element_sibling_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -442,7 +442,7 @@ int dom_node_next_element_sibling_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -474,7 +474,7 @@ int dom_node_attributes_read(dom_object *obj, zval *retval)
        dom_object *intern;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -502,7 +502,7 @@ int dom_node_owner_document_read(dom_object *obj, zval *retval)
        xmlDocPtr docp;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -533,7 +533,7 @@ int dom_node_namespace_uri_read(dom_object *obj, zval *retval)
        char *str = NULL;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -573,7 +573,7 @@ int dom_node_prefix_read(dom_object *obj, zval *retval)
        char *str = NULL;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -611,7 +611,7 @@ int dom_node_prefix_write(dom_object *obj, zval *newval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -682,7 +682,7 @@ int dom_node_local_name_read(dom_object *obj, zval *retval)
        xmlNode *nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -708,7 +708,7 @@ int dom_node_base_uri_read(dom_object *obj, zval *retval)
        xmlChar *baseuri;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -736,7 +736,7 @@ int dom_node_text_content_read(dom_object *obj, zval *retval)
        char *str = NULL;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -758,7 +758,7 @@ int dom_node_text_content_write(dom_object *obj, zval *newval)
        zend_string *str;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/notation.c b/ext/dom/notation.c
index f1e0c1ba38894ca68dd897d4e4361de3d9d0248e..15148d9a6fca4bd5ed59c664e107d2d113faaf88 100644 (file)
--- a/ext/dom/notation.c
+++ b/ext/dom/notation.c
@@ -42,7 +42,7 @@ int dom_notation_public_id_read(dom_object *obj, zval *retval)
        xmlEntityPtr nodep = (xmlEntityPtr) dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -67,7 +67,7 @@ int dom_notation_system_id_read(dom_object *obj, zval *retval)
        xmlEntityPtr nodep = (xmlEntityPtr) dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/parentnode.c b/ext/dom/parentnode.c
index 375c692dcad85805b7fc5fd153feb988fa4b3614..25f79595a330ccc3f644118bae9894ba893fb21a 100644 (file)
--- a/ext/dom/parentnode.c
+++ b/ext/dom/parentnode.c
@@ -35,7 +35,7 @@ int dom_parent_node_first_element_child_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -68,7 +68,7 @@ int dom_parent_node_last_element_child_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -102,7 +102,7 @@ int dom_parent_node_child_element_count(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/processinginstruction.c b/ext/dom/processinginstruction.c
index 73367442eb1c7877ddc9ac8c386b25d6aeed2ae1..9c4e2083870944b6bdcfbf88d726a74f6c5292f7 100644 (file)
--- a/ext/dom/processinginstruction.c
+++ b/ext/dom/processinginstruction.c
@@ -75,7 +75,7 @@ int dom_processinginstruction_target_read(dom_object *obj, zval *retval)
        xmlNodePtr nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -99,7 +99,7 @@ int dom_processinginstruction_data_read(dom_object *obj, zval *retval)
        nodep = dom_object_get_node(obj);
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 
@@ -119,7 +119,7 @@ int dom_processinginstruction_data_write(dom_object *obj, zval *newval)
        zend_string *str;
 
        if (nodep == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }
 

diff --git a/ext/dom/tests/DOMCharacterData_data_error_002.phpt b/ext/dom/tests/DOMCharacterData_data_error_002.phpt
index 401e0f2f8cd4a834bc53a9ebec1d5a40c0958e40..a88ee509bfbff6d43999d9e225e9e1f436ba92c8 100644 (file)
--- a/ext/dom/tests/DOMCharacterData_data_error_002.phpt
+++ b/ext/dom/tests/DOMCharacterData_data_error_002.phpt
@@ -8,7 +8,11 @@ Eric Berg <ehberg@gmail.com>
 --FILE--
 <?php
 $character_data = new DOMCharacterData();
-print $character_data->data;
+try {
+    print $character_data->data;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMCharacterData_length_error_001.phpt b/ext/dom/tests/DOMCharacterData_length_error_001.phpt
index 34d5273be4c104126cebf2e0a9d0f6cc3cee24e7..79d936bdd0ac1769137ada5d5c9bc69a30861493 100644 (file)
--- a/ext/dom/tests/DOMCharacterData_length_error_001.phpt
+++ b/ext/dom/tests/DOMCharacterData_length_error_001.phpt
@@ -8,7 +8,11 @@ Jason Bouffard <jbouffard1@yahoo.com>
 --FILE--
 <?php
 $character_data = new DOMCharacterData();
-print $character_data->length;
+try {
+    print $character_data->length;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMDocumentType_entities_error_001.phpt b/ext/dom/tests/DOMDocumentType_entities_error_001.phpt
index 4d047a00580d737f6c4d6ea3751aeaff38630826..f93ae2138686e6f4d9c281da22a667b157a41c3a 100644 (file)
--- a/ext/dom/tests/DOMDocumentType_entities_error_001.phpt
+++ b/ext/dom/tests/DOMDocumentType_entities_error_001.phpt
@@ -8,7 +8,11 @@ Eric Lee Stewart <ericleestewart@gmail.com>
 --FILE--
 <?php
 $doctype = new DOMDocumentType();
-$entities = $doctype->entities;
+try {
+    $doctype->entities;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMDocumentType_internalSubset_error_001.phpt b/ext/dom/tests/DOMDocumentType_internalSubset_error_001.phpt
index 58259147778141fe73beacffe227622377eae0f3..b05712ce24d33ef0068703f1829b18d2f7f0969a 100644 (file)
--- a/ext/dom/tests/DOMDocumentType_internalSubset_error_001.phpt
+++ b/ext/dom/tests/DOMDocumentType_internalSubset_error_001.phpt
@@ -8,7 +8,11 @@ Eric Lee Stewart <ericleestewart@gmail.com>
 --FILE--
 <?php
 $doctype = new DOMDocumentType();
-$internalSubset = $doctype->internalSubset;
+try {
+    $doctype->internalSubset;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMDocumentType_name_error_001.phpt b/ext/dom/tests/DOMDocumentType_name_error_001.phpt
index 026ea2393af4378069925bb0ec0e96200d0cd851..7eda99dca5aab6de330813d1d11887732b46c0de 100644 (file)
--- a/ext/dom/tests/DOMDocumentType_name_error_001.phpt
+++ b/ext/dom/tests/DOMDocumentType_name_error_001.phpt
@@ -8,7 +8,11 @@ Eric Lee Stewart <ericleestewart@gmail.com>
 --FILE--
 <?php
 $doctype = new DOMDocumentType();
-$name = $doctype->name;
+try {
+    $doctype->name;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMDocumentType_notations_error_001.phpt b/ext/dom/tests/DOMDocumentType_notations_error_001.phpt
index a18f47f3f1c9a2e884aeeefd242f9cace6e55d4a..53b8c9c0d1b733d3b4354a70baf7b97cfa39ce55 100644 (file)
--- a/ext/dom/tests/DOMDocumentType_notations_error_001.phpt
+++ b/ext/dom/tests/DOMDocumentType_notations_error_001.phpt
@@ -8,7 +8,11 @@ Eric Lee Stewart <ericleestewart@gmail.com>
 --FILE--
 <?php
 $doctype = new DOMDocumentType();
-$notations = $doctype->notations;
+try {
+    $notations = $doctype->notations;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMDocumentType_publicId_error_001.phpt b/ext/dom/tests/DOMDocumentType_publicId_error_001.phpt
index df6c2bd908741c80e075daa02ea80a4c5b554193..c46f94faeb9499af9a378b621467e22e3e182493 100644 (file)
--- a/ext/dom/tests/DOMDocumentType_publicId_error_001.phpt
+++ b/ext/dom/tests/DOMDocumentType_publicId_error_001.phpt
@@ -8,7 +8,11 @@ Eric Lee Stewart <ericleestewart@gmail.com>
 --FILE--
 <?php
 $doctype = new DOMDocumentType();
-$publicId = $doctype->publicId;
+try {
+    $publicId = $doctype->publicId;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/tests/DOMDocumentType_systemId_error_001.phpt b/ext/dom/tests/DOMDocumentType_systemId_error_001.phpt
index e302fcc7f17f0cd504c6f980d847e8f3e78bafbb..a8dfe3bdfbc6bc14d154611ac193c149e71ff484 100644 (file)
--- a/ext/dom/tests/DOMDocumentType_systemId_error_001.phpt
+++ b/ext/dom/tests/DOMDocumentType_systemId_error_001.phpt
@@ -8,7 +8,11 @@ Eric Lee Stewart <ericleestewart@gmail.com>
 --FILE--
 <?php
 $doctype = new DOMDocumentType();
-$systemId = $doctype->systemId;
+try {
+    $systemId = $doctype->systemId;
+} catch (DOMException $exception) {
+    echo $exception->getMessage() . "\n";
+}
 ?>
---EXPECTF--
-Warning: main(): Invalid State Error in %s on line %d
+--EXPECT--
+Invalid State Error

diff --git a/ext/dom/text.c b/ext/dom/text.c
index 7a45be246360ef04e2e96894e1cbbcb3499ea5eb..c19d3e91946a0299d2e291e30106eeb575a2a07f 100644 (file)
--- a/ext/dom/text.c
+++ b/ext/dom/text.c
@@ -74,7 +74,7 @@ int dom_text_whole_text_read(dom_object *obj, zval *retval)
        node = dom_object_get_node(obj);
 
        if (node == NULL) {
-               php_dom_throw_error(INVALID_STATE_ERR, 0);
+               php_dom_throw_error(INVALID_STATE_ERR, 1);
                return FAILURE;
        }