Skip to content

Commit 50412a6

Browse files
committed
Don't use multibyte regex functions
1 parent a7bebe7 commit 50412a6

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

src/PHPHtmlParser/Dom.php

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -359,8 +359,8 @@ protected function clean($str)
359359
}
360360

361361
// remove white space before closing tags
362-
$str = mb_eregi_replace("'\s+>", "'>", $str);
363-
$str = mb_eregi_replace('"\s+>', '">', $str);
362+
$str = preg_replace("#'\s+>#i", "'>", $str);
363+
$str = preg_replace('#"\s+>#i', '">', $str);
364364

365365
// clean out the \n\r
366366
$replace = ' ';
@@ -370,31 +370,31 @@ protected function clean($str)
370370
$str = str_replace(["\r\n", "\r", "\n"], $replace, $str);
371371

372372
// strip the doctype
373-
$str = mb_eregi_replace("<!doctype(.*?)>", '', $str);
373+
$str = preg_replace("#<!doctype(.*?)>#i", '', $str);
374374

375375
// strip out comments
376-
$str = mb_eregi_replace("<!--(.*?)-->", '', $str);
376+
$str = preg_replace("#<!--(.*?)-->#i", '', $str);
377377

378378
// strip out cdata
379-
$str = mb_eregi_replace("<!\[CDATA\[(.*?)\]\]>", '', $str);
379+
$str = preg_replace("#<!\[CDATA\[(.*?)\]\]>#i", '', $str);
380380

381381
// strip out <script> tags
382382
if ($this->options->get('removeScripts') == true) {
383-
$str = mb_eregi_replace("<\s*script[^>]*[^/]>(.*?)<\s*/\s*script\s*>", '', $str);
384-
$str = mb_eregi_replace("<\s*script\s*>(.*?)<\s*/\s*script\s*>", '', $str);
383+
$str = preg_replace("#<\s*script[^>]*[^/]>(.*?)<\s*/\s*script\s*>#i", '', $str);
384+
$str = preg_replace("#<\s*script\s*>(.*?)<\s*/\s*script\s*>#i", '', $str);
385385
}
386386

387387
// strip out <style> tags
388388
if ($this->options->get('removeStyles') == true) {
389-
$str = mb_eregi_replace("<\s*style[^>]*[^/]>(.*?)<\s*/\s*style\s*>", '', $str);
390-
$str = mb_eregi_replace("<\s*style\s*>(.*?)<\s*/\s*style\s*>", '', $str);
389+
$str = preg_replace("#<\s*style[^>]*[^/]>(.*?)<\s*/\s*style\s*>#", '', $str);
390+
$str = preg_replace("#<\s*style\s*>(.*?)<\s*/\s*style\s*>#", '', $str);
391391
}
392392

393393
// strip out server side scripts
394-
$str = mb_eregi_replace("(<\?)(.*?)(\?>)", '', $str);
394+
$str = preg_replace("#(<\?)(.*?)(\?>)#i", '', $str);
395395

396396
// strip smarty scripts
397-
$str = mb_eregi_replace("(\{\w)(.*?)(\})", '', $str);
397+
$str = preg_replace("#(\{\w)(.*?)(\})#i", '', $str);
398398

399399
return $str;
400400
}

src/PHPHtmlParser/Dom/TextNode.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ class TextNode extends LeafNode
3838
public function __construct($text)
3939
{
4040
// remove double spaces
41-
$text = mb_ereg_replace('\s+', ' ', $text);
41+
$text = preg_replace('#\s+#', ' ', $text);
4242

4343
// restore line breaks
4444
$text = str_replace('&#10;', "\n", $text);

0 commit comments

Comments
 (0)