Tpt has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/92284


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(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ProofreadPage 
refs/changes/84/92284/1

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: newchange
Gerrit-Change-Id: Icdcd033e8b2d7e57e969bd3a540c772f9dafc5bb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ProofreadPage
Gerrit-Branch: master
Gerrit-Owner: Tpt <[email protected]>

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

Reply via email to