jenkins-bot has submitted this change and it was merged.

Change subject: Remove has_valid_template() test
......................................................................


Remove has_valid_template() test

Because l10n_tests.py is validating L10N templates,
this additional test is superfluous.

Change-Id: I1828360ce7520b6ac7e8c1a977890cdf8110117b
---
M scripts/redirect.py
1 file changed, 2 insertions(+), 24 deletions(-)

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



diff --git a/scripts/redirect.py b/scripts/redirect.py
index 67c4e84..f50e183 100755
--- a/scripts/redirect.py
+++ b/scripts/redirect.py
@@ -81,7 +81,6 @@
 __version__ = '$Id$'
 #
 
-import re
 import sys
 import datetime
 import pywikibot
@@ -391,27 +390,6 @@
         self.action = action
         self.generator = generator
         self.exiting = False
-        self._valid_template = None
-
-    def has_valid_template(self, twtitle):
-        """
-        Check whether a template from translatewiki.net exists on the wiki.
-
-        We assume we are always working on self.site
-
-        @param twtitle - a string which is the i18n key
-
-        """
-        if self._valid_template is None:
-            self._valid_template = False
-            if i18n.twhas_key(self.site, twtitle):
-                template_msg = i18n.twtranslate(self.site, twtitle)
-                template = re.findall(u'.*?{{(.*?)[|}]', template_msg)
-                if template:
-                    title = template[0]
-                    page = pywikibot.Page(self.site, title, ns=10)
-                    self._valid_template = page.exists()
-        return self._valid_template
 
     def delete_broken_redirects(self):
         # get reason for deletion text
@@ -497,8 +475,8 @@
                         assert targetPage.site == self.site, (
                             u'target page is on different site %s'
                             % targetPage.site)
-                        if self.has_valid_template(
-                                'redirect-broken-redirect-template'):
+                        if i18n.twhas_key(self.site,
+                                          'redirect-broken-redirect-template'):
                             pywikibot.output(u"No sysop in user-config.py, "
                                              u"put page to speedy deletion.")
                             content = redir_page.get(get_redirect=True)

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1828360ce7520b6ac7e8c1a977890cdf8110117b
Gerrit-PatchSet: 3
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Xqt <[email protected]>
Gerrit-Reviewer: John Vandenberg <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to