Merge "exception: Avoid preg_replace for literal swap"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Thu, 27 Sep 2018 19:45:41 +0000 (19:45 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 27 Sep 2018 19:45:41 +0000 (19:45 +0000)
includes/exception/MWException.php
includes/exception/MWExceptionRenderer.php

index 652a87d..ffeafff 100644 (file)
@@ -87,7 +87,9 @@ class MWException extends Exception {
                        $res = wfMsgReplaceArgs( $fallback, $args );
                        // If an exception happens inside message rendering,
                        // {{SITENAME}} sometimes won't be replaced.
-                       $res = preg_replace( '/\{\{SITENAME\}\}/', $wgSitename, $res );
+                       $res = strtr( $res, [
+                               '{{SITENAME}}' => $wgSitename,
+                       ] );
                }
                return $res;
        }
index 1f1cabe..5036383 100644 (file)
@@ -205,7 +205,9 @@ class MWExceptionRenderer {
                        $res = wfMsgReplaceArgs( $fallback, $args );
                        // If an exception happens inside message rendering,
                        // {{SITENAME}} sometimes won't be replaced.
-                       $res = preg_replace( '/\{\{SITENAME\}\}/', $wgSitename, $res );
+                       $res = strtr( $res, [
+                               '{{SITENAME}}' => $wgSitename,
+                       ] );
                }
                return $res;
        }