John Vandenberg has uploaded a new change for review.

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

Change subject: Deprecate unused 'default' argument for magicwords
......................................................................

Deprecate unused 'default' argument for magicwords

compat has a 'default' argument for each of the methods which
return a magic word.  It is now unused and deprecated in compat.
Use @deprecated_args to also deprecate it in core.

Change-Id: Ia600aed3fc43d950e9c98851029c20fd895a4438
---
M pywikibot/site.py
1 file changed, 15 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/99/172499/1

diff --git a/pywikibot/site.py b/pywikibot/site.py
index ba1b00c..268ebbd 100644
--- a/pywikibot/site.py
+++ b/pywikibot/site.py
@@ -685,20 +685,18 @@
                                        old_name='normalizeNamespace',
                                        class_name='BaseSite')
 
-    def redirect(self, default=True):
-        """Return list of localized redirect tags for the site.
-
-        If default is True, falls back to 'REDIRECT' if the site has no
-        special redirect tag.
-
-        """
+    @deprecated_args(default=None)
+    def redirect(self):
+        """Return list of localized redirect tags for the site."""
         return [u"REDIRECT"]
 
-    def pagenamecodes(self, default=True):
+    @deprecated_args(default=None)
+    def pagenamecodes(self):
         """Return list of localized PAGENAME tags for the site."""
         return [u"PAGENAME"]
 
-    def pagename2codes(self, default=True):
+    @deprecated_args(default=None)
+    def pagename2codes(self):
         """Return list of localized PAGENAMEE tags for the site."""
         return [u"PAGENAMEE"]
 
@@ -1980,6 +1978,7 @@
             return pywikibot.Timestamp.fromtimestampformat(
                 self.expand_text("{{CURRENTTIMESTAMP}}"))
 
+    @need_version("1.14")
     def getmagicwords(self, word):
         """Return list of localized "word" magic words for the site."""
         if not hasattr(self, "_magicwords"):
@@ -1992,12 +1991,9 @@
         else:
             return [word]
 
-    def redirect(self, default=True):
-        """Return the preferred localized #REDIRECT keyword.
-
-        Argument is ignored (but maintained for backwards-compatibility).
-
-        """
+    @deprecated_args(default=None)
+    def redirect(self):
+        """Return the localized #REDIRECT keyword."""
         # return the magic word without the preceding '#' character
         return self.getmagicwords("redirect")[0].lstrip("#")
 
@@ -2018,11 +2014,13 @@
             pattern = None
         return BaseSite.redirectRegex(self, pattern)
 
-    def pagenamecodes(self, default=True):
+    @deprecated_args(default=None)
+    def pagenamecodes(self):
         """Return list of localized PAGENAME tags for the site."""
         return self.getmagicwords("pagename")
 
-    def pagename2codes(self, default=True):
+    @deprecated_args(default=None)
+    def pagename2codes(self):
         """Return list of localized PAGENAMEE tags for the site."""
         return self.getmagicwords("pagenamee")
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia600aed3fc43d950e9c98851029c20fd895a4438
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: John Vandenberg <[email protected]>

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

Reply via email to