diff --git a/app/Http/Controllers/BookController.php b/app/Http/Controllers/BookController.php index 7f8bada8c..0092a78a9 100644 --- a/app/Http/Controllers/BookController.php +++ b/app/Http/Controllers/BookController.php @@ -9,6 +9,7 @@ use Illuminate\Support\Facades\Auth; use Illuminate\Support\Str; use Oxbow\Http\Requests; use Oxbow\Repos\BookRepo; +use Oxbow\Repos\ChapterRepo; use Oxbow\Repos\PageRepo; class BookController extends Controller @@ -16,16 +17,19 @@ class BookController extends Controller protected $bookRepo; protected $pageRepo; + protected $chapterRepo; /** * BookController constructor. - * @param BookRepo $bookRepo - * @param PageRepo $pageRepo + * @param BookRepo $bookRepo + * @param PageRepo $pageRepo + * @param ChapterRepo $chapterRepo */ - public function __construct(BookRepo $bookRepo, PageRepo $pageRepo) + public function __construct(BookRepo $bookRepo, PageRepo $pageRepo, ChapterRepo $chapterRepo) { $this->bookRepo = $bookRepo; $this->pageRepo = $pageRepo; + $this->chapterRepo = $chapterRepo; parent::__construct(); } @@ -133,6 +137,71 @@ class BookController extends Controller return view('books/delete', ['book' => $book, 'current' => $book]); } + /** + * Shows the view which allows pages to be re-ordered and sorted. + * @param string $bookSlug + * @return \Illuminate\View\View + */ + public function sort($bookSlug) + { + $this->checkPermission('book-update'); + $book = $this->bookRepo->getBySlug($bookSlug); + $books = $this->bookRepo->getAll(); + return view('books/sort', ['book' => $book, 'current' => $book, 'books' => $books]); + } + + public function getSortItem($bookSlug) + { + $book = $this->bookRepo->getBySlug($bookSlug); + return view('books/sort-box', ['book' => $book]); + } + + /** + * Saves an array of sort mapping to pages and chapters. + * + * @param string $bookSlug + * @param Request $request + * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector + */ + public function saveSort($bookSlug, Request $request) + { + $this->checkPermission('book-update'); + $book = $this->bookRepo->getBySlug($bookSlug); + + // Return if no map sent + if (!$request->has('sort-tree')) { + return redirect($book->getUrl()); + } + + $sortedBooks = []; + // Sort pages and chapters + $sortMap = json_decode($request->get('sort-tree')); + $defaultBookId = $book->id; + foreach ($sortMap as $index => $bookChild) { + $id = $bookChild->id; + $isPage = $bookChild->type == 'page'; + $bookId = $this->bookRepo->exists($bookChild->book) ? $bookChild->book : $defaultBookId; + $model = $isPage ? $this->pageRepo->getById($id) : $this->chapterRepo->getById($id); + $isPage ? $this->pageRepo->setBookId($bookId, $model) : $this->chapterRepo->setBookId($bookId, $model); + $model->priority = $index; + if ($isPage) { + $model->chapter_id = ($bookChild->parentChapter === false) ? 0 : $bookChild->parentChapter; + } + $model->save(); + if (!in_array($bookId, $sortedBooks)) { + $sortedBooks[] = $bookId; + } + } + + // Add activity for books + foreach ($sortedBooks as $bookId) { + $updatedBook = $this->bookRepo->getById($bookId); + Activity::add($updatedBook, 'book_sort', $updatedBook->id); + } + + return redirect($book->getUrl()); + } + /** * Remove the specified book from storage. * diff --git a/app/Http/Controllers/PageController.php b/app/Http/Controllers/PageController.php index 4bcdbb80b..c0ffa14b8 100644 --- a/app/Http/Controllers/PageController.php +++ b/app/Http/Controllers/PageController.php @@ -142,50 +142,6 @@ class PageController extends Controller return redirect($page->getUrl()); } - /** - * Shows the view which allows pages to be re-ordered and sorted. - * @param $bookSlug - * @return \Illuminate\View\View - */ - public function sortPages($bookSlug) - { - $this->checkPermission('book-update'); - $book = $this->bookRepo->getBySlug($bookSlug); - return view('pages/sort', ['book' => $book, 'current' => $book]); - } - - /** - * Saves an array of sort mapping to pages and chapters. - * - * @param $bookSlug - * @param Request $request - * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector - */ - public function savePageSort($bookSlug, Request $request) - { - $this->checkPermission('book-update'); - $book = $this->bookRepo->getBySlug($bookSlug); - // Return if no map sent - if (!$request->has('sort-tree')) { - return redirect($book->getUrl()); - } - - // Sort pages and chapters - $sortMap = json_decode($request->get('sort-tree')); - foreach ($sortMap as $index => $bookChild) { - $id = $bookChild->id; - $isPage = $bookChild->type == 'page'; - $model = $isPage ? $this->pageRepo->getById($id) : $this->chapterRepo->getById($id); - $model->priority = $index; - if ($isPage) { - $model->chapter_id = ($bookChild->parentChapter === false) ? 0 : $bookChild->parentChapter; - } - $model->save(); - } - Activity::add($book, 'book_sort', $book->id); - return redirect($book->getUrl()); - } - /** * Show the deletion page for the specified page. * @param $bookSlug diff --git a/app/Http/routes.php b/app/Http/routes.php index 07da97ae7..9681ad1f1 100644 --- a/app/Http/routes.php +++ b/app/Http/routes.php @@ -12,14 +12,16 @@ Route::group(['middleware' => 'auth'], function () { Route::get('/{slug}/edit', 'BookController@edit'); Route::put('/{slug}', 'BookController@update'); Route::delete('/{id}', 'BookController@destroy'); + Route::get('/{slug}/sort-item', 'BookController@getSortItem'); Route::get('/{slug}', 'BookController@show'); Route::get('/{slug}/delete', 'BookController@showDelete'); + Route::get('/{bookSlug}/sort', 'BookController@sort'); + Route::put('/{bookSlug}/sort', 'BookController@saveSort'); + // Pages Route::get('/{bookSlug}/page/create', 'PageController@create'); Route::post('/{bookSlug}/page', 'PageController@store'); - Route::get('/{bookSlug}/sort', 'PageController@sortPages'); - Route::put('/{bookSlug}/sort', 'PageController@savePageSort'); Route::get('/{bookSlug}/page/{pageSlug}', 'PageController@show'); Route::get('/{bookSlug}/page/{pageSlug}/edit', 'PageController@edit'); Route::get('/{bookSlug}/page/{pageSlug}/delete', 'PageController@showDelete'); diff --git a/app/Repos/BookRepo.php b/app/Repos/BookRepo.php index 5a5da5138..88b3a8b1e 100644 --- a/app/Repos/BookRepo.php +++ b/app/Repos/BookRepo.php @@ -35,6 +35,16 @@ class BookRepo return $this->book->where('slug', '=', $slug)->first(); } + /** + * Checks if a book exists. + * @param $id + * @return bool + */ + public function exists($id) + { + return $this->book->where('id', '=', $id)->exists(); + } + /** * Get a new book instance from request input. * @param $input diff --git a/app/Repos/ChapterRepo.php b/app/Repos/ChapterRepo.php index b423ee8ba..9c03f29b5 100644 --- a/app/Repos/ChapterRepo.php +++ b/app/Repos/ChapterRepo.php @@ -80,4 +80,15 @@ class ChapterRepo return $chapters; } + public function setBookId($bookId, Chapter $chapter) + { + $chapter->book_id = $bookId; + foreach($chapter->activity as $activity) { + $activity->book_id = $bookId; + $activity->save(); + } + $chapter->save(); + return $chapter; + } + } \ No newline at end of file diff --git a/app/Repos/PageRepo.php b/app/Repos/PageRepo.php index d150f6268..7dfc1976a 100644 --- a/app/Repos/PageRepo.php +++ b/app/Repos/PageRepo.php @@ -186,6 +186,17 @@ class PageRepo return $query->count() > 0; } + public function setBookId($bookId, Page $page) + { + $page->book_id = $bookId; + foreach($page->activity as $activity) { + $activity->book_id = $bookId; + $activity->save(); + } + $page->save(); + return $page; + } + /** * Gets a suitable slug for the resource * diff --git a/resources/assets/js/global.js b/resources/assets/js/global.js index d078946c5..a75a4dc74 100644 --- a/resources/assets/js/global.js +++ b/resources/assets/js/global.js @@ -13,7 +13,7 @@ $(function () { $('.chapter-toggle').click(function(e) { e.preventDefault(); $(this).toggleClass('open'); - $(this).closest('.book-child').find('.inset-list').slideToggle(180); + $(this).closest('.chapter').find('.inset-list').slideToggle(180); }); }); \ No newline at end of file diff --git a/resources/assets/sass/_lists.scss b/resources/assets/sass/_lists.scss index 2f75ea342..f06e647ba 100644 --- a/resources/assets/sass/_lists.scss +++ b/resources/assets/sass/_lists.scss @@ -37,6 +37,7 @@ cursor: pointer; margin: 0 0 $-l 0; transition: all ease-in-out 180ms; + user-select: none; i { transition: all ease-in-out 180ms; transform: rotate(0deg); @@ -154,10 +155,18 @@ // Sortable Lists .sortable-page-list, .sortable-page-list ul { list-style: none; + background-color: #FFF; +} +.sort-box { + margin-bottom: $-m; + padding: 0 $-l 0 $-l; + border-left: 4px solid $color-book; } .sortable-page-list { margin-left: 0; - box-shadow: 0 0 1px 0 rgba(0, 0, 0, 0.1); + > ul { + margin-left: 0; + } ul { margin-bottom: 0; margin-top: 0; diff --git a/resources/assets/sass/_tables.scss b/resources/assets/sass/_tables.scss index fe01976ec..031131158 100644 --- a/resources/assets/sass/_tables.scss +++ b/resources/assets/sass/_tables.scss @@ -16,4 +16,8 @@ table.table { tr:hover { background-color: #EEE; } +} + +table { + max-width: 100%; } \ No newline at end of file diff --git a/resources/views/books/sort-box.blade.php b/resources/views/books/sort-box.blade.php new file mode 100644 index 000000000..1af2d1a42 --- /dev/null +++ b/resources/views/books/sort-box.blade.php @@ -0,0 +1,20 @@ +