jenkins-bot merged this change.

View Change

Approvals: Dvorapa: Looks good to me, but someone else must approve Zhuyifei1999: Looks good to me, but someone else must approve Dalba: Looks good to me, approved jenkins-bot: Verified
[bugfix] Set default current_page on top

current_page must be set on each treat call
but due to its magic it must be set only once.
Therefore use a local variable.

Bug: T189635
Change-Id: If963578b2a39219e52208d9f62f3287b3643ab26
---
M scripts/category.py
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/category.py b/scripts/category.py
index bedd4c9..1aade2f 100755
--- a/scripts/category.py
+++ b/scripts/category.py
@@ -349,17 +349,17 @@
# loading it will throw an error if we don't jump out before
return
else:
+ current_page = page
if page.namespace() == page.site.namespaces.TEMPLATE:
docs = page.site.doc_subpage # return tuple
for doc in docs:
doc_page = pywikibot.Page(
page.site, page.title() + doc)
if doc_page.exists():
- self.current_page = doc_page
+ current_page = doc_page
includeonly = ['includeonly']
break
- if not self.current_page:
- self.current_page = page
+ self.current_page = current_page
if self.current_page.exists():
# Load the page
text = self.current_page.text

To view, visit change 420170. To unsubscribe, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If963578b2a39219e52208d9f62f3287b3643ab26
Gerrit-Change-Number: 420170
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Dalba <dalba.wiki@gmail.com>
Gerrit-Reviewer: Dvorapa <dvorapa@seznam.cz>
Gerrit-Reviewer: John Vandenberg <jayvdb@gmail.com>
Gerrit-Reviewer: Matěj Suchánek <matejsuchanek97@gmail.com>
Gerrit-Reviewer: Mpaa <mpaa.wiki@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: Zhuyifei1999 <zhuyifei1999@gmail.com>
Gerrit-Reviewer: Zoranzoki21 <zorandori4444@gmail.com>
Gerrit-Reviewer: jenkins-bot <>