Use strings consistently for database/api/gpx status

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 8d9ee11f94929360b0b3ca7728ed1eddfbfd8741..fb561423640b723372f63f6f4d864b88225577ff 100644 (file)
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -147,14 +147,14 @@ class ApplicationController < ActionController::Base
   end
 
   def check_api_readable
   end
 
   def check_api_readable

-    if api_status == :offline
+    if api_status == "offline"

       report_error "Database offline for maintenance", :service_unavailable
       false
     end
   end
 
   def check_api_writable
       report_error "Database offline for maintenance", :service_unavailable
       false
     end
   end
 
   def check_api_writable

-    unless api_status == :online
+    unless api_status == "online"

       report_error "Database offline for maintenance", :service_unavailable
       false
     end
       report_error "Database offline for maintenance", :service_unavailable
       false
     end


@@ -162,21 +162,21 @@ class ApplicationController < ActionController::Base
 
   def database_status
     if Settings.status == "database_offline"
 
   def database_status
     if Settings.status == "database_offline"

-      :offline
+      "offline"

     elsif Settings.status == "database_readonly"
     elsif Settings.status == "database_readonly"

-      :readonly
+      "readonly"

     else
     else

-      :online
+      "online"

     end
   end
 
   def api_status
     status = database_status
     end
   end
 
   def api_status
     status = database_status

-    if status == :online
+    if status == "online"

       if Settings.status == "api_offline"
       if Settings.status == "api_offline"

-        status = :offline
+        status = "offline"

       elsif Settings.status == "api_readonly"
       elsif Settings.status == "api_readonly"

-        status = :readonly
+        status = "readonly"

       end
     end
     status
       end
     end
     status


@@ -184,7 +184,7 @@ class ApplicationController < ActionController::Base
 
   def gpx_status
     status = database_status
 
   def gpx_status
     status = database_status

-    status = :offline if status == :online && Settings.status == "gpx_offline"
+    status = "offline" if status == "online" && Settings.status == "gpx_offline"

     status
   end
 
     status
   end