jenkins-bot submitted this change.

View Change


Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[bugfix] Fix analyzer in dataextend.py

There is no join method on list. Remove the bogus code.

Change-Id: I8cb3bde17926fc2398fa937924e8389d37cdec52
---
M scripts/dataextend.py
1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/scripts/dataextend.py b/scripts/dataextend.py
index 7f326bc..d3572e5 100644
--- a/scripts/dataextend.py
+++ b/scripts/dataextend.py
@@ -11005,10 +11005,8 @@
section = self.findbyre(r'(?s)class="aboutMeSection(.*?)</section>', html)
if section:
result += self.findallbyre(r'>([^<>]*)<', section)
- section = self.findlongtext(html)
- if section:
- lines = section.split('.')
- result += [lines[0], lines[:1].join(','), lines[:2].join(',')]
+ # this would also analyze self.findlongtext(html)
+ # but the code was removed
return result

def findlongtext(self, html: str):

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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I8cb3bde17926fc2398fa937924e8389d37cdec52
Gerrit-Change-Number: 879619
Gerrit-PatchSet: 2
Gerrit-Owner: Matěj Suchánek <matejsuchanek97@gmail.com>
Gerrit-Reviewer: Andre Engels <andreengels@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged