Created Client::BaseController to hide layout and logged_in filters

This way increasing the chances of forgetting to ensure logging in.

rails_application/app/controllers/client/base_controller.rb

@@ -0,0 +1,16 @@
+module Client
+  class BaseController < ApplicationController
+
+    layout "client_panel"
+    before_action :ensure_logged_in
+
+    private
+
+    def ensure_logged_in
+      if ClientOrders::Client.find_by(uid: cookies[:client_id]).nil?
+        redirect_to logout_path
+        return
+      end
+    end
+  end
+end

rails_application/app/controllers/client/clients_controller.rb

@@ -1,6 +1,7 @@
 module Client
-  class ClientsController < ApplicationController
-    layout "client_panel"
+  class ClientsController < BaseController
+
+    skip_before_action :ensure_logged_in, only: [:index, :login, :logout]
 
     def index
       if cookies[:client_id]
@@ -31,6 +32,7 @@ def logout
       redirect_to clients_path
     end
 
+    private
 
     def correct_password?(client_id, password)
       password_hash = Digest::SHA256.hexdigest(password)

rails_application/app/controllers/client/orders_controller.rb

@@ -1,16 +1,5 @@
 module Client
-  class OrdersController < ApplicationController
-
-    layout 'client_panel'
-
-    before_action :ensure_logged_in
-
-    def ensure_logged_in
-      if ClientOrders::Client.find_by(uid: cookies[:client_id]).nil?
-        redirect_to logout_path
-        return
-      end
-    end
+  class OrdersController < BaseController
 
     def index
       render html: ClientOrders::OrdersList.build(view_context, cookies[:client_id]), layout: true

rails_application/app/controllers/client/products_controller.rb

@@ -1,6 +1,5 @@
 module Client
-  class ProductsController < ApplicationController
-    layout 'client_panel'
+  class ProductsController < BaseController
 
     def index
       render html: PublicOffer::ProductsList.build(view_context), layout: true