From 93928aedd509f9b6f328c1886bcf0f7cb4fd66a0 Mon Sep 17 00:00:00 2001
From: Zin Moe <zin.moe@code2lab.com>
Date: Tue, 14 Jan 2020 17:23:52 +0630
Subject: [PATCH] change cloumn auth_token to app_token in employees

---
 app/controllers/settings/employees_controller.rb     |  6 +++---
 app/models/employee.rb                               | 12 +++++++-----
 app/views/settings/employees/show.html.erb           | 10 +++++-----
 config/locales/en.yml                                |  4 ++--
 config/routes.rb                                     |  2 +-
 ...14103715_change_column_auth_token_to_app_token.rb |  5 +++++
 6 files changed, 23 insertions(+), 16 deletions(-)
 create mode 100644 db/migrate/20200114103715_change_column_auth_token_to_app_token.rb

diff --git a/app/controllers/settings/employees_controller.rb b/app/controllers/settings/employees_controller.rb
index fd5fd796..bcbec6ff 100755
--- a/app/controllers/settings/employees_controller.rb
+++ b/app/controllers/settings/employees_controller.rb
@@ -40,8 +40,8 @@ class Settings::EmployeesController < ApplicationController
     end
   end 
 
-  def change_auth_token
-    @employee.auth_token = @employee.generate_auth_token #SecureRandom.hex(10)
+  def change_app_token
+    @employee.app_token = @employee.generate_app_token #SecureRandom.hex(10)
     @employee.save
     flash[:notice] = 'Auth Token was successfully updated.' 
     redirect_to settings_employee_url(@employee)
@@ -78,6 +78,6 @@ class Settings::EmployeesController < ApplicationController
 
     # Never trust parameters from the scary internet, only allow the white list through.
     def employee_params
-      params.require(:employee).permit(:name, :role, :is_active, :emp_id, :password,:order_queue_station_id, :image_path, :app_id, :auth_token)
+      params.require(:employee).permit(:name, :role, :is_active, :emp_id, :password,:order_queue_station_id, :image_path, :app_id, :app_token)
     end
 end
diff --git a/app/models/employee.rb b/app/models/employee.rb
index 907aa9ad..9d18c6c6 100755
--- a/app/models/employee.rb
+++ b/app/models/employee.rb
@@ -12,7 +12,7 @@ class Employee < ApplicationRecord
   validates :emp_id, uniqueness: true, numericality: true, length: {in: 1..4}, allow_blank: true
   validates :password, numericality: true, length: {in: 3..9}, allow_blank: true
 
-  before_create :generate_app_id, :generate_auth_token #, if: Proc.new { self.role == "app" }
+  before_create :generate_app_id, :generate_app_token , if: Proc.new { self.role == "application" }
   # Employee Image Uploader
   mount_uploader :image_path, EmployeeImageUploader
 
@@ -93,13 +93,15 @@ class Employee < ApplicationRecord
   end
 
   def generate_app_id 
-    return if self.role != 'app'
+    # return if self.role != 'application'
     self.app_id = SecureRandom.urlsafe_base64(nil, false)
+    rescue ActiveRecord::RecordNotUnique
+      retry
   end
 
-  def generate_auth_token
-    return if self.role != 'app'
-    self.auth_token = SecureRandom.hex(10)
+  def generate_app_token
+    # return if self.role != 'application'
+    self.app_token = SecureRandom.hex(10)
     rescue ActiveRecord::RecordNotUnique
       retry
   end
diff --git a/app/views/settings/employees/show.html.erb b/app/views/settings/employees/show.html.erb
index b5508f1b..807dc54a 100755
--- a/app/views/settings/employees/show.html.erb
+++ b/app/views/settings/employees/show.html.erb
@@ -54,22 +54,22 @@
           <th><%= t("views.right_panel.detail.employee_photo") %></th>
           <td><%= image_tag @employee.image_path, :size => '200x200'%></td>
         </tr>
-        <% if @employee.role == 'app' %>
+        <% if @employee.role == 'application' %>
           <tr>
             <th><%= t("views.right_panel.detail.employee_app_id") %></th>
             <td><%= @employee.app_id %></td>
           </tr>
           <tr>
-            <th><%= t("views.right_panel.detail.employee_auth_token") %></th>
-            <td><%= @employee.auth_token %></td>
+            <th><%= t("views.right_panel.detail.employee_app_token") %></th>
+            <td><%= @employee.app_token %></td>
           </tr>
         <% end %>
         <tr>
           <th></th>
           <td>
             <%= link_to t("views.btn.edit"), edit_settings_employee_path(@employee),:class => 'btn btn-primary btn-lg waves-effect' %>
-            <% if @employee.role == 'app' %>
-              <%= link_to t("views.btn.change_auth_token"), settings_change_auth_token_url(id: @employee.id), class: 'btn btn-primary btn-lg waves-effect' %>
+            <% if @employee.role == 'application' %>
+              <%= link_to t("views.btn.change_app_token"), settings_change_app_token_url(id: @employee.id), class: 'btn btn-primary btn-lg waves-effect' %>
             <% end %>
             <%if @employee.role != "administrator"%>
             <button class="delete btn btn-danger btn-sm waves-effect" data-ref="<%=settings_employee_path(@employee)%>" data-method="delete">
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 68c7bc53..31781d4f 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -112,7 +112,7 @@ en:
       del: "DEL"
       clr: "CLR"
       assign: "ASSIGN"
-      change_auth_token: "Change Auth Token"
+      change_app_token: "Change App Token"
 
       print_order_summary: "Print Order Summary"
       memeber_card: "Member Card"
@@ -680,7 +680,7 @@ en:
         delete_btn_txt: "to delete "
         update_btn_txt: "to update "
         employee_app_id: "App ID"
-        employee_auth_token: "Auth Token"
+        employee_app_token: "App Token"
 
   helpers:
     page_entries_info:
diff --git a/config/routes.rb b/config/routes.rb
index 0ba5e788..b9f592d8 100755
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -433,7 +433,7 @@ scope "(:locale)", locale: /en|mm/ do
     end
     #employees
     resources :employees 
-    get '/:id/change_auth_token' => 'employees#change_auth_token', as: 'change_auth_token'
+    get '/:id/change_app_token' => 'employees#change_app_token', as: 'change_app_token'
 
     #menu
     resources :menus do
diff --git a/db/migrate/20200114103715_change_column_auth_token_to_app_token.rb b/db/migrate/20200114103715_change_column_auth_token_to_app_token.rb
new file mode 100644
index 00000000..92a20d72
--- /dev/null
+++ b/db/migrate/20200114103715_change_column_auth_token_to_app_token.rb
@@ -0,0 +1,5 @@
+class ChangeColumnAuthTokenToAppToken < ActiveRecord::Migration[5.1]
+  def change
+    rename_column :employees, :auth_token, :app_token
+  end
+end