Browse Source

Don't expect a path for tidy_repair_string() and fix the tests

pull/271/head
Rasmus Lerdorf 15 years ago
parent
commit
fb2ce0bb00
  1. 2
      ext/tidy/tests/019.phpt
  2. 2
      ext/tidy/tests/022.phpt
  3. 10
      ext/tidy/tidy.c

2
ext/tidy/tests/019.phpt

@ -38,5 +38,5 @@ Warning: tidy_repair_file(): Filename cannot be empty in %s on line %d
Warning: tidy_repair_file(): Filename cannot be empty in %s on line %d
Warning: tidy_repair_file() expects parameter 1 to be string, array given in %s on line %d
Warning: tidy_repair_file() expects parameter 1 to be a valid path, array given in %s on line %d
Done

2
ext/tidy/tests/022.phpt

@ -38,5 +38,5 @@ Warning: tidy_repair_file(): Filename cannot be empty in %s on line %d
Warning: tidy_repair_file(): Filename cannot be empty in %s on line %d
Warning: tidy_repair_file() expects parameter 1 to be string, array given in %s on line %d
Warning: tidy_repair_file() expects parameter 1 to be a valid path, array given in %s on line %d
Done

10
ext/tidy/tidy.c

@ -558,15 +558,17 @@ static void php_tidy_quick_repair(INTERNAL_FUNCTION_PARAMETERS, zend_bool is_fil
TidyBuffer *errbuf;
zval **config = NULL;
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "p|Zsb", &arg1, &arg1_len, &config, &enc, &enc_len, &use_include_path) == FAILURE) {
RETURN_FALSE;
}
if (is_file) {
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "p|Zsb", &arg1, &arg1_len, &config, &enc, &enc_len, &use_include_path) == FAILURE) {
RETURN_FALSE;
}
if (!(data = php_tidy_file_to_mem(arg1, use_include_path, &data_len TSRMLS_CC))) {
RETURN_FALSE;
}
} else {
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|Zsb", &arg1, &arg1_len, &config, &enc, &enc_len, &use_include_path) == FAILURE) {
RETURN_FALSE;
}
data = arg1;
data_len = arg1_len;
}

Loading…
Cancel
Save