Merge pull request #5984 from wallabag/fix/api-tags-nb-entries

Add `nbEntries` to the API tags list response

src/Wallabag/ApiBundle/Controller/TagRestController.php

@@ -23,7 +23,7 @@ class TagRestController extends WallabagRestController
 
         $tags = $this->getDoctrine()
             ->getRepository('WallabagCoreBundle:Tag')
-            ->findAllTags($this->getUser()->getId());
+            ->findAllFlatTagsWithNbEntries($this->getUser()->getId());
 
         $json = $this->get('jms_serializer')->serialize($tags, 'json');
 

src/Wallabag/CoreBundle/Resources/views/themes/material/Tag/tags.html.twig

@@ -33,8 +33,4 @@
             {% endfor %}
         </ul>
     </div>
-
-    <div>
-
-    </div>
 {% endblock %}

tests/Wallabag/ApiBundle/Controller/TagRestControllerTest.php

@@ -20,14 +20,13 @@ class TagRestControllerTest extends WallabagApiTestCase
         $this->assertGreaterThan(0, $content);
         $this->assertArrayHasKey('id', $content[0]);
         $this->assertArrayHasKey('label', $content[0]);
+        $this->assertArrayHasKey('nbEntries', $content[0]);
 
         $tagLabels = array_map(function ($i) {
             return $i['label'];
         }, $content);
 
         $this->assertNotContains($this->otherUserTagLabel, $tagLabels, 'There is a possible tag leak');
-
-        return end($content);
     }
 
     public function testDeleteUserTag()