jenkins-bot submitted this change.

View Change


Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[IMPR] Use re.sub instead of replaceExcept with empty exception list

Change-Id: Ieb1e540eba8f7e05f061f06322f6f013ad7ef662
---
M scripts/delinker.py
1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/scripts/delinker.py b/scripts/delinker.py
index 4d3d0b4..e17bdc5 100755
--- a/scripts/delinker.py
+++ b/scripts/delinker.py
@@ -46,7 +46,7 @@
SingleSiteBot,
calledModuleName,
)
-from pywikibot.textlib import case_escape, ignore_case, replaceExcept
+from pywikibot.textlib import case_escape, ignore_case


class CommonsDelinker(SingleSiteBot, ConfigParserBot, AutomaticTWSummaryBot):
@@ -119,7 +119,7 @@

def treat_page(self):
"""Delink a single page."""
- new = replaceExcept(self.current_page.text, self.image_regex, '', [])
+ new = re.sub(self.image_regex, '', self.current_page.text)
self.put_current(new)

def teardown(self):

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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ieb1e540eba8f7e05f061f06322f6f013ad7ef662
Gerrit-Change-Number: 958510
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: D3r1ck01 <dalangi-ctr@wikimedia.org>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged