From c764c38d7caf300dce7509e87d2bd268eee0161d Mon Sep 17 00:00:00 2001
From: Xavier-Do <xdo@odoo.com>
Date: Thu, 1 Sep 2022 13:55:17 +0000
Subject: [PATCH] [IMP] website: pregenerate frontend assets

closes odoo/odoo#99176

Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
---
 addons/web/tests/test_assets.py    |  2 +-
 addons/website/models/ir_qweb.py   | 50 ++++++++++++++++++++++++++++++
 odoo/addons/base/models/ir_qweb.py |  8 +++--
 3 files changed, 56 insertions(+), 4 deletions(-)

diff --git a/addons/web/tests/test_assets.py b/addons/web/tests/test_assets.py
index c3e1c8d8bf7..b277f82fb04 100644
--- a/addons/web/tests/test_assets.py
+++ b/addons/web/tests/test_assets.py
@@ -84,4 +84,4 @@ class TestLoad(HttpCase):
 
         with patch('odoo.addons.base.models.assetsbundle.AssetsBundle.save_attachment', save_attachment):
             self.url_open('/web').raise_for_status()
-            #self.url_open('/').raise_for_status() #currently breaking because of the website_id
+            self.url_open('/').raise_for_status()
diff --git a/addons/website/models/ir_qweb.py b/addons/website/models/ir_qweb.py
index 08c2fcf5a09..a109fb7f31f 100644
--- a/addons/website/models/ir_qweb.py
+++ b/addons/website/models/ir_qweb.py
@@ -1,6 +1,8 @@
 # -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 import re
+import logging
+
 from collections import OrderedDict
 
 from odoo import models
@@ -11,6 +13,8 @@ from odoo.addons.http_routing.models.ir_http import url_for
 from odoo.osv import expression
 from odoo.addons.website.models import ir_http
 
+
+_logger = logging.getLogger(__name__)
 re_background_image = re.compile(r"(background-image\s*:\s*url\(\s*['\"]?\s*)([^)'\"]+)")
 
 
@@ -149,3 +153,49 @@ class IrQWeb(models.AbstractModel):
             atts['style'] = re_background_image.sub(lambda m: '%s%s' % (m.group(1), website.get_cdn_url(m.group(2))), atts['style'])
 
         return atts
+
+    def _pregenerate_assets_bundles(self):
+        # website is adding a website_id to the extra part of the attachement url (/1)
+
+        # /web/assets/2223-ee56665/1/web.assets_common.min.css
+        # /web/assets/2224-47bce88/1/web.assets_frontend.min.css
+        # /web/assets/2225-8b64a2b/1/web.assets_common_minimal.min.js
+        # /web/assets/2226-17d3428/1/web.assets_frontend_minimal.min.js
+        # /web/assets/2227-b9cd4ba/1/web.assets_tests.min.js
+        # /web/assets/2228-7424684/1/web.assets_common_lazy.min.js
+        # /web/assets/2229-25b1d52/1/web.assets_frontend_lazy.min.js
+
+        # this means that the previously generated attachment wont be used on the website
+        # the main reason is to avoid invalidating other website attachement, but the
+        # version part combine with the initial extra (rtl) should be enough to ensure they are identical.
+        # we dont expect to have any pregenerated rtl/website attachment so we don't manage assets with extra
+
+        nodes = super()._pregenerate_assets_bundles()
+        website = self.env['website'].search([], order='id', limit=1)
+        if not website:
+            return nodes
+        nb_created = 0
+        for node in nodes:
+            bundle_info = node[1]
+            bundle_url = bundle_info.get('src', '') or bundle_info.get('href', '')
+            if bundle_url.startswith('/web/assets/'):
+                # example: "/web/assets/2152-ee56665/web.assets_common.min.js"
+                _, _, _, id_unique, name = bundle_url.split('/')
+                attachment_id, unique = id_unique.split('-')
+                url_pattern = f'/web/assets/%s-%s/{website.id}/{name}'
+                existing = self.env['ir.attachment'].search([('url', '=like', url_pattern % ('%', '%'))])
+                if existing:
+                    if f'-{unique}/' in existing.url:
+                        continue
+                    _logger.runbot(f'Updating exiting assets {existing.url} for website {website.id}')
+                    # we assume that most of the time the first website bundles will be the same as the base one
+                    # if the unique changes, it is most likely because sources where update since install.
+                    # this is mainly for dev downloading a database from runbot and trying to execute tests locally
+                    existing.unlink()
+                new = self.env['ir.attachment'].browse(int(attachment_id)).copy()
+                new.url = url_pattern % (new.id, unique)
+                nb_created += 1
+        if nb_created:
+            _logger.runbot('%s bundle(s) were copied for website %s', nb_created, website.id)
+
+        return nodes
diff --git a/odoo/addons/base/models/ir_qweb.py b/odoo/addons/base/models/ir_qweb.py
index 7d0e7f40636..896d7ba9f35 100644
--- a/odoo/addons/base/models/ir_qweb.py
+++ b/odoo/addons/base/models/ir_qweb.py
@@ -2533,15 +2533,17 @@ class IrQWeb(models.AbstractModel):
                     js_bundles.add(asset)
                 if css:
                     css_bundles.add(asset)
-
+        nodes = []
         start = time.time()
         for bundle in sorted(js_bundles):
-            self._generate_asset_nodes(bundle, css=False, js=True)
+            nodes += self._generate_asset_nodes(bundle, css=False, js=True)
         _logger.info('JS Assets bundles generated in %s seconds', time.time()-start)
         start = time.time()
         for bundle in sorted(css_bundles):
-            self._generate_asset_nodes(bundle, css=True, js=False)
+            nodes += self._generate_asset_nodes(bundle, css=True, js=False)
         _logger.info('CSS Assets bundles generated in %s seconds', time.time()-start)
+        return nodes
+
 
 def render(template_name, values, load, **options):
     """ Rendering of a qweb template without database and outside the registry.