From 2fd278ecf4cc476b8088d5440d3a316ba5c35ec5 Mon Sep 17 00:00:00 2001
From: Thomas NOEL <tnoel@entrouvert.com>
Date: Tue, 27 Sep 2016 16:26:40 +0200
Subject: [PATCH] contacts: display user roles in contact zone (#12630)

---
 .../templates/contacts/contact_detail_fragment.html       | 15 ++++++++-------
 welco/contacts/views.py                                   |  2 ++
 welco/static/css/style.css                                |  1 +
 welco/static/js/welco.js                                  |  3 +++
 4 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/welco/contacts/templates/contacts/contact_detail_fragment.html b/welco/contacts/templates/contacts/contact_detail_fragment.html
index 8451907..b21335b 100644
--- a/welco/contacts/templates/contacts/contact_detail_fragment.html
+++ b/welco/contacts/templates/contacts/contact_detail_fragment.html
@@ -2,16 +2,17 @@
 <div class="contact">
 <input type="hidden" id="current-selected-user" name="current-selected-user" value="{{user_id}}"/>
 <h3>{{user_display_name}}</h3>
+<ul>
 {% if user_var_address or user_var_city %}
-<p>{{user_var_address}} / {{user_var_zipcode}} {{user_var_city}}</p>
+<li>{% trans 'Address:' %} {{user_var_address}} / {{user_var_zipcode}} {{user_var_city}}</li>
 {% endif %}
-{% if user_var_phone or user_var_mobile or user_var_email %}
-<p>
-  {% if user_var_phone %}{% trans 'Phone:' %} {{user_var_phone}}{% endif %}
-  {% if user_var_mobile %}{% trans 'Mobile:' %} {{user_var_mobile}}{% endif %}
-  {% if user_var_email %}{% trans 'Email:' %} {{user_var_email}}{% endif %}
-</p>
+{% if user_var_phone %}<li>{% trans 'Phone:' %} {{user_var_phone}}</li>{% endif %}
+{% if user_var_mobile %}<li>{% trans 'Mobile:' %} {{user_var_mobile}}</li>{% endif %}
+{% if user_var_email %}<li>{% trans 'Email:' %} {{user_var_email}}</li>{% endif %}
+{% if user_roles %}
+<li>{% trans 'Roles:' %} {% for role in user_roles %}{{ role.text }}{% if not forloop.last %}, {% endif %}{% endfor %}</li>
 {% endif %}
+</ul>
 
 {% if forms %}
 <ul class="formdatas">
diff --git a/welco/contacts/views.py b/welco/contacts/views.py
index 5849df6..9fcdd28 100644
--- a/welco/contacts/views.py
+++ b/welco/contacts/views.py
@@ -86,6 +86,8 @@ def search_json(request):
             more = [user.get('user_var_address'), user.get('user_var_phone'),
                 user.get('user_var_birthdate'), user.get('user_var_email')]
             user['more'] = ' / '.join([x for x in more if x])
+            if user.get('user_roles'):
+                user['roles'] = ' / '.join([r['text'] for r in user['user_roles']])
             user['slug'] = 'user-%s' % user['user_id']
     else:
         result = {'data': []}
diff --git a/welco/static/css/style.css b/welco/static/css/style.css
index c618d6c..848e263 100644
--- a/welco/static/css/style.css
+++ b/welco/static/css/style.css
@@ -251,6 +251,7 @@ ul.formdatas a.remove {
 	color: #3c3c33;
 }
 
+ul.result li a span.roles,
 ul.result li a span.more {
 	display: block;
 	color: #666;
diff --git a/welco/static/js/welco.js b/welco/static/js/welco.js
index 32cd7ce..e709986 100644
--- a/welco/static/js/welco.js
+++ b/welco/static/js/welco.js
@@ -298,6 +298,9 @@ $(function() {
                        if (elem.more) {
                          new_elem = new_elem + ' <span class="more">' + elem.more + '</span>';
                        }
+                       if (elem.roles) {
+                         new_elem = new_elem + ' <span class="roles">' + elem.roles + '</span>';
+                       }
                        new_elem = new_elem + '</a></li>'
                        $(new_elem).appendTo(search_result_ul);
                     });
-- 
2.9.3