diff options
Diffstat (limited to 'ext/dom/attr.c')
-rw-r--r-- | ext/dom/attr.c | 83 |
1 files changed, 27 insertions, 56 deletions
diff --git a/ext/dom/attr.c b/ext/dom/attr.c index d45ca8820a..7cef0230a4 100644 --- a/ext/dom/attr.c +++ b/ext/dom/attr.c @@ -1,8 +1,8 @@ /* +----------------------------------------------------------------------+ - | PHP Version 5 | + | PHP Version 7 | +----------------------------------------------------------------------+ - | Copyright (c) 1997-2013 The PHP Group | + | Copyright (c) 1997-2014 The PHP Group | +----------------------------------------------------------------------+ | This source file is subject to version 3.01 of the PHP license, | | that is bundled with this package in the file LICENSE, and is | @@ -55,13 +55,12 @@ const zend_function_entry php_dom_attr_class_functions[] = { /* {{{ proto void DOMAttr::__construct(string name, [string value]); */ PHP_METHOD(domattr, __construct) { - zval *id; xmlAttrPtr nodep = NULL; xmlNodePtr oldnode = NULL; dom_object *intern; char *name, *value = NULL; - int name_len, value_len, name_valid; + size_t name_len, value_len, name_valid; zend_error_handling error_handling; zend_replace_error_handling(EH_THROW, dom_domexception_class_entry, &error_handling TSRMLS_CC); @@ -71,7 +70,7 @@ PHP_METHOD(domattr, __construct) } zend_restore_error_handling(&error_handling TSRMLS_CC); - intern = (dom_object *)zend_object_store_get_object(id TSRMLS_CC); + intern = Z_DOMOBJ_P(id); name_valid = xmlValidateName((xmlChar *) name, 0); if (name_valid != 0) { @@ -79,20 +78,18 @@ PHP_METHOD(domattr, __construct) RETURN_FALSE; } - nodep = xmlNewProp(NULL, (xmlChar *) name, value); + nodep = xmlNewProp(NULL, (xmlChar *) name, (xmlChar *) value); if (!nodep) { php_dom_throw_error(INVALID_STATE_ERR, 1 TSRMLS_CC); RETURN_FALSE; } - if (intern != NULL) { - oldnode = dom_object_get_node(intern); - if (oldnode != NULL) { - php_libxml_node_free_resource(oldnode TSRMLS_CC); - } - php_libxml_increment_node_ptr((php_libxml_node_object *)intern, (xmlNodePtr)nodep, (void *)intern TSRMLS_CC); + oldnode = dom_object_get_node(intern); + if (oldnode != NULL) { + php_libxml_node_free_resource(oldnode TSRMLS_CC); } + php_libxml_increment_node_ptr((php_libxml_node_object *)intern, (xmlNodePtr)nodep, (void *)intern TSRMLS_CC); } /* }}} end DOMAttr::__construct */ @@ -102,7 +99,7 @@ readonly=yes URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#ID-1112119403 Since: */ -int dom_attr_name_read(dom_object *obj, zval **retval TSRMLS_DC) +int dom_attr_name_read(dom_object *obj, zval *retval TSRMLS_DC) { xmlAttrPtr attrp; @@ -113,8 +110,7 @@ int dom_attr_name_read(dom_object *obj, zval **retval TSRMLS_DC) return FAILURE; } - ALLOC_ZVAL(*retval); - ZVAL_STRING(*retval, (char *) (attrp->name), 1); + ZVAL_STRING(retval, (char *) attrp->name); return SUCCESS; } @@ -126,11 +122,10 @@ readonly=yes URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#ID-862529273 Since: */ -int dom_attr_specified_read(dom_object *obj, zval **retval TSRMLS_DC) +int dom_attr_specified_read(dom_object *obj, zval *retval TSRMLS_DC) { /* TODO */ - ALLOC_ZVAL(*retval); - ZVAL_TRUE(*retval); + ZVAL_TRUE(retval); return SUCCESS; } @@ -141,26 +136,21 @@ readonly=no URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#ID-221662474 Since: */ -int dom_attr_value_read(dom_object *obj, zval **retval TSRMLS_DC) +int dom_attr_value_read(dom_object *obj, zval *retval TSRMLS_DC) { - xmlAttrPtr attrp; + xmlAttrPtr attrp = (xmlAttrPtr) dom_object_get_node(obj); xmlChar *content; - attrp = (xmlAttrPtr) dom_object_get_node(obj); - if (attrp == NULL) { php_dom_throw_error(INVALID_STATE_ERR, 0 TSRMLS_CC); return FAILURE; } - ALLOC_ZVAL(*retval); - - if ((content = xmlNodeGetContent((xmlNodePtr) attrp)) != NULL) { - ZVAL_STRING(*retval, content, 1); + ZVAL_STRING(retval, (char *) content); xmlFree(content); } else { - ZVAL_EMPTY_STRING(*retval); + ZVAL_EMPTY_STRING(retval); } return SUCCESS; @@ -169,10 +159,8 @@ int dom_attr_value_read(dom_object *obj, zval **retval TSRMLS_DC) int dom_attr_value_write(dom_object *obj, zval *newval TSRMLS_DC) { - zval value_copy; - xmlAttrPtr attrp; - - attrp = (xmlAttrPtr) dom_object_get_node(obj); + zend_string *str; + xmlAttrPtr attrp = (xmlAttrPtr) dom_object_get_node(obj); if (attrp == NULL) { php_dom_throw_error(INVALID_STATE_ERR, 0 TSRMLS_CC); @@ -183,21 +171,11 @@ int dom_attr_value_write(dom_object *obj, zval *newval TSRMLS_DC) node_list_unlink(attrp->children TSRMLS_CC); } - if (newval->type != IS_STRING) { - if(Z_REFCOUNT_P(newval) > 1) { - value_copy = *newval; - zval_copy_ctor(&value_copy); - newval = &value_copy; - } - convert_to_string(newval); - } - - xmlNodeSetContentLen((xmlNodePtr) attrp, Z_STRVAL_P(newval), Z_STRLEN_P(newval) + 1); + str = zval_get_string(newval); - if (newval == &value_copy) { - zval_dtor(newval); - } + xmlNodeSetContentLen((xmlNodePtr) attrp, (xmlChar *) str->val, str->len + 1); + zend_string_release(str); return SUCCESS; } @@ -208,10 +186,9 @@ readonly=yes URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#Attr-ownerElement Since: DOM Level 2 */ -int dom_attr_owner_element_read(dom_object *obj, zval **retval TSRMLS_DC) +int dom_attr_owner_element_read(dom_object *obj, zval *retval TSRMLS_DC) { xmlNodePtr nodep, nodeparent; - int ret; nodep = dom_object_get_node(obj); @@ -220,18 +197,13 @@ int dom_attr_owner_element_read(dom_object *obj, zval **retval TSRMLS_DC) return FAILURE; } - ALLOC_ZVAL(*retval); - nodeparent = nodep->parent; if (!nodeparent) { - ZVAL_NULL(*retval); + ZVAL_NULL(retval); return SUCCESS; } - if (NULL == (*retval = php_dom_create_object(nodeparent, &ret, *retval, obj TSRMLS_CC))) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Cannot create required DOM object"); - return FAILURE; - } + php_dom_create_object(nodeparent, retval, obj TSRMLS_CC); return SUCCESS; } @@ -243,11 +215,10 @@ readonly=yes URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#Attr-schemaTypeInfo Since: DOM Level 3 */ -int dom_attr_schema_type_info_read(dom_object *obj, zval **retval TSRMLS_DC) +int dom_attr_schema_type_info_read(dom_object *obj, zval *retval TSRMLS_DC) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Not yet implemented"); - ALLOC_ZVAL(*retval); - ZVAL_NULL(*retval); + ZVAL_NULL(retval); return SUCCESS; } |