Skip to content

Commit

Permalink
unoconv also handles cache_result late
Browse files Browse the repository at this point in the history
 * See previous commit. `cache_result` has the same issue as
  `file_required`.
  • Loading branch information
felliott committed Aug 23, 2016
1 parent 3304737 commit a4869fd
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 3 additions & 3 deletions mfr/core/extension.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,15 @@ def __init__(self, metadata, file_path, url, assets_url, export_url):
'url': self.url,
'export_url': self.export_url,
'file_path': self.file_path,
'cache_result': self.cache_result,
# 'error': 'error_t',
# 'elapsed': 'elpased_t',
})

# unoconv gets file_required from its subrenderer, which is constructed at the end of
# __init__.
# unoconv gets file_required and cache_result from its subrenderer, which is constructed
# at the end of __init__
try:
self.renderer_metrics.add('file_required', self.file_required)
self.renderer_metrics.add('cache_result', self.cache_result)
except AttributeError:
pass

Expand Down
2 changes: 2 additions & 0 deletions mfr/extensions/unoconv/render.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ def __init__(self, metadata, file_path, url, assets_url, export_url):

# can't call file_required until renderer is built
self.renderer_metrics.add('file_required', self.file_required)
self.renderer_metrics.add('cache_result', self.cache_result)

self.metrics.merge({
'map': {
'renderer': self.map['renderer'],
Expand Down

0 comments on commit a4869fd

Please sign in to comment.