summaryrefslogtreecommitdiff
path: root/addons/link_tracker/models/link_tracker.py
diff options
context:
space:
mode:
authorryv-odoo <ryv@odoo.com>2019-10-10 13:24:27 +0000
committerryv-odoo <ryv@odoo.com>2019-10-10 15:19:43 +0000
commite0ba61d9fb6f15c9df5d9cec4b5a6b602dc45899 (patch)
tree2f7b4a59b80aab53de5f41b043daa5e58a3f324a /addons/link_tracker/models/link_tracker.py
parent8c2f6b006fd7e92653706652d5d5eb279887b0fd (diff)
[REM] website_links: remove favicon, icon_src for performance
The link tracker model (link.tracker) contained favicon (store) and icon_src computed fields. Computing favicon took {3-6.5%} of installing odoo (with demo data) due to syncronous http call to a external link (google). Moreover, these fields had no sense to be store in odoo and only used in one template. This template was modified to display the same information (icon) as before but without storing fields and with a lazy loading of icons. TASK_ID: 2083715 closes odoo/odoo#38405 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Diffstat (limited to 'addons/link_tracker/models/link_tracker.py')
-rw-r--r--addons/link_tracker/models/link_tracker.py17
1 files changed, 0 insertions, 17 deletions
diff --git a/addons/link_tracker/models/link_tracker.py b/addons/link_tracker/models/link_tracker.py
index bf4d25fe465..e623b4e92f6 100644
--- a/addons/link_tracker/models/link_tracker.py
+++ b/addons/link_tracker/models/link_tracker.py
@@ -44,8 +44,6 @@ class LinkTracker(models.Model):
redirected_url = fields.Char(string='Redirected URL', compute='_compute_redirected_url')
short_url_host = fields.Char(string='Host of the short URL', compute='_compute_short_url_host')
title = fields.Char(string='Page Title', store=True)
- favicon = fields.Char(string='Favicon', compute='_compute_favicon', store=True)
- icon_src = fields.Char(string='Favicon Source', compute='_compute_icon_src')
# Tracking
link_code_ids = fields.One2many('link.tracker.code', 'link_id', string='Codes')
code = fields.Char(string='Short URL code', compute='_compute_code')
@@ -72,11 +70,6 @@ class LinkTracker(models.Model):
record = self.env['link.tracker.code'].search([('link_id', '=', tracker.id)], limit=1, order='id DESC')
tracker.code = record.code
- @api.depends('favicon')
- def _compute_icon_src(self):
- for tracker in self:
- tracker.icon_src = 'data:image/png;base64,' + tracker.favicon
-
@api.depends('url')
def _compute_redirected_url(self):
for tracker in self:
@@ -101,16 +94,6 @@ class LinkTracker(models.Model):
return title
- @api.depends('url')
- def _compute_favicon(self):
- for tracker in self:
- try:
- icon = requests.get('http://www.google.com/s2/favicons', params={'domain': tracker.url}, timeout=5).content
- icon_base64 = base64.b64encode(icon).replace(b"\n", b"").decode('ascii')
- except:
- icon_base64 = 'iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAACXBIWXMAAAsSAAALEgHS3X78AAACiElEQVQ4EaVTzU8TURCf2tJuS7tQtlRb6UKBIkQwkRRSEzkQgyEc6lkOKgcOph78Y+CgjXjDs2i44FXY9AMTlQRUELZapVlouy3d7kKtb0Zr0MSLTvL2zb75eL838xtTvV6H/xELBptMJojeXLCXyobnyog4YhzXYvmCFi6qVSfaeRdXdrfaU1areV5KykmX06rcvzumjY/1ggkR3Jh+bNf1mr8v1D5bLuvR3qDgFbvbBJYIrE1mCIoCrKxsHuzK+Rzvsi29+6DEbTZz9unijEYI8ObBgXOzlcrx9OAlXyDYKUCzwwrDQx1wVDGg089Dt+gR3mxmhcUnaWeoxwMbm/vzDFzmDEKMMNhquRqduT1KwXiGt0vre6iSeAUHNDE0d26NBtAXY9BACQyjFusKuL2Ry+IPb/Y9ZglwuVscdHaknUChqLF/O4jn3V5dP4mhgRJgwSYm+gV0Oi3XrvYB30yvhGa7BS70eGFHPoTJyQHhMK+F0ZesRVVznvXw5Ixv7/C10moEo6OZXbWvlFAF9FVZDOqEABUMRIkMd8GnLwVWg9/RkJF9sA4oDfYQAuzzjqzwvnaRUFxn/X2ZlmGLXAE7AL52B4xHgqAUqrC1nSNuoJkQtLkdqReszz/9aRvq90NOKdOS1nch8TpL555WDp49f3uAMXhACRjD5j4ykuCtf5PP7Fm1b0DIsl/VHGezzP1KwOiZQobFF9YyjSRYQETRENSlVzI8iK9mWlzckpSSCQHVALmN9Az1euDho9Xo8vKGd2rqooA8yBcrwHgCqYR0kMkWci08t/R+W4ljDCanWTg9TJGwGNaNk3vYZ7VUdeKsYJGFNkfSzjXNrSX20s4/h6kB81/271ghG17l+rPTAAAAAElFTkSuQmCC'
- tracker.favicon = icon_base64
-
@api.model
def create(self, vals):
create_vals = vals.copy()