rrichards Sat Jan 6 18:03:43 2007 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/xmlwriter php_xmlwriter.c
/php-src/ext/xmlwriter/tests bug39504.phpt
Log:
MFH: parameter entity argument (pe) should be bool
update test
http://cvs.php.net/viewvc.cgi/php-src/ext/xmlwriter/php_xmlwriter.c?r1=1.20.2.12.2.11&r2=1.20.2.12.2.12&diff_format=u
Index: php-src/ext/xmlwriter/php_xmlwriter.c
diff -u php-src/ext/xmlwriter/php_xmlwriter.c:1.20.2.12.2.11
php-src/ext/xmlwriter/php_xmlwriter.c:1.20.2.12.2.12
--- php-src/ext/xmlwriter/php_xmlwriter.c:1.20.2.12.2.11 Sat Jan 6
15:53:32 2007
+++ php-src/ext/xmlwriter/php_xmlwriter.c Sat Jan 6 18:03:43 2007
@@ -17,7 +17,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: php_xmlwriter.c,v 1.20.2.12.2.11 2007/01/06 15:53:32 bjori Exp $ */
+/* $Id: php_xmlwriter.c,v 1.20.2.12.2.12 2007/01/06 18:03:43 rrichards Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -1347,13 +1347,14 @@
int name_len, content_len, retval;
/* Optional parameters */
char *pubid = NULL, *sysid = NULL, *ndataid = NULL;
- int pe = 0, pubid_len, sysid_len, ndataid_len;
+ zend_bool pe = 0;
+ int pubid_len, sysid_len, ndataid_len;
#ifdef ZEND_ENGINE_2
zval *this = getThis();
if (this) {
- if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|lsss",
+ if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|bsss",
&name, &name_len, &content, &content_len, &pe, &pubid,
&pubid_len,
&sysid, &sysid_len, &ndataid, &ndataid_len) == FAILURE)
{
return;
@@ -1362,7 +1363,7 @@
} else
#endif
{
- if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC,
"rss|lsss", &pind,
+ if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC,
"rss|bsss", &pind,
&name, &name_len, &content, &content_len, &pe, &pubid,
&pubid_len,
&sysid, &sysid_len, &ndataid, &ndataid_len) == FAILURE)
{
return;
http://cvs.php.net/viewvc.cgi/php-src/ext/xmlwriter/tests/bug39504.phpt?r1=1.2.2.2&r2=1.2.2.3&diff_format=u
Index: php-src/ext/xmlwriter/tests/bug39504.phpt
diff -u php-src/ext/xmlwriter/tests/bug39504.phpt:1.2.2.2
php-src/ext/xmlwriter/tests/bug39504.phpt:1.2.2.3
--- php-src/ext/xmlwriter/tests/bug39504.phpt:1.2.2.2 Sat Jan 6 15:53:33 2007
+++ php-src/ext/xmlwriter/tests/bug39504.phpt Sat Jan 6 18:03:43 2007
@@ -11,8 +11,22 @@
xmlwriter_start_element($xw, "root");
xmlwriter_end_document($xw);
print xmlwriter_flush($xw, true);
+print "\n";
+
+$xw = new XMLWriter();
+$xw->openMemory();
+$xw->startDocument(NULL, "UTF-8");
+$xw->startDtd("root");
+$xw->writeDtdEntity("c", NULL, 0, "-//W3C//TEXT copyright//EN",
"http://www.w3.org/xmlspec/copyright.xml";);
+$xw->endDtd();
+$xw->startElement("root");
+$xw->endDocument();
+print $xw->flush(true);
?>
--EXPECTF--
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE root [<!ENTITY ent2 "val2">]><root/>
+
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE root [<!ENTITY c PUBLIC "-//W3C//TEXT copyright//EN"
"http://www.w3.org/xmlspec/copyright.xml";>]><root/>
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
