Skip to content
Snippets Groups Projects
Commit 626669f9 authored by Julien (jvoisin) Voisin's avatar Julien (jvoisin) Voisin
Browse files

Add some typing to epub.py

parent 497f5f71
No related branches found
No related tags found
No related merge requests found
...@@ -2,6 +2,7 @@ import logging ...@@ -2,6 +2,7 @@ import logging
import re import re
import uuid import uuid
import xml.etree.ElementTree as ET # type: ignore import xml.etree.ElementTree as ET # type: ignore
from typing import Dict, Any
from . import archive, office from . import archive, office
...@@ -38,7 +39,7 @@ class EPUBParser(archive.ZipParser): ...@@ -38,7 +39,7 @@ class EPUBParser(archive.ZipParser):
except (TypeError, UnicodeDecodeError): except (TypeError, UnicodeDecodeError):
return {file_path: 'harmful content', } return {file_path: 'harmful content', }
def _specific_cleanup(self, full_path: str): def _specific_cleanup(self, full_path: str) -> bool:
if full_path.endswith('hmh.opf') or full_path.endswith('content.opf'): if full_path.endswith('hmh.opf') or full_path.endswith('content.opf'):
return self.__handle_contentopf(full_path) return self.__handle_contentopf(full_path)
elif full_path.endswith('OEBPS/toc.ncx'): elif full_path.endswith('OEBPS/toc.ncx'):
...@@ -47,7 +48,7 @@ class EPUBParser(archive.ZipParser): ...@@ -47,7 +48,7 @@ class EPUBParser(archive.ZipParser):
return self.__handle_ops_xml(full_path) return self.__handle_ops_xml(full_path)
return True return True
def __handle_ops_xml(self, full_path: str): def __handle_ops_xml(self, full_path: str) -> bool:
try: try:
tree, namespace = office._parse_xml(full_path) tree, namespace = office._parse_xml(full_path)
except ET.ParseError: # pragma: nocover except ET.ParseError: # pragma: nocover
...@@ -63,7 +64,7 @@ class EPUBParser(archive.ZipParser): ...@@ -63,7 +64,7 @@ class EPUBParser(archive.ZipParser):
return True return True
def __handle_tocncx(self, full_path: str): def __handle_tocncx(self, full_path: str) -> bool:
try: try:
tree, namespace = office._parse_xml(full_path) tree, namespace = office._parse_xml(full_path)
except ET.ParseError: # pragma: nocover except ET.ParseError: # pragma: nocover
...@@ -79,7 +80,7 @@ class EPUBParser(archive.ZipParser): ...@@ -79,7 +80,7 @@ class EPUBParser(archive.ZipParser):
short_empty_elements=False) short_empty_elements=False)
return True return True
def __handle_contentopf(self, full_path: str): def __handle_contentopf(self, full_path: str) -> bool:
try: try:
tree, namespace = office._parse_xml(full_path) tree, namespace = office._parse_xml(full_path)
except ET.ParseError: except ET.ParseError:
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment