From 55a72607e28af3f2d13977a17e9bb6a2be1b8a0c Mon Sep 17 00:00:00 2001
From: Rico van Zelst <ricovzelst@pm.me>
Date: Mon, 1 Apr 2024 16:27:07 +0200
Subject: [PATCH] refactor: improve filtering of hidden posts in
 PostsController - Refactored the code in `PostsController.php` to filter out
 hidden posts before sorting by date for better performance and accuracy. -
 Updated the blade template `listposts.blade.php` to adjust formatting and
 structure for displaying posts.

---
 app/Http/Controllers/PostsController.php        |  4 +++-
 .../views/components/posts/listposts.blade.php  | 17 +++++++++--------
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/app/Http/Controllers/PostsController.php b/app/Http/Controllers/PostsController.php
index 76ad576..e63aee1 100644
--- a/app/Http/Controllers/PostsController.php
+++ b/app/Http/Controllers/PostsController.php
@@ -10,7 +10,9 @@ class PostsController extends Controller
 {
     public function index()
     {
-        $posts = Sheets::all()->sortByDesc('date');
+        $posts = Sheets::all()->filter(function ($post) {
+            return !$post->hidden;
+        })->sortByDesc('date');
         $paginatedPosts = Paginate::collection($posts, 6);
 
         return view('posts.index', [
diff --git a/resources/views/components/posts/listposts.blade.php b/resources/views/components/posts/listposts.blade.php
index ffc87dc..88e55a9 100644
--- a/resources/views/components/posts/listposts.blade.php
+++ b/resources/views/components/posts/listposts.blade.php
@@ -2,22 +2,23 @@
 
 <div class="container flex flex-col items-center justify-center p-4 mx-auto mt-3">
     <div class="grid w-full grid-cols-1 gap-12 md:grid-cols-2 auto-cols-max">
-        @foreach($posts as $post)
-            @if($post->hidden)
+        @foreach ($posts as $post)
+            @if ($post->hidden)
                 @continue
             @endif
-            <article class="relative items-center inline-block text-gray-200 border shadow-md bg-stone-800/40 rounded-2xl border-stone-800 hover:border-orange-500/10 hover:shadow-orange-500/10 h-80">
-                <span
-                    class="absolute px-1 py-1 text-sm font-medium text-gray-100 rounded-lg top-4 left-4 bg-black/60">
+            <article
+                class="relative items-center inline-block text-gray-200 border shadow-md bg-stone-800/40 rounded-2xl border-stone-800 hover:border-orange-500/10 hover:shadow-orange-500/10 h-80">
+                <span class="absolute px-1 py-1 text-sm font-medium text-gray-100 rounded-lg top-4 left-4 bg-black/60">
                     <abbr itemprop="datePublished">{{ Carbon::parse($post->date)->format('F d, Y') }}</abbr>
                 </span>
-                <img src="{{ $post->thumbnail }}" alt="Post Thumbnail" class="object-cover w-full h-48 aspect-video rounded-t-2xl">
+                <img src="{{ $post->thumbnail }}" alt="Post Thumbnail"
+                    class="object-cover w-full h-48 aspect-video rounded-t-2xl">
                 <div class="p-4">
                     <h2 class="mb-2 text-xl font-bold line-clamp-1" itemprop="name">{{ $post->title }}</h2>
                     <p class="text-sm line-clamp-3" itemprop="headline">{{ $post->description }}</p>
                 </div>
-                <a href="{{ route('posts.show', $post->slug)}}" itemprop="url"
-                   class="absolute text-sm text-orange-400 bottom-4 right-4 hover:text-orange-600">Read more</a>
+                <a href="{{ route('posts.show', $post->slug) }}" itemprop="url"
+                    class="absolute text-sm text-orange-400 bottom-4 right-4 hover:text-orange-600">Read more</a>
             </article>
         @endforeach
     </div>