[refactor] use human names for commands

jnewland · Oct 7, 2013 · add22b0 · add22b0
1 parent 4085ad9
commit add22b0
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/lib/nagiosharder.rb b/lib/nagiosharder.rb
@@ -58,7 +58,7 @@ def post_command(body)
 
     def acknowledge_service(host, service, comment)
       request = {
-        :cmd_typ => 34,
+        :cmd_typ => COMMANDS[:acknowledge_service_problem],
         :cmd_mod => 2,
         :com_author => @user,
         :com_data => comment,
@@ -74,7 +74,7 @@ def acknowledge_service(host, service, comment)
 
     def acknowledge_host(host, comment)
       request = {
-        :cmd_typ => 33,
+        :cmd_typ => COMMANDS[:acknowledge_host_problem],
         :cmd_mod => 2,
         :com_author => @user,
         :com_data => comment,
@@ -89,7 +89,7 @@ def acknowledge_host(host, comment)
 
     def unacknowledge_service(host, service)
       request = {
-        :cmd_typ => 52,
+        :cmd_typ => COMMANDS[:remove_service_acknowledgement],
         :cmd_mod => 2,
         :host => host,
         :service => service
@@ -101,7 +101,7 @@ def unacknowledge_service(host, service)
     def schedule_service_downtime(host, service, options = {})
       request = {
         :cmd_mod => 2,
-        :cmd_typ => 56,
+        :cmd_typ => COMMANDS[:schedule_service_downtime],
         :com_author => options[:author] || "#{@user} via nagiosharder",
         :com_data => options[:comment] || 'scheduled downtime by nagiosharder',
         :host => host,
@@ -130,7 +130,7 @@ def schedule_service_downtime(host, service, options = {})
     def schedule_host_downtime(host, options = {})
       request = {
         :cmd_mod => 2,
-        :cmd_typ => 55,
+        :cmd_typ => COMMANDS[:schedule_host_downtime],
         :com_author => options[:author] || "#{@user} via nagiosharder",
         :com_data => options[:comment] || 'scheduled downtime by nagiosharder',
         :host => host,
@@ -184,7 +184,7 @@ def schedule_service_check(host, service)
         :host => host,
         :service => service,
         :force_check => true,
-        :cmd_typ => 7,
+        :cmd_typ => COMMANDS[:schedule_service_check],
         :cmd_mod => 2
       }
       post_command(request)
@@ -296,10 +296,10 @@ def disable_service_notifications(host, service, options = {})
       }
 
       if service
-        request[:cmd_typ] = 23
+        request[:cmd_typ] = COMMANDS[:disable_service_notifications]
         request[:service] = service
       else
-        request[:cmd_typ] = 29
+        request[:cmd_typ] = COMMANDS[:disable_host_service_checks]
         request[:ahas] = true
       end
 
@@ -314,10 +314,10 @@ def enable_service_notifications(host, service, options = {})
       }
 
       if service
-        request[:cmd_typ] = 22
+        request[:cmd_typ] = COMMANDS[:enable_service_notifications]
         request[:service] = service
       else
-        request[:cmd_typ] = 28
+        request[:cmd_typ] = COMMANDS[:enable_host_service_notifications]
         request[:ahas] = true
       end