From a37ab7f4c46fdade705f9745c11bfb714cd3717e Mon Sep 17 00:00:00 2001 From: Kunal Mehta Date: Wed, 14 Sep 2016 11:12:14 -0700 Subject: [PATCH] ParserOptions: Remove setters for magic links options This will allow clients to assume that the global configuration state ($wgEnableMagicLinks) is the same as the state in ParserOptions, fixing some consistency issues in wfEscapeWikiText() and Scribunto. The abstraction for the ParserOptions getters is still useful because it keeps the global state out of the Parser. Change-Id: Ic2c447443963eb336c786cacf9dfd041a2df3b77 --- includes/parser/ParserOptions.php | 21 --------------------- 1 file changed, 21 deletions(-) diff --git a/includes/parser/ParserOptions.php b/includes/parser/ParserOptions.php index fd826a204c..25c2aa45bc 100644 --- a/includes/parser/ParserOptions.php +++ b/includes/parser/ParserOptions.php @@ -595,27 +595,6 @@ class ParserOptions { return wfSetVar( $this->mIsPrintable, $x ); } - /** - * @since 1.28 - */ - public function setMagicISBNLinks( $x ) { - return wfSetVar( $this->mMagicISBNLinks, $x ); - } - - /** - * @since 1.28 - */ - public function setMagicPMIDLinks( $x ) { - return wfSetVar( $this->mMagicPMIDLinks, $x ); - } - - /** - * @since 1.28 - */ - public function setMagicRFCLinks( $x ) { - return wfSetVar( $this->mMagicRFCLinks, $x ); - } - /** * Set the redirect target. * -- 2.20.1