jenkins-bot has submitted this change. ( 
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/1055471?usp=email )

Change subject: [IMPR] remove -wiktionary option from interwiki.py script
......................................................................

[IMPR] remove -wiktionary option from interwiki.py script

wiktionary is supported by wikidata. There is  no reason to keep this
option any longer.

Change-Id: Ibf9de85b4e3aa96055ad02adc0f0abad75b8325c
---
M scripts/interwiki.py
1 file changed, 18 insertions(+), 38 deletions(-)

Approvals:
  jenkins-bot: Verified
  Xqt: Looks good to me, approved




diff --git a/scripts/interwiki.py b/scripts/interwiki.py
index 4592eb9..81a8359 100755
--- a/scripts/interwiki.py
+++ b/scripts/interwiki.py
@@ -1,6 +1,5 @@
 #!/usr/bin/env python3
-"""
-Script to check language links for general pages.
+"""Script to check language links for general pages.

 Uses existing translations of a page, plus hints from the command line, to
 download the equivalent pages from other languages. All of such pages are
@@ -110,9 +109,8 @@
                    hints, the bot does nothing.
                    Hitting return without input on the "Which page to check:"
                    prompt has the same effect as using -hintsonly.
-                   Options like -back, -same or -wiktionary are in effect only
-                   after a page has been found to work on.
-                   (note: without ending colon)
+                   Options like -back or -same are in effect only after a page
+                   has been found to work on. (note: without ending colon)

 These arguments are useful to provide hints to the bot:

@@ -163,11 +161,6 @@
                    -same is equivalent to -hint:all:
                    (note: without ending colon)

-    -wiktionary:   similar to -same, but will ONLY accept names that are
-                   identical to the original. Also, if the title is not
-                   capitalized, it will only go through other wikis without
-                   automatic capitalization.
-
     -untranslated: works normally on pages with at least one interlanguage
                    link; asks for hints for pages that have none.

@@ -352,7 +345,12 @@
     titletranslate,
 )
 from pywikibot.backports import Iterable
-from pywikibot.bot import ListOption, OptionHandler, StandardOption
+from pywikibot.bot import (
+    ListOption,
+    OptionHandler,
+    StandardOption,
+    suggest_help,
+)
 from pywikibot.cosmetic_changes import moved_links
 from pywikibot.exceptions import (
     EditConflictError,
@@ -489,11 +487,6 @@
             R = re.compile(r'\[\[(.+?)(?:\]\]|\|)')
             with codecs.open(hintfilename, 'r', config.textfile_encoding) as f:
                 self.hints += R.findall(f.read())
-        elif arg == 'wiktionary':
-            self.same = 'wiktionary'
-            # Don't use auto-translation in -wiktionary mode
-            # where page titles must be the same
-            self.auto = False
         elif arg == 'untranslatedonly':
             self.untranslated = True
             self.untranslatedonly = True
@@ -818,8 +811,7 @@
     def skipPage(self, page, target, counter):
         """Return whether page has to be skipped."""
         return self.isIgnored(target) \
-            or self.namespaceMismatch(page, target, counter) \
-            or self.wiktionaryMismatch(target)
+            or self.namespaceMismatch(page, target, counter)

     @staticmethod
     def get_alternative(
@@ -903,24 +895,6 @@
         # or no origin page yet, also no problem
         return False

-    def wiktionaryMismatch(self, page) -> bool:
-        """Check for ignoring pages."""
-        if self.origin and self.conf.same == 'wiktionary':
-            if page.title().lower() != self.origin.title().lower():
-                pywikibot.info(f'NOTE: Ignoring {page} for {self.origin} in '
-                               f'wiktionary mode')
-                return True
-
-            if (page.title() != self.origin.title()
-                and self.origin.namespace().case == 'case-sensitive'
-                    and page.namespace().case == 'case-sensitive'):
-                pywikibot.info(
-                    f'NOTE: Ignoring {page} for {self.origin} in wiktionary'
-                    ' mode because both languages are uncapitalized.')
-                return True
-
-        return False
-
     def disambigMismatch(self, page, counter):
         """
         Check whether the given page has a different disambiguation status.
@@ -2287,6 +2261,7 @@
     optRestore = False
     append = True
     newPages = None
+    unknown = []

     # Process global args and prepare generator args parser
     local_args = pywikibot.handle_args(args)
@@ -2338,9 +2313,14 @@
             number = int(arg[8:])
         elif arg.startswith('-until:'):
             until = arg[7:]
-        else:
-            if not genFactory.handle_arg(arg) and not singlePageTitle:
+        elif not genFactory.handle_arg(arg):
+            if not (arg.startswith('-') or singlePageTitle):
                 singlePageTitle = arg
+            else:
+                unknown.append(arg)
+
+    if suggest_help(unknown_parameters=unknown):
+        return

     # Do not use additional summary with autonomous mode
     if iwconf.autonomous:

--
To view, visit 
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/1055471?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.wikimedia.org/r/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ibf9de85b4e3aa96055ad02adc0f0abad75b8325c
Gerrit-Change-Number: 1055471
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <i...@gno.de>
Gerrit-Reviewer: D3r1ck01 <dalangi-...@wikimedia.org>
Gerrit-Reviewer: Xqt <i...@gno.de>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
Pywikibot-commits mailing list -- pywikibot-commits@lists.wikimedia.org
To unsubscribe send an email to pywikibot-commits-le...@lists.wikimedia.org

Reply via email to