jenkins-bot has submitted this change. ( 
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/559492 )

Change subject: [IMPR] make showStatus function a bot method and rename it to 
show_status
......................................................................

[IMPR] make showStatus function a bot method and rename it to show_status

Patch detached from Iff53f3818

Change-Id: I9ae0a1a5e47cf9ff940c90b9e32501c7d9346d16
---
M scripts/welcome.py
1 file changed, 26 insertions(+), 25 deletions(-)

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



diff --git a/scripts/welcome.py b/scripts/welcome.py
index e9b0271..355f2c2 100755
--- a/scripts/welcome.py
+++ b/scripts/welcome.py
@@ -557,7 +557,7 @@
                                  .format(self.site))
                 list_loaded = load_word_function(badword_page.get())
             else:
-                showStatus(4)
+                self.show_status(4)
                 pywikibot.output("The bad word page doesn't exist!")
             self._blacklist = elenco + elenco_others + list_loaded
             del elenco, elenco_others, list_loaded
@@ -574,10 +574,10 @@
                                      .format(self.site))
                     list_white = load_word_function(whitelist_page.get())
                 else:
-                    showStatus(4)
+                    self.show_status(4)
                     pywikibot.output("The whitelist's page doesn't exist!")
             else:
-                showStatus(4)
+                self.show_status(4)
                 pywikibot.warning("The whitelist hasn't been set!")

             # Join the whitelist words.
@@ -614,7 +614,7 @@
                 answer = 'y'

             if answer.lower() in ['yes', 'y'] or not globalvar.confirm:
-                showStatus()
+                self.show_status()
                 pywikibot.output(
                     '{} is possibly an unwanted username. It will be reported.'
                     .format(name))
@@ -656,7 +656,7 @@
             if rep_text != '':
                 rep_page.put(text_get + rep_text, summary=com, force=True,
                              minor=True)
-                showStatus(5)
+                self.show_status(5)
                 pywikibot.output('Reported')
             self.BAQueue = []

@@ -677,7 +677,7 @@
             text = log_page.get()
         else:
             # make new log page
-            showStatus()
+            self.show_status()
             pywikibot.output(
                 'Log page is not exist, getting information for page creation')
             text = i18n.translate(self.site, logpage_header,
@@ -735,7 +735,7 @@
         if not globalvar.signFileName:
             sign_page_name = i18n.translate(self.site, random_sign)
             if not sign_page_name:
-                showStatus(4)
+                self.show_status(4)
                 pywikibot.output(
                     "{} doesn't allow random signature, force disable."
                     .format(self.site))
@@ -770,25 +770,25 @@
     def skip_page(self, user) -> bool:
         """Check whether the user is to be skipped."""
         if user.isBlocked():
-            showStatus(3)
+            self.show_status(3)
             pywikibot.output('{} has been blocked!'.format(user.username))

         elif 'bot' in user.groups():
-            showStatus(3)
+            self.show_status(3)
             pywikibot.output('{} is a bot!'.format(user.username))

         elif 'bot' in user.username.lower():
-            showStatus(3)
+            self.show_status(3)
             pywikibot.output('{} might be a global bot!'
                              .format(user.username))

         elif user.editCount() < globalvar.attachEditCount:
             if not user.editCount() == 0:
-                showStatus(1)
+                self.show_status(1)
                 pywikibot.output('{0} has only {1} contributions.'
                                  .format(user.username, user.editCount()))
             elif not globalvar.quiet:
-                showStatus(1)
+                self.show_status(1)
                 pywikibot.output('{} has no contributions.'
                                  .format(user.username))
         else:
@@ -804,12 +804,12 @@
                 if self.skip_page(user):
                     continue

-                showStatus(2)
+                self.show_status(2)
                 pywikibot.output('{} has enough edits to be welcomed.'
                                  .format(user.username))
                 ustp = user.getUserTalkPage()
                 if ustp.exists():
-                    showStatus(3)
+                    self.show_status(3)
                     pywikibot.output('{} has been already welcomed.'
                                      .format(user.username))
                     continue
@@ -838,7 +838,7 @@
                     # append welcomed, welcome_count++
                     ustp.put(welcome_text, welcome_comment, minor=False)
                 except pywikibot.EditConflict:
-                    showStatus(4)
+                    self.show_status(4)
                     pywikibot.output(
                         'An edit conflict has occurred, skipping this user.')
                 else:
@@ -846,7 +846,7 @@

                 welcomed_count = len(self.welcomed_users)
                 if globalvar.makeWelcomeLog:
-                    showStatus(5)
+                    self.show_status(5)
                     if welcomed_count == 0:
                         count = 'No users have'
                     elif welcomed_count == 1:
@@ -861,7 +861,7 @@
                             welcomed_count = 0

             if globalvar.makeWelcomeLog and welcomed_count > 0:
-                showStatus()
+                self.show_status()
                 if welcomed_count == 1:
                     pywikibot.output('Putting the log of the latest user...')
                 else:
@@ -872,12 +872,12 @@
                     continue
                 self.welcomed_users = []
             if hasattr(self, '_BAQueue'):
-                showStatus()
+                self.show_status()
                 pywikibot.output('Putting bad name to report page...')
                 self.reportBadAccount(None, final=True)
             try:
                 if globalvar.recursive:
-                    showStatus()
+                    self.show_status()
                     if locale.getlocale()[1]:
                         strfstr = time.strftime(
                             '%d %b %Y %H:%M:%S (UTC)', time.gmtime())
@@ -895,11 +895,12 @@
             except KeyboardInterrupt:
                 break

-
-def showStatus(n=0) -> None:
-    """Output colorized status."""
-    pywikibot.output(color_format('{color}[{0:5}]{default} ',
-                                  _MSGS[n], color=_COLORS[n]), newline=False)
+    @staticmethod
+    def show_status(n=0):
+        """Output colorized status."""
+        pywikibot.output(color_format('{color}[{0:5}]{default} ',
+                                      _MSGS[n], color=_COLORS[n]),
+                         newline=False)


 def load_word_function(raw) -> List[str]:
@@ -1022,7 +1023,7 @@
         bot.run()
     except KeyboardInterrupt:
         if bot.welcomed_users:
-            showStatus()
+            bot.show_status()
             pywikibot.output('Put welcomed users before quit...')
             bot.makelogpage(bot.welcomed_users)
         pywikibot.output('\nQuitting...')

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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I9ae0a1a5e47cf9ff940c90b9e32501c7d9346d16
Gerrit-Change-Number: 559492
Gerrit-PatchSet: 3
Gerrit-Owner: Xqt <i...@gno.de>
Gerrit-Reviewer: D3r1ck01 <xsavitar.w...@aol.com>
Gerrit-Reviewer: Xqt <i...@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-CC: Nullzero <sorawee.pw...@gmail.com>
Gerrit-MessageType: merged
_______________________________________________
Pywikibot-commits mailing list
Pywikibot-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/pywikibot-commits

Reply via email to