From fb18db918aa98b4d3313fcce019224bad2f577a8 Mon Sep 17 00:00:00 2001
From: Annika Backstrom <annika@sixohthree.com>
Date: Thu, 30 May 2024 21:55:08 +0100
Subject: [PATCH] Add "add bookmark" functionality

---
 app/Http/Controllers/BookmarkController.php | 28 +++++++++++----------
 app/Models/Bookmark.php                     | 14 +++++++++++
 resources/views/bookmarks/edit.blade.php    | 17 +++++++------
 resources/views/layouts/app.blade.php       |  1 +
 4 files changed, 39 insertions(+), 21 deletions(-)

diff --git a/app/Http/Controllers/BookmarkController.php b/app/Http/Controllers/BookmarkController.php
index 8bfe32e..5b37794 100644
--- a/app/Http/Controllers/BookmarkController.php
+++ b/app/Http/Controllers/BookmarkController.php
@@ -25,7 +25,7 @@ class BookmarkController extends Controller
      */
     public function create()
     {
-        //
+        return view('bookmarks.edit', ['bookmark' => null]);
     }
 
     /**
@@ -33,7 +33,19 @@ class BookmarkController extends Controller
      */
     public function store(Request $request)
     {
-        //
+        $bookmark = new Bookmark;
+        $bookmark->title = $request->post('title');
+        $bookmark->description = $request->post('description', '');
+        $bookmark->href = $request->post('href');
+        $bookmark->save();
+
+        $bookmark->syncTagsFromString($request->post('tags', ''));
+
+        return redirect()->action(
+            [self::class, "show"], [
+            "bookmark" => $bookmark,
+            ]
+        );
     }
 
     /**
@@ -55,7 +67,6 @@ class BookmarkController extends Controller
     {
         return view(
             'bookmarks.edit', [
-                'edit' => true,
                 'bookmark' => $bookmark,
             ]
         );
@@ -71,16 +82,7 @@ class BookmarkController extends Controller
         $bookmark->href = $request->post('href');
         $bookmark->save();
 
-        $tags_input = trim($request->post('tags', ''));
-        $tags_input = preg_split('/\s+/', $tags_input);
-
-        $tags = [];
-        foreach ($tags_input as $tag_input) {
-            $tag = Tag::firstOrCreate(['name' => $tag_input]);
-            $tags[$tag->id] = true;
-        }
-
-        $bookmark->tags()->sync(array_keys($tags));
+        $bookmark->syncTagsFromString($request->post('tags', ''));
 
         return redirect()->action(
             [self::class, "show"], [
diff --git a/app/Models/Bookmark.php b/app/Models/Bookmark.php
index ebc2078..31e25ee 100644
--- a/app/Models/Bookmark.php
+++ b/app/Models/Bookmark.php
@@ -15,6 +15,20 @@ class Bookmark extends Model
 
     protected $with = ['tags'];
 
+    public function syncTagsFromString(string $tags_string): array
+    {
+        $tags_input = trim($tags_string);
+        $tags_input = preg_split('/\s+/', $tags_input);
+
+        $tags = [];
+        foreach ($tags_input as $tag_input) {
+            $tag = Tag::firstOrCreate(['name' => $tag_input]);
+            $tags[$tag->id] = true;
+        }
+
+        return $this->tags()->sync(array_keys($tags));
+    }
+
     public function tags(): BelongsToMany
     {
         return $this->belongsToMany(Tag::class);
diff --git a/resources/views/bookmarks/edit.blade.php b/resources/views/bookmarks/edit.blade.php
index 12bfc27..5c3fd1d 100644
--- a/resources/views/bookmarks/edit.blade.php
+++ b/resources/views/bookmarks/edit.blade.php
@@ -1,39 +1,40 @@
 @extends('layouts.app')
 
 @section('content')
+    <h2>{{ $bookmark ? "Edit Bookmark" : "Add Bookmark" }}</h2>
     <p>
         <a href="{{ action('BookmarkController@index') }}">&larr; Back</a>
     </p>
-    <form method="post" class="form" action="{{ action('BookmarkController@update', ['bookmark' => $bookmark]) }}">
+    <form method="post" class="form" action="{{ $bookmark ? action('BookmarkController@update', ['bookmark' => $bookmark]) : action('BookmarkController@store') }}">
         @csrf
         <div class="bookmark">
-            <div class="bookmark-title form-row"g>
+            <div class="bookmark-title form-row">
                 <label class="form-label" for="title">
                     Title:
                 </label>
-                <input type="text" class="form-input" name="title" value="{{ $bookmark->title }}">
+                <input type="text" class="form-input" name="title" value="{{ $bookmark?->title }}" tabindex="1">
             </div>
             <div class="bookmark-href form-row form-row-stacked">
                 <label class="form-label" for="href">
                     URL:
                 </label>
-                <input type="text" class="form-input" name="href" value="{{ $bookmark->href }}">
+                <input type="text" class="form-input" name="href" value="{{ $bookmark?->href }}" tabindex="3">
             </div>
             <div class="bookmark-description form-row form-row-stacked">
                 <label class="form-label" for="description">
                     Description:
                 </label>
-                <textarea name="description" class="form-textarea">{{ $bookmark->description }}</textarea>
+                <textarea name="description" class="form-textarea" tabindex="2">{{ $bookmark?->description }}</textarea>
             </div>
             <div class="bookmark-tags form-row form-row-stacked">
                 <label class="form-label" for="description">
                     Tags:
                 </label>
-                <input type="text" class="form-input" name="tags" value="@foreach ($bookmark->tags as $tag){{ $tag->name }} @endforeach">
+                <input type="text" class="form-input" name="tags" tabindex="4" value="@foreach ($bookmark?->tags ?? [] as $tag){{ $tag->name }} @endforeach">
             </div>
             <div class="form-row align-right bookmark-actions">
-                <a class="form-button" href="{{ action('BookmarkController@show', ['bookmark' => $bookmark]) }}">Cancel</a>
-                <input type="submit" class="form-button" value="Save">
+                <a class="form-button" tabindex="5" href="{{ $bookmark ? action('BookmarkController@show', ['bookmark' => $bookmark]) : action('BookmarkController@index') }}">Cancel</a>
+                <input type="submit" tabindex="6" class="form-button" value="Save">
             </div>
         </div>
     </form>
diff --git a/resources/views/layouts/app.blade.php b/resources/views/layouts/app.blade.php
index 96e6cdc..021c494 100644
--- a/resources/views/layouts/app.blade.php
+++ b/resources/views/layouts/app.blade.php
@@ -8,6 +8,7 @@
     <div class="container">
         <header>
             <h1><a href="{{ url("/") }}">url snail</a></h1>
+            <a href="{{ action('BookmarkController@create') }}">+</a>
         </header>
         @yield('content')
     </div>