jenkins-bot has submitted this change and it was merged.

Change subject: Escape index entry name in regex
......................................................................


Escape index entry name in regex

Change-Id: Icdcd033e8b2d7e57e969bd3a540c772f9dafc5bb
---
M includes/index/ProofreadIndexPage.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Tpt: Looks good to me, approved
  Rtdwivedi: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/includes/index/ProofreadIndexPage.php 
b/includes/index/ProofreadIndexPage.php
index 47adf88..d722219 100644
--- a/includes/index/ProofreadIndexPage.php
+++ b/includes/index/ProofreadIndexPage.php
@@ -200,7 +200,7 @@
                $text = $this->getText();
                $values = array();
                foreach( $this->config as $varName => $property ) {
-                       $tagPattern = "/\n\|" . $varName . 
"=(.*?)\n(\||\}\})/is";
+                       $tagPattern = "/\n\|" . preg_quote( $varName, '/' ) . 
"=(.*?)\n(\||\}\})/is";
                        if ( preg_match( $tagPattern, $text, $matches ) ) {
                                $values[$varName] = $matches[1];
                        }

-- 
To view, visit https://gerrit.wikimedia.org/r/92284
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icdcd033e8b2d7e57e969bd3a540c772f9dafc5bb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ProofreadPage
Gerrit-Branch: master
Gerrit-Owner: Tpt <[email protected]>
Gerrit-Reviewer: Rtdwivedi <[email protected]>
Gerrit-Reviewer: Tpt <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to