Update book form so cancel URL is explicitly passed in

- Added to prevent future possibility of 'shelf' var being introduced in
scope and therefore causing a side-effect of redirect logic.

diff --git a/resources/views/books/create.blade.php b/resources/views/books/create.blade.php
index 6de81cd462db648bcf11e294eff640c1c67c6f91..db3e90e51913a8d6870417a1687984107a050ec7 100644 (file)
--- a/resources/views/books/create.blade.php
+++ b/resources/views/books/create.blade.php
@@ -28,7 +28,7 @@
         <main class="content-wrap card">
             <h1 class="list-heading">{{ trans('entities.books_create') }}</h1>
             <form action="{{ isset($bookshelf) ? $bookshelf->getUrl('/create-book') : url('/books') }}" method="POST" enctype="multipart/form-data">
-                @include('books.form')
+                @include('books.form', ['returnLocation' => isset($bookshelf) ? $bookshelf->getUrl() : url('/books')])
             </form>
         </main>
     </div>

diff --git a/resources/views/books/edit.blade.php b/resources/views/books/edit.blade.php
index 400fd6e817a0a3dc6bdd5d7289abf59c0386f0c8..ac11b58e201df206b3a8cc4b58d8422a16606877 100644 (file)
--- a/resources/views/books/edit.blade.php
+++ b/resources/views/books/edit.blade.php
@@ -18,7 +18,7 @@
             <h1 class="list-heading">{{ trans('entities.books_edit') }}</h1>
             <form action="{{ $book->getUrl() }}" method="POST" enctype="multipart/form-data">
                 <input type="hidden" name="_method" value="PUT">
-                @include('books.form', ['model' => $book])
+                @include('books.form', ['model' => $book, 'returnLocation' => $book->getUrl()])
             </form>
         </main>
     </div>

diff --git a/resources/views/books/form.blade.php b/resources/views/books/form.blade.php
index 91a3899de34f9d06e21015d840c85540e195046b..a3235036e02c81c76bfefa799787d3cfd27bf7e3 100644 (file)
--- a/resources/views/books/form.blade.php
+++ b/resources/views/books/form.blade.php
@@ -31,20 +31,11 @@
         <label for="tag-manager">{{ trans('entities.book_tags') }}</label>
     </button>
     <div class="collapse-content" collapsible-content>
-        @include('components.tag-manager', ['entity' => isset($book)?$book:null, 'entityType' => 'chapter'])
+        @include('components.tag-manager', ['entity' => $book ?? null, 'entityType' => 'chapter'])
     </div>
 </div>
 
 <div class="form-group text-right">
-    <?php
-        if (isset($bookshelf)) {
-            $cancelUrl = $bookshelf->getUrl();
-        } else if (isset($book)) {
-            $cancelUrl = $book->getUrl();
-        } else {
-            $cancelUrl = '/books';
-        }
-    ?>
-    <a href="{{ $cancelUrl }}" class="button outline">{{ trans('common.cancel') }}</a>
+    <a href="{{ $returnLocation }}" class="button outline">{{ trans('common.cancel') }}</a>
     <button type="submit" class="button">{{ trans('entities.books_save') }}</button>
 </div>
\ No newline at end of file