DrTrigon has submitted this change and it was merged.
Change subject: *solve error "ValueError: invalid literal for float(): N/A"
*minor docu changes and clean-up
......................................................................
*solve error "ValueError: invalid literal for float(): N/A"
*minor docu changes and clean-up
Change-Id: I8e9854c003380523fc54aa4aab6160ef2316b736
---
M catimages.py
1 file changed, 16 insertions(+), 3 deletions(-)
Approvals:
DrTrigon: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/catimages.py b/catimages.py
index 10b3376..acff497 100644
--- a/catimages.py
+++ b/catimages.py
@@ -2703,9 +2703,10 @@
result.append({ 'ID': int(s["index"]) + 1,
'Format': u'%s/%s' %
(s["codec_type"], s.get("codec_name",u'?')),
- 'Rate': rate or u'-',
+ 'Rate': rate or None,
'Dimensions': dim or (None, None),
- 'Duration': float(s['duration']),
+ 'Duration': None if (s['duration'].lower()
== 'n/a')
+ else float(s['duration']),
})
if 'image' in d["format"]["format_name"]:
@@ -3826,6 +3827,18 @@
return (tagged, logged)
def _make_infoblock(self, cat, res, tmpl_available=None):
+ """ Create infoblocks for pasting into wikitext from Templates
+ available on the wiki.
+
+ Nested values are flattened and numbered for output. Invalid or
+ unknown values can be marked e.g. by using u'-' or None.
+ Values like None, [] (empty list), ... that resolve by bool() to
+ False are hidden/omitted and not outputted at all. Unknown values
+ should be hidden to save space (make human readable) and be handled
+ by the Templates.
+ Unknown values that are NEEDED should be set to u'-' everything
+ else (not needed) to None, [] and so on.
+ """
if not res:
return u''
@@ -3852,7 +3865,7 @@
for item in res:
result.append( u" {{FileContentsByBot/%s" % cat )
for key in titles:
- if item[key]: # (work-a-round for empty 'Eyes')
+ if item[key]: # hide/omit (work-a-round for empty 'Eyes')
result.append( self._output_format_flatten(key, item[key]) )
result.append( u" }}" )
result.append( u"}}" )
--
To view, visit
https://gerrit.wikimedia.org/r/77638
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8e9854c003380523fc54aa4aab6160ef2316b736
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/compat
Gerrit-Branch: master
Gerrit-Owner: DrTrigon <dr.trigon(a)surfeu.ch>
Gerrit-Reviewer: DrTrigon <dr.trigon(a)surfeu.ch>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Legoktm <legoktm.wikipedia(a)gmail.com>
Gerrit-Reviewer: jenkins-bot