using mb_eregi_replace instead of preg_replace

civicrm · Dec 5, 2018 · 9a048e2 · 9a048e2
1 parent 354ee85
commit 9a048e2
Showing 1 changed file with 10 additions and 12 deletions.
diff --git a/CRM/Utils/PDF/Utils.php b/CRM/Utils/PDF/Utils.php
@@ -106,20 +106,18 @@ public static function html2pdf(&$text, $fileName = 'civicrm.pdf', $output = FAL
 
     // Strip <html>, <header>, and <body> tags from each page
     $htmlElementstoStrip = array(
-      '@<head[^>]*?>.*?</head>@siu',
-      '@<script[^>]*?>.*?</script>@siu',
-      '@<body>@siu',
-      '@</body>@siu',
-      '@<html[^>]*?>@siu',
-      '@</html>@siu',
-      '@<!DOCTYPE[^>]*?>@siu',
+      '<head[^>]*?>.*?</head>',
+      '<script[^>]*?>.*?</script>',
+      '<body>',
+      '</body>',
+      '<html[^>]*?>',
+      '</html>',
+      '<!DOCTYPE[^>]*?>',
     );
-    $htmlElementsInstead = array('', '', '', '', '', '');
     foreach ($pages as & $page) {
-      $page = preg_replace($htmlElementstoStrip,
-        $htmlElementsInstead,
-        $page
-      );
+      foreach ($htmlElementstoStrip as $pattern){
+        $page = mb_eregi_replace($pattern, '', $page);
+      }
     }
     // Glue the pages together
     $html .= implode("\n<div style=\"page-break-after: always\"></div>\n", $pages);