* #89 fix check_format function * #89 fix check_format function - fix default return
This commit is contained in:
committed by
Tom Kralidis
parent
44d347a09f
commit
75e0a599dc
+17
-6
@@ -642,17 +642,28 @@ def check_format(args, headers):
|
||||
:returns: format value
|
||||
"""
|
||||
|
||||
# Optional f=html or f=json query param
|
||||
# overrides accept
|
||||
format_ = args.get('f')
|
||||
if format_:
|
||||
return format_
|
||||
|
||||
# Format not specified: get from accept headers
|
||||
# format_ = 'text/html'
|
||||
headers_ = None
|
||||
if 'accept' in headers.keys():
|
||||
format_ = headers['accept']
|
||||
headers_ = headers['accept']
|
||||
elif 'Accept' in headers.keys():
|
||||
format_ = headers['Accept']
|
||||
headers_ = headers['Accept']
|
||||
|
||||
if format_ == 'text/html':
|
||||
format_ = 'html'
|
||||
elif format_ == 'application/json':
|
||||
format_ = 'json'
|
||||
format_ = None
|
||||
if headers_:
|
||||
headers_ = headers_.split(',')
|
||||
|
||||
if 'text/html' in headers_:
|
||||
format_ = 'html'
|
||||
elif 'application/json' in headers_:
|
||||
format_ = 'json'
|
||||
|
||||
return format_
|
||||
|
||||
|
||||
Reference in New Issue
Block a user