using mb_eregi_replace instead of preg_replace

civicrm · Mar 1, 2019 · fe276fa · fe276fa
1 parent 3efecc6
commit fe276fa
Showing 1 changed file with 11 additions and 12 deletions.
diff --git a/CRM/Utils/PDF/Utils.php b/CRM/Utils/PDF/Utils.php
@@ -95,21 +95,20 @@ public static function html2pdf($text, $fileName = 'civicrm.pdf', $output = FALS
     <div id=\"crm-container\">\n";
 
     // Strip <html>, <header>, and <body> tags from each page
+
     $htmlElementstoStrip = [
-      '@<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 = ['', '', '', '', '', ''];
     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);