jenkins-bot has submitted this change and it was merged.
Change subject: Replace getImagePageHtml() with fileIsShared()
......................................................................
Replace getImagePageHtml() with fileIsShared()
ImagePage.getImagePageHtml() was used in the original 2007 checkimages
script to detect if a local page exists for the same name as a file on
the shared repository. ImagePage now has a cheaper method fileIsShared()
which achieves the same goal.
Likewise getImagePageHtml() was used in the original 2007 unusedfiles
script to detect if the rendered page contained:
<table id="mw_metadata" class="mw_metadata">
which indicated that it had an image linked to it. Using fileUrl()
and fileIsShared() achieves the same goal.
Change-Id: I5fd3f34b9081fdbb20b853052fe1c4c30e360d93
---
M scripts/checkimages.py
M scripts/unusedfiles.py
2 files changed, 5 insertions(+), 11 deletions(-)
Approvals:
Ladsgroup: Looks good to me, but someone else must approve
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/checkimages.py b/scripts/checkimages.py
index 5c63a33..41b0aac 100644
--- a/scripts/checkimages.py
+++ b/scripts/checkimages.py
@@ -912,9 +912,7 @@
return True # continue with the check-part
pywikibot.output(u'%s is on commons!' % self.imageName)
- on_commons_text = self.image.getImagePageHtml()
- if re.search(r"\<div
class\=(?:'|\")sharedUploadNotice(?:'|\")\>",
- on_commons_text):
+ if self.image.fileIsShared():
pywikibot.output(
u"But, the file doesn't exist on your project!
Skip...")
# We have to skip the check part for that image because
diff --git a/scripts/unusedfiles.py b/scripts/unusedfiles.py
index a4c7fd9..7307368 100644
--- a/scripts/unusedfiles.py
+++ b/scripts/unusedfiles.py
@@ -42,10 +42,6 @@
'fa': u'\n\n{{جا:اخطار به کاربر برای تصاویر بدون
استفاده|%(title)s}}--~~~~',
'it': u'\n\n{{Utente:Filbot/Immagine orfana}}',
}
-except_text = {
- 'en': u'<table id="mw_metadata"
class="mw_metadata">',
- 'it': u'<table id="mw_metadata"
class="mw_metadata">',
-}
class UnusedFilesBot(Bot):
@@ -63,9 +59,8 @@
template_to_the_image)
template_user = i18n.translate(self.site,
template_to_the_user)
- except_text_translated = i18n.translate(self.site, except_text)
summary = i18n.translate(self.site, comment, fallback=True)
- if not all([template_image, template_user, except_text_translated, comment]):
+ if not all([template_image, template_user, comment]):
raise pywikibot.Error(u'This script is not localized for %s site.'
% self.site)
self.summary = summary
@@ -76,8 +71,9 @@
pywikibot.output(u"File '%s' does not exist (see bug
69133)."
% image.title())
continue
- if (except_text_translated.encode('utf-8')
- not in image.getImagePageHtml() and
+ # Use fileUrl() and fileIsShared() to confirm it is local media
+ # rather than a local page with the same name as shared media.
+ if (image.fileUrl() and not image.fileIsShared() and
u'http://' not in image.text):
if template_image in image.text:
pywikibot.output(u"%s done already"
--
To view, visit
https://gerrit.wikimedia.org/r/153293
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5fd3f34b9081fdbb20b853052fe1c4c30e360d93
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: John Vandenberg <jayvdb(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: Ricordisamoa <ricordisamoa(a)openmailbox.org>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>