diff options
| author | qmo-odoo <qmo@odoo.com> | 2020-01-16 08:36:13 +0000 |
|---|---|---|
| committer | qmo-odoo <qmo@odoo.com> | 2020-03-26 14:59:16 +0000 |
| commit | 91eded2eb280ddb2dc4acfe92cab5c98a7554fee (patch) | |
| tree | 1e8d9964a5b97b6aa6baa93e3ec8f5c2a6d9f835 /addons/website_event | |
| parent | 307263c22d9ea1eecc1f6fba478c24a7a845f2c7 (diff) | |
[IMP] website_event, event: Add event.tag.category and event.tag + rename event.type display name
This commit changes the display name for the model event.type from
"Event Category" to "Event Template"
This change was needed before introducing the new model event.tag.category
which will be used to group tags (event.tag)
This commit introduces two new models: event.tag.category & event.tag
The tags will be grouped by category and will be used to filter events.
In the front-end, if the user activates the "filter by categories", each
event.tag.category will generate a new dropdown of related tags.
Clicking on one of these tags will add a new "tag badge" above the events
(same way as in eLearning).
LINKS:
TaskID:2162438
PR: #43856
Diffstat (limited to 'addons/website_event')
| -rw-r--r-- | addons/website_event/controllers/main.py | 37 | ||||
| -rw-r--r-- | addons/website_event/security/ir.model.access.csv | 2 | ||||
| -rw-r--r-- | addons/website_event/static/src/scss/website_event.scss | 8 | ||||
| -rw-r--r-- | addons/website_event/views/event_templates.xml | 98 |
4 files changed, 92 insertions, 53 deletions
diff --git a/addons/website_event/controllers/main.py b/addons/website_event/controllers/main.py index 570074d7b4b..8691222101d 100644 --- a/addons/website_event/controllers/main.py +++ b/addons/website_event/controllers/main.py @@ -5,6 +5,8 @@ import re import werkzeug from werkzeug.datastructures import OrderedMultiDict +from ast import literal_eval +from collections import defaultdict from datetime import datetime, timedelta from dateutil.relativedelta import relativedelta @@ -12,6 +14,7 @@ from odoo import fields, http, _ from odoo.addons.http_routing.models.ir_http import slug from odoo.addons.website.controllers.main import QueryURL from odoo.http import request +from odoo.osv import expression from odoo.tools.misc import get_lang, format_date @@ -28,6 +31,7 @@ class WebsiteEventController(http.Controller): searches.setdefault('search', '') searches.setdefault('date', 'all') + searches.setdefault('tags', '') searches.setdefault('type', 'all') searches.setdefault('country', 'all') @@ -70,6 +74,19 @@ class WebsiteEventController(http.Controller): if searches['search']: domain_search['search'] = [('name', 'ilike', searches['search'])] + search_tags = self._extract_searched_event_tags(searches) + if search_tags: + # Example: You filter on age: 10-12 and activity: football. + # Doing it this way allows to only get events who are tagged "age: 10-12" AND "activity: football". + # Add another tag "age: 12-15" to the search and it would fetch the ones who are tagged: + # ("age: 10-12" OR "age: 12-15") AND "activity: football + grouped_tags = defaultdict(list) + for tag in search_tags: + grouped_tags[tag.category_id].append(tag) + domain_search['tags'] = [] + for group in grouped_tags: + domain_search['tags'] = expression.AND([domain_search['tags'], [('tag_ids', 'in', [tag.id for tag in grouped_tags[group]])]]) + current_date = None current_type = None current_country = None @@ -102,11 +119,6 @@ class WebsiteEventController(http.Controller): date[3] = Event.search_count(dom_without('date') + date[2]) domain = dom_without('type') - types = Event.read_group(domain, ["id", "event_type_id"], groupby=["event_type_id"], orderby="event_type_id") - types.insert(0, { - 'event_type_id_count': sum([int(type['event_type_id_count']) for type in types]), - 'event_type_id': ("all", _("All Categories")) - }) domain = dom_without('country') countries = Event.read_group(domain, ["id", "country_id"], groupby="country_id", orderby="country_id") @@ -141,10 +153,11 @@ class WebsiteEventController(http.Controller): 'current_type': current_type, 'event_ids': events, # event_ids used in website_event_track so we keep name as it is 'dates': dates, - 'types': types, + 'categories': request.env['event.tag.category'].search([]), 'countries': countries, 'pager': pager, 'searches': searches, + 'search_tags': search_tags, 'keep': keep, } @@ -342,3 +355,15 @@ class WebsiteEventController(http.Controller): 'google_url': urls.get('google_url'), 'iCal_url': urls.get('iCal_url') }) + + def _extract_searched_event_tags(self, searches): + tags = request.env['event.tag'] + if searches.get('tags'): + try: + tag_ids = literal_eval(searches['tags']) + except: + pass + else: + # perform a search to filter on existing / valid tags implicitely + tags = request.env['event.tag'].search([('id', 'in', tag_ids)]) + return tags diff --git a/addons/website_event/security/ir.model.access.csv b/addons/website_event/security/ir.model.access.csv index 1024bdc8213..e55cfc58bbf 100644 --- a/addons/website_event/security/ir.model.access.csv +++ b/addons/website_event/security/ir.model.access.csv @@ -4,3 +4,5 @@ access_event_event_portal,event.event.portal,event.model_event_event,base.group_ access_event_type_public,event.type.public,event.model_event_type,,1,0,0,0 access_event_event_ticket_public,event.event.ticket.public,event.model_event_event_ticket,base.group_public,1,0,0,0 access_event_event_ticket_portal,event.event.ticket.portal,event.model_event_event_ticket,base.group_portal,1,0,0,0 +access_event_category_public,event.tag.category.public,event.model_event_tag_category,,1,0,0,0 +access_event_tag_public,event.tag.public,event.model_event_tag,,1,0,0,0 diff --git a/addons/website_event/static/src/scss/website_event.scss b/addons/website_event/static/src/scss/website_event.scss index 4be4eeacba2..127874736f7 100644 --- a/addons/website_event/static/src/scss/website_event.scss +++ b/addons/website_event/static/src/scss/website_event.scss @@ -1,11 +1,3 @@ -.o_wevent_event_searchbar_form { - flex: 0 0 100%; - - @include media-breakpoint-up(lg) { - flex: 0 0 33%; - } -} - .o_wevent_index_topbar_filters { .dropdown-toggle { border: $border-width solid gray('400'); diff --git a/addons/website_event/views/event_templates.xml b/addons/website_event/views/event_templates.xml index 3bd32369a92..35592fed462 100644 --- a/addons/website_event/views/event_templates.xml +++ b/addons/website_event/views/event_templates.xml @@ -16,6 +16,9 @@ <div t-attf-class="o_wevent_events_list #{opt_events_list_cards and 'opt_event_list_cards_bg'}"> <div class="container"> <div class="row"> + <t t-call="website_event.searched_tags"/> + </div> + <div class="row"> <div id="o_wevent_index_main_col" t-attf-class="col-md my-5 #{opt_events_list_columns and 'opt_events_list_columns' or 'opt_events_list_rows'}"> <div class="row"> <!-- Events List --> @@ -42,20 +45,40 @@ <template id="index_topbar" name="Topbar"> <nav class="navbar navbar-light border-top shadow-sm d-print-none"> <div class="container"> - <span class="navbar-brand h4 my-0 mr-4">Events</span> - <ul class="o_wevent_index_topbar_filters nav mr-n2"/> - <t t-call="website_event.events_search_box"> - <t t-set="_classes" t-value="'ml-auto pl-lg-3'"/> - <t t-set="_searches" t-value="searches"/> - </t> + <div class="row d-flex justify-content-between w-100 mr-0"> + <div class="col-lg-8 col-sm-12"> + <span class="navbar-brand h4 my-0">Events</span> + <ul class="o_wevent_index_topbar_filters nav"/> + </div> + <div class="col-lg-3 col-sm-12 d-flex align-items-center flex-wrap px16"> + <t t-call="website_event.events_search_box"> + <t t-set="_searches" t-value="searches"/> + </t> + <a href="/event" class="btn btn-info text-nowrap mr-0 w-100" role="button" title="Clear filters"> + <i class="fa fa-eraser"/> Clear filters + </a> + </div> + </div> </div> </nav> </template> +<template id="searched_tags" name="Searched tags"> + <div class="d-flex align-items-center mt16"> + <t t-foreach="search_tags" t-as="tag"> + <span class="align-items-baseline border d-inline-flex pl-2 rounded ml16 mb-2 bg-white"> + <i class="fa fa-tag mr-2 text-muted"/> + <t t-esc="tag.display_name"/> + <a t-att-href="'/event?%s' % keep_query('*', tags=str((search_tags - tag).ids))" class="btn border-0 py-1">×</a> + </span> + </t> + </div> +</template> + <!-- Filter - Date --> <template id="event_time" inherit_id="website_event.index_topbar" customize_show="True" name="Filter by Date"> <xpath expr="//ul[hasclass('o_wevent_index_topbar_filters')]" position="inside"> - <li class="nav-item dropdown mr-2 my-1 my-lg-0"> + <li class="nav-item dropdown mr-2 my-1"> <a href="#" role="button" class="btn dropdown-toggle" data-toggle="dropdown"> <i class="fa fa-calendar"/> <t t-if="current_date" t-esc="current_date"/> @@ -75,33 +98,36 @@ </xpath> </template> +<<<<<<< HEAD <!-- Filter - Category --> <template id="event_category" inherit_id="website_event.index_topbar" active="False" customize_show="True" name="Filter by Category"> +======= +<template id="event_category_tag" inherit_id="website_event.index_topbar" active="False" customize_show="True" name="Filter by Tags"> +>>>>>>> 27cb059... [IMP] website_event, event: Add event.tag.category and event.tag + rename event.type display name <xpath expr="//ul[hasclass('o_wevent_index_topbar_filters')]" position="inside"> - <li class="nav-item dropdown mr-2 my-1 my-lg-0"> - <a href="#" role="button" class="btn dropdown-toggle" data-toggle="dropdown"> - <i class="fa fa-folder-open"/> - <t t-if="current_type" t-esc="current_type.name"/> - <t t-else="">All Categories</t> - </a> - <div class="dropdown-menu"> - <t t-foreach="types" t-as="type"> - <t t-if="type['event_type_id']"> - <a t-att-href="keep('/event', type=type['event_type_id'][0])" t-attf-class="dropdown-item d-flex align-items-center justify-content-between #{searches.get('type') == str(type['event_type_id'] and type['event_type_id'][0]) and 'active'}"> - <t t-esc="type['event_type_id'][1]"/> - <span t-esc="type['event_type_id_count']" class="badge badge-pill badge-primary ml-3"/> + <t t-foreach="categories" t-as="category"> + <li t-if="category.tag_ids" class="nav-item dropdown mr-2 my-1"> + <a href="#" role="button" class="btn dropdown-toggle" data-toggle="dropdown"> + <i class="fa fa-folder-open"/> + <t t-esc="category.name"/> + </a> + <div class="dropdown-menu"> + <t t-foreach="category.tag_ids" t-as="tag"> + <a t-att-href="'/event?%s' % keep_query('*', tags=str((search_tags - tag).ids if tag in search_tags else (tag | search_tags).ids))" + t-attf-class="dropdown-item d-flex align-items-center justify-content-between #{'active' if tag in search_tags else ''}"> + <t t-esc="tag.name"/> </a> </t> - </t> - </div> - </li> + </div> + </li> + </t> </xpath> </template> <!-- Filter - Location --> <template id="event_location" inherit_id="website_event.index_topbar" active="False" customize_show="True" name="Filter by Country"> <xpath expr="//ul[hasclass('o_wevent_index_topbar_filters')]" position="inside"> - <li class="nav-item dropdown mr-2 my-1 my-lg-0"> + <li class="nav-item dropdown mr-2 my-1"> <a href="#" role="button" class="btn dropdown-toggle" data-toggle="dropdown"> <i class="fa fa-map-marker"/> <t t-if="current_country" t-esc="current_country.name"/> @@ -130,7 +156,7 @@ <!-- Search Box --> <template id="events_search_box" inherit_id="website.website_search_box" primary="True"> <xpath expr="//div[@role='search']" position="replace"> - <form t-attf-class="o_wevent_event_searchbar_form o_wait_lazy_js my-1 my-lg-0 #{_classes}" + <form t-attf-class="o_wevent_event_searchbar_form o_wait_lazy_js w-100 my-1 my-lg-0 #{_classes}" t-att-action="action if action else '/event'" method="get"> <t t-set="search" t-value="search or _searches and _searches['search']"/> <t>$0</t> @@ -208,6 +234,14 @@ </div> <!-- Location --> <div t-if="not event.is_online" itemprop="location" t-field="event.address_id" t-options="{'widget': 'contact', 'fields': ['city'], 'no_marker': 'true'}"/> + <div class="mt8 d-flex align-items-center"> + <t t-foreach="event.tag_ids" t-as="tag"> + <a t-att-href="'/event?%s' % keep_query('*', tags=str((search_tags - tag).ids if tag in search_tags else (tag | search_tags).ids))" + t-attf-class="badge mr8 #{'badge-primary' if tag in search_tags else 'badge-light'}"> + <span t-esc="tag.name"/> + </a> + </t> + </div> </main> </div> <!-- Footer --> @@ -231,7 +265,7 @@ <template id="opt_events_list_cards" inherit_id="website_event.events_list" active="True" customize_show="True" name="'Cards' Design"/> -<template id="opt_events_list_categories" inherit_id="website_event.events_list" active="True" customize_show="True" name="Show categories"> +<template id="opt_events_list_categories" inherit_id="website_event.events_list" active="True" customize_show="True" name="Show Templates"> <xpath expr="//main/*" position="before"> <a t-if="event.event_type_id" t-attf-href="/event?type=#{event.event_type_id.id}" t-attf-class="badge bg-secondary o_wevent_badge #{opt_events_list_columns and 'o_wevent_badge_event' or 'float-right'}" t-field="event.event_type_id"/> </xpath> @@ -290,20 +324,6 @@ </xpath> </template> -<!-- Index - Sidebar - Categories --> -<template id="index_sidebar_categories" inherit_id="website_event.index_sidebar" active="True" customize_show="True" name="Categories" priority="50"> - <xpath expr="//div[@id='o_wevent_index_sidebar']" position="inside"> - <div class="o_wevent_sidebar_block"> - <h6 class="o_wevent_sidebar_title">Categories</h6> - <t t-foreach="types" t-as="type"> - <t t-if="type['event_type_id']"> - <a t-att-href="keep('/event', type=type['event_type_id'][0])" class="badge badge-secondary o_wevent_badge mb-2"><t t-esc="type['event_type_id'][1]"/></a> - </t> - </t> - </div> - </xpath> -</template> - <!-- Index - Sidebar - Quotes --> <template id="index_sidebar_quotes" inherit_id="website_event.index_sidebar" active="True" customize_show="True" name="Quotes" priority="60"> <xpath expr="//div[@id='o_wevent_index_sidebar']" position="inside"> |
