jenkins-bot submitted this change.

View Change

Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[IMPR] cleanup PyCharm warnings

Change-Id: I83cd603f8eaed0b08c9186cbf5e3117abf6ef38d
---
M pywikibot/proofreadpage.py
1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/pywikibot/proofreadpage.py b/pywikibot/proofreadpage.py
index 561a968..3f6ba26 100644
--- a/pywikibot/proofreadpage.py
+++ b/pywikibot/proofreadpage.py
@@ -34,7 +34,7 @@
from requests.exceptions import ReadTimeout

try:
- from bs4 import BeautifulSoup, FeatureNotFound
+ from bs4 import BeautifulSoup
except ImportError as e:
BeautifulSoup = e

@@ -42,6 +42,7 @@
"""Raise BeautifulSoup when called, if bs4 is not available."""
raise BeautifulSoup
else:
+ from bs4 import FeatureNotFound
try:
BeautifulSoup('', 'lxml')
except FeatureNotFound:
@@ -231,7 +232,7 @@
left, sep, right = base.rpartition('.')
ext = right if sep else ''

- return (base, ext, num)
+ return base, ext, num

@property
def index(self):
@@ -610,7 +611,7 @@
pywikibot.warning('ReadTimeout %s: %s' % (cmd_uri, e))
except Exception as e:
pywikibot.error('"{}": {}'.format(cmd_uri, e))
- return (True, e)
+ return True, e
else:
pywikibot.debug('{}: {}'.format(ocr_tool, response.text),
_logger)
@@ -622,7 +623,7 @@
return True, ReadTimeout

if 400 <= response.status < 600:
- return (True, 'Http response status {}'.format(response.status))
+ return True, 'Http response status {}'.format(response.status)

data = json.loads(response.text)

@@ -639,9 +640,9 @@

if error:
pywikibot.error('OCR query %s: %s' % (cmd_uri, _text))
- return (error, _text)
+ return error, _text
else:
- return (error, parser_func(_text))
+ return error, parser_func(_text)

def _do_hocr(self):
"""Do hocr using https://phetools.toolforge.org/hocr_cgi.py?cmd=hocr.
@@ -682,7 +683,7 @@
except ValueError:
error_text = 'No prp-page-image src found for %s.' % self
pywikibot.error(error_text)
- return (True, error_text)
+ return True, error_text

try:
cmd_fmt = self._OCR_CMDS[ocr_tool]
@@ -806,7 +807,8 @@

self._cached = False

- def _parse_redlink(self, href):
+ @staticmethod
+ def _parse_redlink(href):
"""Parse page title when link in Index is a redlink."""
p_href = re.compile(
r'/w/index\.php\?title=(.+?)&action=edit&redlink=1')
@@ -1041,7 +1043,8 @@
raise KeyError('Page number ".../{}" not in range.'
.format(page_number))

- def _get_from_label(self, mapping_dict, label):
+ @staticmethod
+ def _get_from_label(mapping_dict, label):
"""Helper function to get info from label."""
# Convert label to string if an integer is passed.
if isinstance(label, int):

To view, visit change 634359. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I83cd603f8eaed0b08c9186cbf5e3117abf6ef38d
Gerrit-Change-Number: 634359
Gerrit-PatchSet: 1
Gerrit-Owner: Mpaa <mpaa.wiki@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged