Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-09 03:04:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Check Book: Add a check for empty identifier elements

Browse Source
This commit is contained in:
Kovid Goyal 2017-03-30 06:02:01 +05:30
parent db8a4190d5
commit 14849c9005
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/calibre/ebooks/oeb/polish/check/opf.py
View File

@ -241,6 +241,14 @@ class NoUID(BaseError):
return True return True
class EmptyIdentifier(BaseError):
HELP = xml(_('The <dc:identifier> element must not be empty.'))
def __init__(self, name, lnum):
BaseError.__init__(self, _('Empty identifier element'), name, lnum)
class BadSpineMime(BaseError): class BadSpineMime(BaseError):
def __init__(self, name, iid, mt, lnum, opf_name): def __init__(self, name, iid, mt, lnum, opf_name):
@ -370,6 +378,9 @@ def check_opf(container):
uid = container.opf.get('unique-identifier', None) uid = container.opf.get('unique-identifier', None)
if uid is None or not container.opf_xpath('/opf:package/opf:metadata/dc:identifier[@id=%r]' % uid): if uid is None or not container.opf_xpath('/opf:package/opf:metadata/dc:identifier[@id=%r]' % uid):
errors.append(NoUID(container.opf_name)) errors.append(NoUID(container.opf_name))
for elem in container.opf_xpath('/opf:package/opf:metadata/dc:identifier'):
if not elem.text or not elem.text.strip():
errors.append(EmptyIdentifier(container.opf_name, elem.sourceline))
for item, name, linear in container.spine_iter: for item, name, linear in container.spine_iter:
mt = container.mime_map[name] mt = container.mime_map[name]