Mercurial > odfcast > odfcast
annotate odfcast/convert.py @ 47:e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
author | Björn Ricks <bjoern.ricks@intevation.de> |
---|---|
date | Tue, 28 Oct 2014 10:51:42 +0100 |
parents | 6d511e93a331 |
children | 64f3e3a28bd4 |
rev | line source |
---|---|
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
1 # -*- coding: utf-8 -*- |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
2 |
29
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
3 import logging |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
4 import tempfile |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
5 |
16
f4920ab1d27c
Add basic html interface for convert
Björn Ricks <bjoern.ricks@intevation.de>
parents:
4
diff
changeset
|
6 from flask import request, Response, json, render_template |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
7 from flask.views import MethodView |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
8 |
4
46f31348fe01
Implement a TemplateConvertView to allow convert odt templates with variable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
3
diff
changeset
|
9 from py3o.template import Template |
46f31348fe01
Implement a TemplateConvertView to allow convert odt templates with variable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
3
diff
changeset
|
10 |
19
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
11 from PyPDF2 import PdfFileMerger |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
12 |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
13 from werkzeug.utils import escape |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
14 |
29
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
15 log = logging.getLogger(__name__) |
19
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
16 |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
17 ALLOWED_FORMATS = ["pdf", "doc", "docx", "odt"] |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
18 |
19
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
19 PDF_MIMETYPE = "application/pdf" |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
20 JSON_MIMETYPE = "application/json" |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
21 HTML_MIMETYPE = "text/html" |
19
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
22 |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
23 MIMETYPES = { |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
24 "odt": "application/vnd.oasis.opendocument.text", |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
25 "doc": "application/msword", |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
26 "docx": "application/vnd.openxmlformats-officedocument" |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
27 ".wordprocessingml.document", |
19
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
28 "pdf": PDF_MIMETYPE, |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
29 } |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
30 |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
31 DEFAULT_MIMETYPE = "application/octet-stream" |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
32 |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
33 |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
34 class ErrorResponse(Response): |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
35 |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
36 BAD_REQUEST_ERROR_CODE = 400 |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
37 |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
38 def __init__(self, title, error_code, details, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
39 html_error_code=BAD_REQUEST_ERROR_CODE): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
40 data, mime_type = self.get_response_data(title, error_code, details) |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
41 super(ErrorResponse, self).__init__(response=data, mimetype=mime_type, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
42 status=html_error_code) |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
43 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
44 def json(self, title, error_code, details): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
45 return json.dumps({ |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
46 "error": title, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
47 "error_code": error_code, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
48 "details": details, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
49 }), JSON_MIMETYPE |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
50 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
51 def html(self, title, error_code, details): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
52 data = "" |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
53 return ( |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
54 u'<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">\n' |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
55 u'<title>%(code)s %(name)s</title>\n' |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
56 u'<h1>%(name)s</h1>\n' |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
57 u'%(details)s\n' |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
58 ) % { |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
59 "code": error_code, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
60 "name": escape(title), |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
61 "details": escape(details), |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
62 } |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
63 return data, HTML_MIMETYPE |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
64 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
65 def get_response_data(self, title, error_code, details): |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
66 if self.is_wants_json(): |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
67 return self.json(title, error_code, details) |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
68 return self.html(title, error_code, details) |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
69 |
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
70 def is_wants_json(self): |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
71 best = request.accept_mimetypes.best_match([JSON_MIMETYPE, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
72 HTML_MIMETYPE]) |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
73 return best == JSON_MIMETYPE and \ |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
74 request.accept_mimetypes[best] > \ |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
75 request.accept_mimetypes[HTML_MIMETYPE] |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
76 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
77 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
78 class TemplateErrorResponse(ErrorResponse): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
79 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
80 TEMPLATE_ERROR_CODE = 100 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
81 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
82 def __init__(self, details, error_code=TEMPLATE_ERROR_CODE): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
83 super(TemplateErrorResponse, self).__init__( |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
84 title="TemplateError", error_code=error_code, details=details, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
85 html_error_code=500) |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
86 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
87 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
88 class ConversionErrorResponse(ErrorResponse): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
89 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
90 CONVERSION_ERROR_CODE = 200 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
91 |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
92 def __init__(self, details, error_code=CONVERSION_ERROR_CODE): |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
93 super(TemplateErrorResponse, self).__init__( |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
94 title="ConversionError", error_code=error_code, details=details, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
95 html_error_code=500) |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
96 |
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
97 |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
98 class ConvertView(MethodView): |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
99 |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
100 def __init__(self, pyuno_driver_name="", hostname="localhost", port=2001): |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
101 driver_module = self._load_driver_module(pyuno_driver_name) |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
102 self.convertor = driver_module.Convertor(hostname, port) |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
103 |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
104 def _load_driver_module(self, pyuno_driver_name): |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
105 return __import__(pyuno_driver_name, globals(), locals(), |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
106 ["Convertor"]) |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
107 |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
108 def is_format_supported(self, fformat): |
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
109 return fformat and fformat.lower() in ALLOWED_FORMATS |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
110 |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
111 def post(self): |
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
112 ffile = request.files['file'] |
29
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
113 if not ffile.filename: |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
114 return ErrorResponse( |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
115 "Upload file missing", error_code=101, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
116 details="Please upload a file for conversion", |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
117 html_error_code=400) |
29
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
118 |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
119 fformat = request.form['format'] |
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
120 if not self.is_format_supported(fformat): |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
121 return ErrorResponse( |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
122 "Invalid format", error_code=102, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
123 details="Format %s not allowed" % fformat, |
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
124 html_error_code=400) |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
125 |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
126 datadict = self.get_datadict() |
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
127 |
29
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
128 mimetype = self.get_mimetype_for_format(fformat) |
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
129 |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
130 outfile = self.save_form_file(ffile) |
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
131 |
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
132 if datadict: |
31
83bca7dc9bfe
Return error response when a template couldn't be rendered
Björn Ricks <bjoern.ricks@intevation.de>
parents:
30
diff
changeset
|
133 try: |
83bca7dc9bfe
Return error response when a template couldn't be rendered
Björn Ricks <bjoern.ricks@intevation.de>
parents:
30
diff
changeset
|
134 tfile = tempfile.NamedTemporaryFile() |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
135 t = Template(outfile, tfile, ignore_undefined_variables=True) |
31
83bca7dc9bfe
Return error response when a template couldn't be rendered
Björn Ricks <bjoern.ricks@intevation.de>
parents:
30
diff
changeset
|
136 t.render(datadict) |
83bca7dc9bfe
Return error response when a template couldn't be rendered
Björn Ricks <bjoern.ricks@intevation.de>
parents:
30
diff
changeset
|
137 outfile.close() |
83bca7dc9bfe
Return error response when a template couldn't be rendered
Björn Ricks <bjoern.ricks@intevation.de>
parents:
30
diff
changeset
|
138 outfile = tfile |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
139 except Exception, e: |
41
a1100ec32be2
Log exception on rendering errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
34
diff
changeset
|
140 log.exception("Template error") |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
141 return TemplateErrorResponse(details=str(e)) |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
142 |
32
e6f2f1481de2
Fix we must check the fformat variable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
31
diff
changeset
|
143 if fformat != "odt": |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
144 try: |
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
145 outfile = self.convert(outfile, fformat) |
42
6d511e93a331
Return requested format when an error has occured
Björn Ricks <bjoern.ricks@intevation.de>
parents:
41
diff
changeset
|
146 except Exception, e: |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
147 log.exception("Conversion error") |
47
e99cbb47eafb
Improve error handling by returning flask/werkzeug responses directly on errors
Björn Ricks <bjoern.ricks@intevation.de>
parents:
42
diff
changeset
|
148 return ConversionErrorResponse(details=str(e)) |
29
1dadc59c4b9a
Make the conversion more error prone
Björn Ricks <bjoern.ricks@intevation.de>
parents:
22
diff
changeset
|
149 |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
150 return Response(outfile, mimetype=mimetype) |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
151 |
16
f4920ab1d27c
Add basic html interface for convert
Björn Ricks <bjoern.ricks@intevation.de>
parents:
4
diff
changeset
|
152 def get(self): |
f4920ab1d27c
Add basic html interface for convert
Björn Ricks <bjoern.ricks@intevation.de>
parents:
4
diff
changeset
|
153 return render_template("convert.html") |
f4920ab1d27c
Add basic html interface for convert
Björn Ricks <bjoern.ricks@intevation.de>
parents:
4
diff
changeset
|
154 |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
155 def save_form_file(self, infile): |
33
00e12392f8d4
Fix saving uploaded form file to a temporary file
Björn Ricks <bjoern.ricks@intevation.de>
parents:
32
diff
changeset
|
156 outfile = tempfile.NamedTemporaryFile() |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
157 infile.save(outfile) |
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
158 infile.close() |
33
00e12392f8d4
Fix saving uploaded form file to a temporary file
Björn Ricks <bjoern.ricks@intevation.de>
parents:
32
diff
changeset
|
159 outfile.seek(0) |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
160 return outfile |
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
161 |
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
162 def convert(self, infile, fformat): |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
163 outfile = tempfile.NamedTemporaryFile() |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
164 |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
165 self.convertor.convert(infile.name, outfile.name, fformat) |
0
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
166 |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
167 infile.close() |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
168 return outfile |
4a34f72f036b
Add initial conversion service
Björn Ricks <bjoern.ricks@intevation.de>
parents:
diff
changeset
|
169 |
3
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
170 def get_mimetype_for_format(self, fformat): |
15807d87930c
Refactor ConvertView to be easier reusable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
0
diff
changeset
|
171 return MIMETYPES.get(fformat, DEFAULT_MIMETYPE) |
4
46f31348fe01
Implement a TemplateConvertView to allow convert odt templates with variable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
3
diff
changeset
|
172 |
46f31348fe01
Implement a TemplateConvertView to allow convert odt templates with variable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
3
diff
changeset
|
173 def get_datadict(self): |
34
89ffa12bd48d
Allow to not passing any datadict to the conversion form
Björn Ricks <bjoern.ricks@intevation.de>
parents:
33
diff
changeset
|
174 vars = request.form.get('datadict') |
30
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
175 if not vars: |
9d65de2ebe22
Merge TemplateConvertView into ConvertView
Björn Ricks <bjoern.ricks@intevation.de>
parents:
29
diff
changeset
|
176 return None |
4
46f31348fe01
Implement a TemplateConvertView to allow convert odt templates with variable
Björn Ricks <bjoern.ricks@intevation.de>
parents:
3
diff
changeset
|
177 return json.loads(vars) |
18
585904e7411a
Add html for converting a odf with a template
Björn Ricks <bjoern.ricks@intevation.de>
parents:
16
diff
changeset
|
178 |
19
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
179 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
180 class MergeView(MethodView): |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
181 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
182 def get(self): |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
183 return render_template("merge.html") |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
184 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
185 def post(self): |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
186 merger = PdfFileMerger() |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
187 ffiles = request.files.getlist('files') |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
188 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
189 for ffile in ffiles: |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
190 merger.append(ffile) |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
191 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
192 outfile = tempfile.NamedTemporaryFile() |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
193 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
194 merger.write(outfile.name) |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
195 |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
196 merger.close() |
2f627039d2b4
Add a MergeView to merge pdfs based on PyPDF2
Björn Ricks <bjoern.ricks@intevation.de>
parents:
18
diff
changeset
|
197 return Response(outfile, mimetype=PDF_MIMETYPE) |
22
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
198 |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
199 |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
200 class TemplateView(MethodView): |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
201 |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
202 template_name = "" |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
203 |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
204 def __init__(self, template_name=None): |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
205 if template_name: |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
206 self.template_name = template_name |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
207 |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
208 def get_template_name(self): |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
209 return self.template_name |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
210 |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
211 def get(self): |
0e40926735e1
Add a generic View to display a template for a get request
Björn Ricks <bjoern.ricks@intevation.de>
parents:
19
diff
changeset
|
212 return render_template(self.get_template_name()) |