jenkins-bot has submitted this change and it was merged.
Change subject: pep257: adding docstrings to pagegenerators.py:
......................................................................
pep257: adding docstrings to pagegenerators.py:
Now pagegenerators.py does not show pep257 errors.
Fixed also lines > 80 chars.
Change-Id: I6ace68bfeceb1f6ddf5f44bb2efe48dbd7b7318d
---
M pywikibot/pagegenerators.py
1 file changed, 13 insertions(+), 4 deletions(-)
Approvals:
XZise: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/pagegenerators.py b/pywikibot/pagegenerators.py
index e00983a..3e77643 100644
--- a/pywikibot/pagegenerators.py
+++ b/pywikibot/pagegenerators.py
@@ -138,7 +138,8 @@
Argument can be given as "-unusedfiles:n" where
n is the maximum number of articles to work on.
--lonelypages Work on all articles that are not linked from any other article.
+-lonelypages Work on all articles that are not linked from any other
+ article.
Argument can be given as "-lonelypages:n" where
n is the maximum number of articles to work on.
@@ -298,7 +299,7 @@
# Insert Category: before category name to avoid parsing problems in
# Link.parse() when categoryname contains ":";
# Part before ":" might be interpreted as an interwiki prefix
- prefix = categoryname.split(":", 1)[0] # whole word if no
":" is present
+ prefix = categoryname.split(":", 1)[0] # whole word if ":"
not present
if prefix not in self.site.namespaces()[14]:
categoryname = u'{0}:{1}'.format(self.site.namespace(14),
categoryname)
@@ -514,7 +515,8 @@
gen = NewimagesPageGenerator(total=int(limit), site=self.site)
elif arg.startswith('-newpages'):
# partial workaround for bug 67249
- # to use -namespace/ns with -newpages, -ns must be given before -newpages
+ # to use -namespace/ns with -newpages, -ns must be given
+ # before -newpages
# otherwise default namespace is 0
namespaces = self.namespaces or 0
total = 60
@@ -537,7 +539,8 @@
mediawikiQuery = pywikibot.input(
u'What do you want to search for?')
# In order to be useful, all namespaces are required
- gen = SearchPageGenerator(mediawikiQuery, namespaces=[], site=self.site)
+ gen = SearchPageGenerator(mediawikiQuery,
+ namespaces=[], site=self.site)
elif arg.startswith('-google'):
gen = GoogleSearchPageGenerator(arg[8:])
elif arg.startswith('-titleregex'):
@@ -725,10 +728,12 @@
def FileLinksGenerator(referredFilePage, step=None, total=None, content=False):
+ """Yield Pages on which the file referredFilePage is
displayed."""
return referredFilePage.usingPages(step=step, total=total, content=content)
def ImagesPageGenerator(pageWithImages, step=None, total=None, content=False):
+ """Yield FilePages displayed on pageWithImages."""
return pageWithImages.imagelinks(step=step, total=total, content=content)
@@ -1055,6 +1060,7 @@
def CombinedPageGenerator(generators):
+ """Yield from each iterable until exhausted, then proceed with the
next."""
return itertools.chain(*generators)
@@ -1176,6 +1182,7 @@
def DequePreloadingGenerator(generator, step=50):
+ """Preload generator of type DequeGenerator."""
assert(isinstance(generator, DequeGenerator))
while True:
@@ -1617,6 +1624,7 @@
yield url
def __iter__(self):
+ """Iterate results."""
# restrict query to local site
localQuery = '%s site:%s' % (self.query, self.site.hostname())
base = 'http://%s%s' % (self.site.hostname(),
@@ -1682,6 +1690,7 @@
yield url
def __iter__(self):
+ """Iterate results."""
# restrict query to local site
localQuery = '%s site:%s' % (self.query, self.site.hostname())
base = 'http://%s%s' % (self.site.hostname(),
--
To view, visit
https://gerrit.wikimedia.org/r/171049
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6ace68bfeceb1f6ddf5f44bb2efe48dbd7b7318d
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Mpaa <mpaa.wiki(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: Nullzero <nullzero.free(a)gmail.com>
Gerrit-Reviewer: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: jenkins-bot <>