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

diff --git a/includes/parser/ParserOptions.php b/includes/parser/ParserOptions.php
index fd826a2..25c2aa4 100644 (file)
--- 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.
         *