Fix issue in chunking likes in groups of 6 that caused error. Fix boj…

…anz#6
olberger · Apr 7, 2017 · d47c4ad · d47c4ad
1 parent c70ef15
commit d47c4ad
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git a/app/views/profile.html.twig b/app/views/profile.html.twig
@@ -24,14 +24,18 @@
     <div class="container">
         {% for likeGroup in groupedLikes %}
         <div class="row">
-            {% for like in likeGroup %}
-              <div class="span2">
+          {% for like in likeGroup %}
+            {% if like.artist is not empty %}
+            <div class="span2">
+
                 <a href="{{ path('artist', {'artist': like.artist.id}) }}" class="amazing_image">
                   <h6 class="photo_hover">{{ like.artist.name }}</h6>
                   <img src="{{ app.request.basepath }}/img/artists/{{ like.artist.image }}">
                 </a>
-              </div>
-            {% endfor %}
+
+            </div>
+            {% endif %}
+          {% endfor %}
         </div>
         {% endfor %}
     </div>

diff --git a/src/MusicBox/Controller/UserController.php b/src/MusicBox/Controller/UserController.php
@@ -20,13 +20,8 @@ public function meAction(Request $request, Application $app)
         $likes = $app['repository.like']->findAllByUser($user->getId(), $limit);
         // Divide artists into groups of 6.
         $groupSize = 6;
-        $groupedLikes = array();
-        $progress = 0;
-        while ($progress < $limit) {
-            $groupedLikes[] = array_slice($likes, $progress, $groupSize);
-            $progress += $groupSize;
-        }
-
+        $groupedLikes = array_chunk($likes, $groupSize);
+
         $data = array(
             'user' => $user,
             'memberSince' => $memberSince,