From 3742f4b06c50fbc60ec499920a5079b77743dbe8 Mon Sep 17 00:00:00 2001
From: Alexey Skobkin <x@skobkin.ru>
Date: Tue, 2 Jun 2015 05:24:38 +0300
Subject: [PATCH] Fix #16.

---
 .../Resources/public/css/main.css             |   4 +
 .../Resources/views/User/show.html.twig       | 115 ++++++++++--------
 2 files changed, 66 insertions(+), 53 deletions(-)

diff --git a/src/Skobkin/Bundle/PointToolsBundle/Resources/public/css/main.css b/src/Skobkin/Bundle/PointToolsBundle/Resources/public/css/main.css
index 2818740..167a190 100644
--- a/src/Skobkin/Bundle/PointToolsBundle/Resources/public/css/main.css
+++ b/src/Skobkin/Bundle/PointToolsBundle/Resources/public/css/main.css
@@ -38,6 +38,10 @@ body > .container {
     margin-top: 30px;
 }
 
+.user-subscriptions-log {
+    margin-top: 30px;
+}
+
 h4.panel-title a {
     text-decoration: none;
 }
diff --git a/src/Skobkin/Bundle/PointToolsBundle/Resources/views/User/show.html.twig b/src/Skobkin/Bundle/PointToolsBundle/Resources/views/User/show.html.twig
index 6d3f344..947131a 100644
--- a/src/Skobkin/Bundle/PointToolsBundle/Resources/views/User/show.html.twig
+++ b/src/Skobkin/Bundle/PointToolsBundle/Resources/views/User/show.html.twig
@@ -9,68 +9,77 @@
     </h1>
 
     <div class="user-subscribers">
-        <div class="panel-group" id="accordion-subscribers">
-            <div class="panel panel-default">
-                <div class="panel-heading"id="headingOne">
-                    <h4 class="panel-title">
-                        <a data-toggle="collapse" data-parent="#accordion-subscribers" aria-expanded="true" href="#collapse-subscribers">
-                            <span class="glyphicon glyphicon-collapse-down"></span> {{ 'Subscribers'|trans }}
-                        </a>
-                    </h4>
-                </div>
-                <div id="collapse-subscribers" class="panel-collapse collapse in">
-                    <div class="panel-body">
-                        <ul class="users mosaic">
-                            {% for user in subscribers %}
-                                <li><a href="{{ url('user_show', {login: user.login}) }}">@{{ user.login }}</a></li>
-                            {% endfor %}
-                        </ul>
+        {% if subscribers|length > 0 %}
+            <div class="panel-group" id="accordion-subscribers">
+                <div class="panel panel-default">
+                    <div class="panel-heading"id="headingOne">
+                        <h4 class="panel-title">
+                            <a data-toggle="collapse" data-parent="#accordion-subscribers" aria-expanded="true" href="#collapse-subscribers">
+                                <span class="glyphicon glyphicon-collapse-down"></span> {{ 'Subscribers'|trans }}
+                            </a>
+                        </h4>
+                    </div>
+                    <div id="collapse-subscribers" class="panel-collapse collapse in">
+                        <div class="panel-body">
+                            <ul class="users mosaic">
+                                {% for user in subscribers %}
+                                    <li><a href="{{ url('user_show', {login: user.login}) }}">@{{ user.login }}</a></li>
+                                {% endfor %}
+                            </ul>
+                        </div>
                     </div>
                 </div>
             </div>
-        </div>
+        {% else %}
+            <div class="alert alert-warning" role="alert">{{ 'No subscribers data found'|trans }}</div>
+        {% endif %}
     </div>
 
-    <div class="user-subscribers">
-        <div class="panel-group" id="accordion-log">
-            <div class="panel panel-default">
-                <div class="panel-heading" id="headingOne">
-                    <h4 class="panel-title">
-                        <a data-toggle="collapse" data-parent="#accordion-log" aria-expanded="true" href="#collapse-log">
-                            <span class="glyphicon glyphicon-collapse-down"></span> {{ 'Subscriptions log'|trans }}
-                        </a>
-                    </h4>
-                </div>
-                <div id="collapse-log" class="panel-collapse collapse in">
-                    <div class="panel-body">
-                        <table class="table table-striped">
-                            <thead>
-                                <tr>
-                                    <td>{{ 'User'|trans }}</td>
-                                    <td>{{ 'Action'|trans }}</td>
-                                    <td>{{ 'Date'|trans }}</td>
-                                </tr>
-                            </thead>
-                            <tbody>
-                                {% for event in log %}
+    <div class="user-subscriptions-log">
+        {% if log|length > 0 %}
+            <div class="panel-group" id="accordion-log">
+                <div class="panel panel-default">
+                    <div class="panel-heading" id="headingOne">
+                        <h4 class="panel-title">
+                            <a data-toggle="collapse" data-parent="#accordion-log" aria-expanded="true" href="#collapse-log">
+                                <span class="glyphicon glyphicon-collapse-down"></span> {{ 'Subscriptions log'|trans }}
+                            </a>
+                        </h4>
+                    </div>
+                    <div id="collapse-log" class="panel-collapse collapse in">
+                        <div class="panel-body">
+                            <table class="table table-striped">
+                                <thead>
                                     <tr>
-                                        <td>
-                                            <a href="{{ url('user_show', {login: event.subscriber.login}) }}">@{{ event.subscriber.login }}</a>
-                                        </td>
-                                        <td>
-                                            <span class="glyphicon {% if event.action == 'subscribe' %}glyphicon-plus{% elseif event.action == 'unsubscribe' %}glyphicon-minus{% endif %}"></span>
-                                        </td>
-                                        <td>
-                                            {# Use DateTime helper: https://sonata-project.org/bundles/intl/master/doc/reference/datetime.html #}
-                                            {{ event.date|date('H:i:s d F Y') }}
-                                        </td>
+                                        <td>{{ 'User'|trans }}</td>
+                                        <td>{{ 'Action'|trans }}</td>
+                                        <td>{{ 'Date'|trans }}</td>
                                     </tr>
-                                {% endfor %}
-                            </tbody>
-                        </table>
+                                </thead>
+                                <tbody>
+                                    {% for event in log %}
+                                        <tr>
+                                            <td>
+                                                <a href="{{ url('user_show', {login: event.subscriber.login}) }}">@{{ event.subscriber.login }}</a>
+                                            </td>
+                                            <td>
+                                                <span class="glyphicon {% if event.action == 'subscribe' %}glyphicon-plus{% elseif event.action == 'unsubscribe' %}glyphicon-minus{% endif %}"></span>
+                                            </td>
+                                            <td>
+                                                {# Use DateTime helper: https://sonata-project.org/bundles/intl/master/doc/reference/datetime.html #}
+                                                {{ event.date|date('H:i:s d F Y') }}
+                                            </td>
+                                        </tr>
+                                    {% endfor %}
+                                </tbody>
+                            </table>
+                        </div>
                     </div>
                 </div>
             </div>
-        </div>
+        {% else %}
+            <div class="alert alert-warning" role="alert">{{ 'No log data found'|trans }}</div>
+        {% endif %}
     </div>
+
 {% endblock %}