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