Remove trailing '.' for cached file with empty R/E

- R/E: renderer name and exporter name - `get_render_name()` and `get_exporter_name()` return '' when rendering/exporting a given type is not supported. Update `cache_file_path` for both to be removed of the trailing dot

Author: cslzchen

mfr/server/handlers/export.py

@@ -33,9 +33,12 @@ async def prepare(self):
 
  self.cache_file_id = '{}.{}'.format(self.metadata.unique_key, self.format)
 
- self.cache_file_path = await self.cache_provider.validate_path(
- '/export/{}.{}'.format(self.cache_file_id, self.exporter_name)
- )
+ if self.exporter_name:
+ cache_file_path_str = '/export/{}.{}'.format(self.cache_file_id, self.exporter_name)
+ else:
+ cache_file_path_str = '/export/{}'.format(self.cache_file_id)
+ self.cache_file_path = await self.cache_provider.validate_path(cache_file_path_str)
+
  self.source_file_path = await self.local_cache_provider.validate_path(
  '/export/{}'.format(self.source_file_id)
  )

mfr/server/handlers/render.py

@@ -27,9 +27,13 @@ async def prepare(self):
  self.renderer_name = utils.get_renderer_name(self.metadata.ext)
 
  self.cache_file_id = self.metadata.unique_key
- self.cache_file_path = await self.cache_provider.validate_path(
- '/render/{}.{}'.format(self.cache_file_id, self.renderer_name)
- )
+
+ if self.renderer_name:
+ cache_file_path_str = '/export/{}.{}'.format(self.cache_file_id, self.renderer_name)
+ else:
+ cache_file_path_str = '/export/{}'.format(self.cache_file_id)
+ self.cache_file_path = await self.cache_provider.validate_path(cache_file_path_str)
+
  self.source_file_path = await self.local_cache_provider.validate_path(
  '/render/{}'.format(self.source_file_id)
  )