feature #765 Filter posts by selected tag (yceruto)

This PR was merged into the master branch.

Discussion
----------

Filter posts by selected tag

Fixes #762

Even though this PR doesn't show any new Symfony feature, it makes the post tags have some sense.

Commits
-------

c6803f0 Filter posts by tag

Author: javiereguiluz

src/Controller/BlogController.php

@@ -16,6 +16,7 @@
 use App\Events;
 use App\Form\CommentType;
 use App\Repository\PostRepository;
+use App\Repository\TagRepository;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Cache;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter;
@@ -46,9 +47,13 @@ class BlogController extends AbstractController
      * Content-Type header for the response.
      * See https://symfony.com/doc/current/quick_tour/the_controller.html#using-formats
      */
-    public function index(int $page, string $_format, PostRepository $posts): Response
+    public function index(Request $request, int $page, string $_format, PostRepository $posts, TagRepository $tags): Response
     {
-        $latestPosts = $posts->findLatest($page);
+        $tag = null;
+        if ($request->query->has('tag')) {
+            $tag = $tags->findOneBy(['name' => $request->query->get('tag')]);
+        }
+        $latestPosts = $posts->findLatest($page, $tag);
 
         // Every template name also has two extensions that specify the format and
         // engine for that template.

src/Repository/PostRepository.php

@@ -12,6 +12,7 @@
 namespace App\Repository;
 
 use App\Entity\Post;
+use App\Entity\Tag;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
 use Doctrine\Common\Persistence\ManagerRegistry;
 use Doctrine\ORM\Query;
@@ -35,21 +36,22 @@ public function __construct(ManagerRegistry $registry)
         parent::__construct($registry, Post::class);
     }
 
-    public function findLatest(int $page = 1): Pagerfanta
+    public function findLatest(int $page = 1, Tag $tag = null): Pagerfanta
     {
-        $query = $this->getEntityManager()
-            ->createQuery('
-                SELECT p, a, t
-                FROM App:Post p
-                JOIN p.author a
-                LEFT JOIN p.tags t
-                WHERE p.publishedAt <= :now
-                ORDER BY p.publishedAt DESC
-            ')
-            ->setParameter('now', new \DateTime())
-        ;
-
-        return $this->createPaginator($query, $page);
+        $qb = $this->createQueryBuilder('p')
+            ->addSelect('a', 't')
+            ->innerJoin('p.author', 'a')
+            ->leftJoin('p.tags', 't')
+            ->where('p.publishedAt <= :now')
+            ->orderBy('p.publishedAt', 'DESC')
+            ->setParameter('now', new \DateTime());
+
+        if (null !== $tag) {
+            $qb->andWhere(':tag MEMBER OF p.tags')
+                ->setParameter('tag', $tag);
+        }
+
+        return $this->createPaginator($qb->getQuery(), $page);
     }
 
     private function createPaginator(Query $query, int $page): Pagerfanta

templates/blog/_post_tags.html.twig

@@ -1,9 +1,11 @@
 {% if not post.tags.empty %}
     <p class="post-tags">
         {% for tag in post.tags %}
-            <span class="label label-default">
+            <a href="{{ path('blog_index', {'tag': tag.name}) }}"
+               class="label label-{{ tag.name == app.request.query.get('tag') ? 'success' : 'default' }}"
+            >
                 <i class="fa fa-tag"></i> {{ tag.name }}
-            </span>
+            </a>
         {% endfor %}
     </p>
 {% endif %}

templates/blog/_rss.html.twig

@@ -1,5 +1,5 @@
 <div class="section rss">
-    <a href="{{ path('blog_rss') }}">
+    <a href="{{ path('blog_rss', app.request.query.all) }}">
         <i class="fa fa-rss" aria-hidden="true"></i> {{ 'menu.rss'|trans }}
     </a>
 </div>

templates/blog/index.html.twig

@@ -26,7 +26,7 @@
 
     {% if posts.haveToPaginate %}
         <div class="navigation text-center">
-            {{ pagerfanta(posts, 'twitter_bootstrap3_translated', {routeName: 'blog_index_paginated'}) }}
+            {{ pagerfanta(posts, 'twitter_bootstrap3_translated', {routeName: 'blog_index_paginated', routeParams: app.request.query.all}) }}
         </div>
     {% endif %}
 {% endblock %}