From 373d59e866a2a2841dac5f6e155343c7093facfd Mon Sep 17 00:00:00 2001
From: mcm-odoo <mcm@odoo.com>
Date: Mon, 19 Aug 2019 07:33:37 +0000
Subject: [PATCH] [IMP] website_slides: add a toggle button to display/hide
 leaderboard

Before this commit, leaderboard was displayed only if user was not connected.
As we feel this is very interesting to show we decided to display it in all
cases (either connected or not) on eLearning frontpage. Activating it is
now done using an entry in the customize menu.

Task 2005981

closes odoo/odoo#35791

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
---
 addons/website_slides/controllers/main.py            |  9 +++------
 .../views/website_slides_templates_homepage.xml      | 12 ++++++++++--
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/addons/website_slides/controllers/main.py b/addons/website_slides/controllers/main.py
index 08d24df5c68..f74728ba065 100644
--- a/addons/website_slides/controllers/main.py
+++ b/addons/website_slides/controllers/main.py
@@ -264,12 +264,9 @@ class WebsiteSlides(WebsiteProfile):
             ]).mapped('challenge_id')
 
         # fetch 'heroes of the week' for non logged people
-        if request.env.user._is_public():
-            users = request.env['res.users'].sudo().search([
-                ('karma', '>', 0),
-                ('website_published', '=', True)], limit=5, order='karma desc')
-        else:
-            users = None
+        users = request.env['res.users'].sudo().search([
+            ('karma', '>', 0),
+            ('website_published', '=', True)], limit=5, order='karma desc')
 
         values = self._prepare_user_values(**post)
         values.update({
diff --git a/addons/website_slides/views/website_slides_templates_homepage.xml b/addons/website_slides/views/website_slides_templates_homepage.xml
index d326044dd42..af2f8bf9346 100644
--- a/addons/website_slides/views/website_slides_templates_homepage.xml
+++ b/addons/website_slides/views/website_slides_templates_homepage.xml
@@ -116,6 +116,9 @@
                                     <div class="col-12 col-sm-6 col-lg-12 d-flex align-items-center">
                                         <t t-call="website_slides.slides_home_user_achievements_small"/>
                                     </div>
+                                    <div class="col-12 col-sm-6 col-lg-12 pl-md-5 pl-lg-0 mt-lg-4">
+                                        <t t-call="website_slides.slides_home_users_small"/>
+                                    </div>
                                 </div>
                             </div>
                         </div>
@@ -365,8 +368,13 @@
     </div>
 </template>
 
-<template id='slides_home_users_small' name="Latest Users">
+<template id='slides_home_users_small' name="Users">
     <div class="o_wslides_home_aside">
+    </div>
+</template>
+
+<template id="toggle_leaderboard" inherit_id="website_slides.slides_home_users_small" active="True" customize_show="True" name='Display Leaderboard'>
+    <xpath expr="//div[hasclass('o_wslides_home_aside')]" position="inside">
         <div class="row o_wslides_home_aside_title">
             <div class="col">
                 <a href="/profile/users" class="float-right">View all</a>
@@ -386,7 +394,7 @@
             </t>
             <t t-else=""><p class="col mt8">No leaderboard currently :(</p></t>
         </div>
-    </div>
+    </xpath>
 </template>
 
 <template id='user_quickkarma_card' name="User QuickKarma Card">