Merge fixes from master for release v0.13.1

app/Http/Controllers/Auth/LoginController.php

@@ -2,6 +2,7 @@
 
 namespace BookStack\Http\Controllers\Auth;
 
+use BookStack\Exceptions\AuthException;
 use BookStack\Http\Controllers\Controller;
 use BookStack\Repos\UserRepo;
 use BookStack\Services\SocialAuthService;

app/Http/Controllers/Auth/RegisterController.php

@@ -51,7 +51,7 @@ class RegisterController extends Controller
      */
     public function __construct(SocialAuthService $socialAuthService, EmailConfirmationService $emailConfirmationService, UserRepo $userRepo)
     {
-        $this->middleware('guest');
+        $this->middleware('guest')->except(['socialCallback', 'detachSocialAccount']);
         $this->socialAuthService = $socialAuthService;
         $this->emailConfirmationService = $emailConfirmationService;
         $this->userRepo = $userRepo;
@@ -297,5 +297,4 @@ class RegisterController extends Controller
         return $this->registerUser($userData, $socialAccount);
     }
 
-
 }

resources/assets/sass/_blocks.scss

@@ -135,7 +135,10 @@
   border-left: 3px solid #BBB;
   background-color: #EEE;
   padding: $-s;
-  display: flex;
+  display: block;
+  > * {
+    display: inline-block;
+  }
   &:before {
     font-family: 'Material-Design-Iconic-Font';
     padding-right: $-s;

resources/assets/sass/_lists.scss

@@ -110,6 +110,8 @@
   border-left: 0px solid #FFF;
   background-color: #FFF;
   &.fixed {
+    background-color: #FFF;
+    z-index: 5;
     position: fixed;
     top: 0;
     padding-left: $-l;

resources/assets/sass/_pages.scss

@@ -248,29 +248,32 @@
 }
 
 .tag-display {
-  margin: $-xl $-m;
-  border: 1px solid #DDD;
-  min-width: 180px;
-  max-width: 320px;
-  opacity: 0.7;
-  z-index: 5;
+  width: 100%;
+  //opacity: 0.7;
   position: relative;
   table {
     width: 100%;
     margin: 0;
     padding: 0;
   }
+  tr:first-child td {
+    padding-top: 0;
+  }
   .heading th {
     padding: $-xs $-s;
-    color: #333;
+    color: rgba(100, 100, 100, 0.7);
+    border: 0;
     font-weight: 400;
   }
   td {
     border: 0;
-    border-bottom: 1px solid #DDD;
+    border-bottom: 1px solid #EEE;
     padding: $-xs $-s;
     color: #444;
   }
+  tr td:first-child {
+    padding-left:0;
+  }
   .tag-value {
     color: #888;
   }

resources/views/pages/page-display.blade.php

@@ -2,26 +2,6 @@
 
     <h1 id="bkmrk-page-title" class="float left">{{$page->name}}</h1>
 
-    @if(count($page->tags) > 0)
-        <div class="tag-display float right">
-            <table>
-                <thead>
-                    <tr class="text-left heading primary-background-light">
-                        <th colspan="2">Page Tags</th>
-                    </tr>
-                </thead>
-                <tbody>
-                    @foreach($page->tags as $tag)
-                        <tr class="tag">
-                            <td @if(!$tag->value) colspan="2" @endif><a href="{{ baseUrl('/search/all?term=%5B' . urlencode($tag->name) .'%5D') }}">{{ $tag->name }}</a></td>
-                            @if($tag->value) <td class="tag-value"><a href="{{ baseUrl('/search/all?term=%5B' . urlencode($tag->name) .'%3D' . urlencode($tag->value) . '%5D') }}">{{$tag->value}}</a></td> @endif
-                        </tr>
-                    @endforeach
-                </tbody>
-            </table>
-        </div>
-    @endif
-
     <div style="clear:left;"></div>
 
     @if (isset($diff) && $diff)

resources/views/pages/show.blade.php

@@ -115,6 +115,8 @@
                     </div>
                 @endif
 
+
+
                 @include('pages/sidebar-tree-list', ['book' => $book, 'sidebarTree' => $sidebarTree, 'pageNav' => $pageNav])
             </div>
 

resources/views/pages/sidebar-tree-list.blade.php

@@ -1,6 +1,22 @@
 
 <div class="book-tree" ng-non-bindable>
 
+    @if(isset($page) && $page->tags->count() > 0)
+        <div class="tag-display">
+            <h6 class="text-muted">Page Tags</h6>
+            <table>
+                <tbody>
+                @foreach($page->tags as $tag)
+                    <tr class="tag">
+                        <td @if(!$tag->value) colspan="2" @endif><a href="{{ baseUrl('/search/all?term=%5B' . urlencode($tag->name) .'%5D') }}">{{ $tag->name }}</a></td>
+                        @if($tag->value) <td class="tag-value"><a href="{{ baseUrl('/search/all?term=%5B' . urlencode($tag->name) .'%3D' . urlencode($tag->value) . '%5D') }}">{{$tag->value}}</a></td> @endif
+                    </tr>
+                @endforeach
+                </tbody>
+            </table>
+        </div>
+    @endif
+
     @if (isset($page) && $page->attachments->count() > 0)
         <h6 class="text-muted">Attachments</h6>
         @foreach($page->attachments as $attachment)

routes/web.php

@@ -153,7 +153,7 @@ Route::group(['middleware' => 'auth'], function () {
 });
 
 // Social auth routes
-Route::get('/login/service/{socialDriver}', 'Auth\RegisterController@getSocialLogin');
+Route::get('/login/service/{socialDriver}', 'Auth\LoginController@getSocialLogin');
 Route::get('/login/service/{socialDriver}/callback', 'Auth\RegisterController@socialCallback');
 Route::get('/login/service/{socialDriver}/detach', 'Auth\RegisterController@detachSocialAccount');
 Route::get('/register/service/{socialDriver}', 'Auth\RegisterController@socialRegister');