From a4550cf1ec4d383715b54cf5eab6a4a3bfad9fe9 Mon Sep 17 00:00:00 2001
From: Paul Catinean <paulcatinean@gmail.com>
Date: Mon, 3 Oct 2016 19:24:20 +0300
Subject: [PATCH 1/4] [WIP] Adding filter menu to differentiate between Odoo
 CMS and OCA website_cms

---
 website_cms/__openerp__.py       |  1 +
 website_cms/models/__init__.py   |  1 +
 website_cms/models/res_config.py | 15 +++++++++++++++
 website_cms/models/website.py    | 13 +++++++++++++
 website_cms/templates/layout.xml | 13 +++++++++++++
 website_cms/templates/menu.xml   | 26 ++++++++++++++++++++++++++
 website_cms/views/res_config.xml | 20 ++++++++++++++++++++
 7 files changed, 89 insertions(+)
 create mode 100644 website_cms/models/res_config.py
 create mode 100644 website_cms/views/res_config.xml

diff --git a/website_cms/__openerp__.py b/website_cms/__openerp__.py
index 65423c9c..ee9aa4bd 100644
--- a/website_cms/__openerp__.py
+++ b/website_cms/__openerp__.py
@@ -30,6 +30,7 @@
         "views/cms_media.xml",
         "views/cms_media_category.xml",
         'views/website_menu.xml',
+        'views/res_config.xml',
         # templates
         "templates/assets.xml",
         "templates/misc.xml",
diff --git a/website_cms/models/__init__.py b/website_cms/models/__init__.py
index 37fc3adf..6d8f80dc 100644
--- a/website_cms/models/__init__.py
+++ b/website_cms/models/__init__.py
@@ -12,3 +12,4 @@
 from . import cms_page
 from . import cms_media
 from . import cms_tag
+from . import res_config
diff --git a/website_cms/models/res_config.py b/website_cms/models/res_config.py
new file mode 100644
index 00000000..2ea0b501
--- /dev/null
+++ b/website_cms/models/res_config.py
@@ -0,0 +1,15 @@
+# -*- coding: utf-8 -*-
+
+from openerp import models, fields
+
+
+class WebsiteConfigSettings(models.TransientModel):
+    """Override website config model."""
+
+    _inherit = "website.config.settings"
+
+    filter_menu = fields.Selection(
+        string='Menu Filter',
+        related='website_id.filter_menu',
+        help='Filter to determine which menu shows in the front-end'
+    )
diff --git a/website_cms/models/website.py b/website_cms/models/website.py
index 7ab4e6c9..6622c189 100644
--- a/website_cms/models/website.py
+++ b/website_cms/models/website.py
@@ -6,6 +6,7 @@
 # from openerp import fields
 from openerp import api
 from openerp import tools
+from openerp import fields
 from openerp.addons.web.http import request
 from openerp.addons.website.models.website import unslug
 
@@ -20,6 +21,18 @@ class Website(models.Model):
 
     _inherit = "website"
 
+    filter_menu = fields.Selection(
+        string="Show Menu",
+        required=True,
+        selection=[
+            ('1', 'Odoo'),
+            ('2', 'CMS'),
+            ('3', 'Odoo & CMS'),
+        ],
+        default='1',
+        help="Filter to determine which menu shows in the front-end"
+    )
+
     @api.model
     @tools.ormcache('max_depth', 'pages', 'nav', 'type_ids', 'published')
     def get_nav_pages(self, max_depth=3, pages=None,
diff --git a/website_cms/templates/layout.xml b/website_cms/templates/layout.xml
index e3fafe55..9dc6136c 100644
--- a/website_cms/templates/layout.xml
+++ b/website_cms/templates/layout.xml
@@ -11,6 +11,19 @@
         <t t-call="website_cms.alternate_languages" />
     </xpath>
 
+    <xpath expr="//ul[@id='top_menu']/t[@t-foreach='website.menu_id.child_id']" position="replace">
+      <t t-if="website.filter_menu in ['1','3']">
+        <t t-foreach="website.menu_id.child_id" t-as="submenu">
+          <t t-call="website.submenu"/>
+        </t>
+      </t>
+      <t t-if="website.filter_menu in ['2','3']">
+        <t t-foreach="request.env['cms.page'].search([])" t-as="submenu">
+          <t t-call="website_cms.submenu"/>
+        </t>
+      </t>
+    </xpath>
+
   </template>
 
   <template id="add_status_message" inherit_id="website.layout">
diff --git a/website_cms/templates/menu.xml b/website_cms/templates/menu.xml
index 5a4f3d53..9a9fda26 100644
--- a/website_cms/templates/menu.xml
+++ b/website_cms/templates/menu.xml
@@ -42,6 +42,32 @@
 
 </template>
 
+<template id="submenu" name="Website CMS Submenu">
+<!--     <li t-if="not submenu.child_id" t-att-class="
+        'active' if submenu.url and submenu.url != '/' and request.httprequest.path == submenu.url.replace('/page/website.', '/page/') else None
+        "> -->
+      <li>
+<!--         <a t-att-href="(website.menu_id.child_id[0] == submenu) and '/' or submenu.url" t-ignore="true" t-att-target="'_blank' if submenu.new_window else None"> -->
+        <a href="#">
+            <span t-field="submenu.name"/>
+        </a>
+    </li>
+<!--     <li t-if="submenu.child_id" t-attf-class="dropdown #{
+        (submenu.url and submenu.url != '/' and any([request.httprequest.path == child.url.replace('/page/website.', '/page/') for child in submenu.child_id if child.url]) or
+         (submenu.url and request.httprequest.path == submenu.url.replace('/page/website.', '/page/'))) and 'active'
+        }"> -->
+    <li>
+        <a class="dropdown-toggle" data-toggle="dropdown" href="#">
+            <span t-field="submenu.name"/> <span class="caret" t-ignore="true"></span>
+        </a>
+        <ul class="dropdown-menu" role="menu">
+            <t t-foreach="submenu.children_ids" t-as="submenu">
+                <t t-call="website_cms.submenu"/>
+            </t>
+        </ul>
+    </li>
+</template>
+
 </data>
 </openerp>
 
diff --git a/website_cms/views/res_config.xml b/website_cms/views/res_config.xml
new file mode 100644
index 00000000..7ce57106
--- /dev/null
+++ b/website_cms/views/res_config.xml
@@ -0,0 +1,20 @@
+<openerp>
+  <data>
+
+    <record id="website_config_settings" model="ir.ui.view">
+      <field name="name">website settings form</field>
+      <field name="model">website.config.settings</field>
+      <field name="inherit_id" ref="website.view_website_config_settings"/>
+      <field name="arch" type="xml">
+
+        <xpath expr="//group[@name='advanced']" position="after">
+          <group string="CMS" name="website-cms">
+            <field name="filter_menu" />
+          </group>
+        </xpath>
+
+      </field>
+    </record>
+
+  </data>
+</openerp>
\ No newline at end of file

From f30ffe43f8392da642c7332085674d00029708b5 Mon Sep 17 00:00:00 2001
From: Paul Catinean <paulcatinean@gmail.com>
Date: Tue, 4 Oct 2016 12:25:22 +0300
Subject: [PATCH 2/4] [IMP] Generating native Odoo menu for CMS pages

---
 website_cms/templates/layout.xml |  2 +-
 website_cms/templates/menu.xml   | 24 ++++++++++--------------
 website_cms/views/res_config.xml |  2 +-
 3 files changed, 12 insertions(+), 16 deletions(-)

diff --git a/website_cms/templates/layout.xml b/website_cms/templates/layout.xml
index 9dc6136c..ca7b4d7f 100644
--- a/website_cms/templates/layout.xml
+++ b/website_cms/templates/layout.xml
@@ -18,7 +18,7 @@
         </t>
       </t>
       <t t-if="website.filter_menu in ['2','3']">
-        <t t-foreach="request.env['cms.page'].search([])" t-as="submenu">
+        <t t-foreach="website.get_nav_pages()" t-as="submenu">
           <t t-call="website_cms.submenu"/>
         </t>
       </t>
diff --git a/website_cms/templates/menu.xml b/website_cms/templates/menu.xml
index 9a9fda26..618c963f 100644
--- a/website_cms/templates/menu.xml
+++ b/website_cms/templates/menu.xml
@@ -43,25 +43,21 @@
 </template>
 
 <template id="submenu" name="Website CMS Submenu">
-<!--     <li t-if="not submenu.child_id" t-att-class="
-        'active' if submenu.url and submenu.url != '/' and request.httprequest.path == submenu.url.replace('/page/website.', '/page/') else None
-        "> -->
-      <li>
-<!--         <a t-att-href="(website.menu_id.child_id[0] == submenu) and '/' or submenu.url" t-ignore="true" t-att-target="'_blank' if submenu.new_window else None"> -->
-        <a href="#">
-            <span t-field="submenu.name"/>
+    <li t-if="not submenu.get('children')" t-att-class="
+        'active' if submenu.get('url') and submenu.get('url') != '/' and request.httprequest.path == submenu.get('url') else None">
+        <a t-att-href="submenu.get('url')" t-ignore="true">
+            <span t-esc="submenu.get('name')"/>
         </a>
     </li>
-<!--     <li t-if="submenu.child_id" t-attf-class="dropdown #{
-        (submenu.url and submenu.url != '/' and any([request.httprequest.path == child.url.replace('/page/website.', '/page/') for child in submenu.child_id if child.url]) or
-         (submenu.url and request.httprequest.path == submenu.url.replace('/page/website.', '/page/'))) and 'active'
-        }"> -->
-    <li>
+    <li t-if="submenu.get('children')" t-attf-class="dropdown #{
+        (submenu.get('url') and submenu.get('url') != '/' and any([request.httprequest.path == child.get('url') for child in submenu.get('children', [])]) or
+         (submenu.get('url') and request.httprequest.path == submenu.get('url'))) and 'active'
+        }">
         <a class="dropdown-toggle" data-toggle="dropdown" href="#">
-            <span t-field="submenu.name"/> <span class="caret" t-ignore="true"></span>
+            <span t-esc="submenu.get('name')"/> <span class="caret" t-ignore="true"></span>
         </a>
         <ul class="dropdown-menu" role="menu">
-            <t t-foreach="submenu.children_ids" t-as="submenu">
+            <t t-foreach="submenu.get('children')" t-as="submenu">
                 <t t-call="website_cms.submenu"/>
             </t>
         </ul>
diff --git a/website_cms/views/res_config.xml b/website_cms/views/res_config.xml
index 7ce57106..b6740563 100644
--- a/website_cms/views/res_config.xml
+++ b/website_cms/views/res_config.xml
@@ -17,4 +17,4 @@
     </record>
 
   </data>
-</openerp>
\ No newline at end of file
+</openerp>

From f512f0df2c59cb4d7241e3cb3bedf666b65c1d17 Mon Sep 17 00:00:00 2001
From: Paul Catinean <paulcatinean@gmail.com>
Date: Tue, 4 Oct 2016 19:00:05 +0300
Subject: [PATCH 3/4] [FIX] PR Fixes

---
 website_cms/templates/menu.xml | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/website_cms/templates/menu.xml b/website_cms/templates/menu.xml
index 618c963f..62172641 100644
--- a/website_cms/templates/menu.xml
+++ b/website_cms/templates/menu.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<openerp>
-<data>
+<odoo>
 
 <!-- User Navbar -->
 
@@ -64,7 +63,6 @@
     </li>
 </template>
 
-</data>
-</openerp>
+</odoo>
 
 

From ffe6525561091b263224c15e90def9d7c82fe50a Mon Sep 17 00:00:00 2001
From: Paul Catinean <paulcatinean@gmail.com>
Date: Tue, 11 Oct 2016 22:16:32 +0300
Subject: [PATCH 4/4] [FIX] PR Fixes

---
 website_cms/templates/menu.xml | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/website_cms/templates/menu.xml b/website_cms/templates/menu.xml
index 62172641..87e67251 100644
--- a/website_cms/templates/menu.xml
+++ b/website_cms/templates/menu.xml
@@ -42,16 +42,14 @@
 </template>
 
 <template id="submenu" name="Website CMS Submenu">
-    <li t-if="not submenu.get('children')" t-att-class="
-        'active' if submenu.get('url') and submenu.get('url') != '/' and request.httprequest.path == submenu.get('url') else None">
+    <t t-set="is_path" t-value="submenu.get('url') != '/' and request.httprequest.path == submenu.get('url')"/>
+    <t t-set="child_in_path" t-value="any([request.httprequest.path == child.get('url') for child in submenu.get('children', [])])"/>
+    <li t-if="not submenu.get('children')" t-att-class="'active' if is_path else None">
         <a t-att-href="submenu.get('url')" t-ignore="true">
             <span t-esc="submenu.get('name')"/>
         </a>
     </li>
-    <li t-if="submenu.get('children')" t-attf-class="dropdown #{
-        (submenu.get('url') and submenu.get('url') != '/' and any([request.httprequest.path == child.get('url') for child in submenu.get('children', [])]) or
-         (submenu.get('url') and request.httprequest.path == submenu.get('url'))) and 'active'
-        }">
+    <li t-if="submenu.get('children')" t-attf-class="dropdown #{(submenu.get('url') and submenu.get('url') != '/' and child_in_path or is_path) and 'active'}">
         <a class="dropdown-toggle" data-toggle="dropdown" href="#">
             <span t-esc="submenu.get('name')"/> <span class="caret" t-ignore="true"></span>
         </a>