Dalba has uploaded a new change for review.

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

Change subject: solve_disambiguation.py: do not suggest duplicate alternatives
......................................................................

solve_disambiguation.py: do not suggest duplicate alternatives

Bug: T138667
Change-Id: I137adebe9a8941db40e84a9785073693a6e3d487
---
M scripts/solve_disambiguation.py
1 file changed, 12 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/77/299277/1

diff --git a/scripts/solve_disambiguation.py b/scripts/solve_disambiguation.py
index 92d8938..039106b 100755
--- a/scripts/solve_disambiguation.py
+++ b/scripts/solve_disambiguation.py
@@ -651,9 +651,16 @@
         return None
 
     def makeAlternativesUnique(self):
-        """Remove duplicate entries stable."""
-        unique = set(self.alternatives)
-        self.alternatives = [alt for alt in self.alternatives if alt in unique]
+        """Remove duplicate items from self.alternatives.
+
+        @note: preserve the order of self.alternatives.
+        @rtype: None
+
+        """
+        seen = set()
+        self.alternatives = [
+            i for i in self.alternatives if i not in seen and not seen.add(i)
+        ]
 
     def listAlternatives(self):
         """Show a list of alternatives."""
@@ -954,7 +961,7 @@
     def findAlternatives(self, disambPage):
         """Extend self.alternatives using correctcap of disambPage.linkedPages.
 
-        @param disambPage: the disabiguation page
+        @param disambPage: the disambiguation page
         @type disambPage: pywikibot.Page
         @return: True if everything goes fine, False otherwise
         @rtype: bool
@@ -1002,6 +1009,7 @@
                         u"The specified page is not a redirect. Skipping.")
                     return False
         elif self.getAlternatives:
+            # not disambPage.isRedirectPage() or self.primary
             try:
                 if self.primary:
                     try:

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I137adebe9a8941db40e84a9785073693a6e3d487
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Dalba <dalba.w...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to