[IMP] website: fallback on odoo social by default in all cases

addons/website/controllers/main.py

@@ -334,9 +334,21 @@ def website_configurator(self, step=1, **kwargs):
             action_url += '&step=' + str(step)
         return request.redirect(action_url)
 
+    def _get_social_default_urls(self):
+        """ Return the default URLs for social networks. """
+        return {
+            'facebook': 'https://www.facebook.com/Odoo',
+            'github': 'https://github.com/odoo',
+            'linkedin': 'https://www.linkedin.com/company/odoo',
+            'youtube': 'https://www.youtube.com/user/OpenERPonline',
+            'instagram': 'https://www.instagram.com/odoo.official',
+            'twitter': 'https://twitter.com/Odoo',
+            'tiktok': 'https://www.tiktok.com/@odoo',
+        }
+
     @http.route(['/website/social/<string:social>'], type='http', auth="public", website=True, sitemap=False)
     def social(self, social, **kwargs):
-        url = getattr(request.website, 'social_%s' % social, False)
+        url = getattr(request.website, 'social_%s' % social, False) or self._get_social_default_urls().get(social)
         if not url:
             raise werkzeug.exceptions.NotFound()
         return request.redirect(url, local=False)

addons/website/tests/test_website_favicon.py

@@ -1,14 +1,17 @@
 # -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
+# TODO in master: rename this file to `test_website` only
+
 from PIL import Image
 
 from odoo.tests import tagged
-from odoo.tests.common import TransactionCase
+from odoo.tests.common import HttpCase, TransactionCase
 from odoo.tools import base64_to_image, image_to_base64
 
 
 @tagged('post_install', '-at_install')
+# TODO: rename the class name, it's a bad copy paste
 class TestWebsiteResetPassword(TransactionCase):
 
     def test_01_website_favicon(self):
@@ -34,3 +37,18 @@ def test_01_website_favicon(self):
         self.assertEqual(image.format, 'ICO')
         self.assertEqual(image.size, (256, 256))
         self.assertEqual(image.getpixel((0, 0)), bg_color)
+
+
+@tagged('-at_install', 'post_install')
+class TestWebsiteHttp(HttpCase):
+    def test_website_default_social_media(self):
+        # Demo data are setting the twitter account of the main company, we need
+        # to remove it for demo data to not interfere with the test
+        self.env.ref('base.main_company').social_twitter = None
+        website = self.env['website'].create({
+            'name': 'Test Website',
+        })
+        self.assertFalse(website.social_twitter)
+        r = self.url_open('/website/social/twitter', allow_redirects=False)
+        self.assertEqual(r.status_code, 303, "Should redirect to Odoo network")
+        self.assertEqual(r.headers.get('Location'), 'https://twitter.com/Odoo', "Should redirect to Odoo network (2)")