diff --git a/app_common/models/base.py b/app_common/models/base.py index 14f8f7b6..45d8a647 100644 --- a/app_common/models/base.py +++ b/app_common/models/base.py @@ -110,13 +110,17 @@ class Base(models.AbstractModel): # Todo: mimetype filter image, file_name = get_image_url2attachment(url) if image and file_name: - attachment = self.env['ir.attachment'].create({ - 'datas': image, - 'name': file_name, - 'website_id': False, - }) - attachment.generate_access_token() - return attachment + try: + attachment = self.env['ir.attachment'].create({ + 'datas': image, + 'name': file_name, + 'website_id': False, + }) + attachment.generate_access_token() + return attachment + except Exception as e: + _logger.error('get_image_url2attachment error: %s' % str(e)) + return False else: return False @@ -124,20 +128,23 @@ class Base(models.AbstractModel): def get_image_base642attachment(self, data): image, file_name = get_image_base642attachment(data) if image and file_name: - attachment = self.env['ir.attachment'].create({ - 'datas': image, - 'name': file_name, - 'website_id': False, - }) - attachment.generate_access_token() - return attachment + try: + attachment = self.env['ir.attachment'].create({ + 'datas': image, + 'name': file_name, + 'website_id': False, + }) + attachment.generate_access_token() + return attachment + except Exception as e: + _logger.error('get_image_base642attachment error: %s' % str(e)) + return False else: return False def get_ua_type(self): return get_ua_type() - def get_image_from_url(url): if not url: return None