zabbix_manager 5.0.4 → 5.0.7

data/lib/zabbix_manager/classes/roles.rb

@@ -1,24 +1,26 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Roles < Basic
     # The method name used for interacting with Role via Zabbix API
     #
     # @return [String]
     def method_name
-      'role'
+      "role"
     end
 
     # The key field name used for Role objects via Zabbix API
     #
     # @return [String]
     def key
-      'roleid'
+      "roleid"
     end
 
     # The id field name used for identifying specific Role objects via Zabbix API
     #
     # @return [String]
     def identify
-      'name'
+      "name"
     end
 
     # Set permissions for usergroup using Zabbix API
@@ -30,13 +32,13 @@ class ZabbixManager
     def rules(data)
       rules = data[:rules] || 2
       result = @client.api_request(
-        method: 'role.update',
+        method: "role.update",
         params: {
           roleid: data[:roleid],
           rules: data[:hostgroupids].map { |t| { permission: permission, id: t } }
         }
       )
-      result ? result['usrgrpids'][0].to_i : nil
+      result ? result["usrgrpids"][0].to_i : nil
     end
 
     # Add users to usergroup using Zabbix API
@@ -60,10 +62,10 @@ class ZabbixManager
       log "[DEBUG] Call dump_by_id with parameters: #{data.inspect}"
 
       @client.api_request(
-        method: 'role.get',
+        method: "role.get",
         params: {
-          output: 'extend',
-          selectRules: 'extend',
+          output: "extend",
+          selectRules: "extend",
           roleids: data[:id]
         }
       )
@@ -77,18 +79,18 @@ class ZabbixManager
     # @return [Array] Returns array of Graph ids
     def get_ids_by_name(data)
       result = @client.api_request(
-        method: 'role.get',
+        method: "role.get",
         params: {
           filter: {
             name: data[:name]
           },
-          output: 'extend'
+          output: "extend"
         }
       )
 
-      result.map do |rule|
-        rule['roleid']
-      end.compact
+      result.filter_map do |rule|
+        rule["roleid"]
+      end
     end
 
     # Update users in Userroles using Zabbix API
@@ -101,14 +103,14 @@ class ZabbixManager
       user_groups = data[:usrgrpids].map do |t|
         {
           usrgrpid: t,
-          userids: data[:userids],
+          userids: data[:userids]
         }
       end
       result = @client.api_request(
-        method: 'usergroup.update',
-        params: user_groups,
+        method: "usergroup.update",
+        params: user_groups
       )
-      result ? result['usrgrpids'][0].to_i : nil
+      result ? result["usrgrpids"][0].to_i : nil
     end
   end
 end

data/lib/zabbix_manager/classes/screens.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Screens < Basic
     # extracted from frontends/php/include/defines.inc.php
@@ -23,14 +25,14 @@ class ZabbixManager
     #
     # @return [String]
     def method_name
-      'screen'
+      "screen"
     end
 
     # The id field name used for identifying specific Screen objects via Zabbix API
     #
     # @return [String]
     def identify
-      'name'
+      "name"
     end
 
     # Delete Screen object using Zabbix API
@@ -40,8 +42,8 @@ class ZabbixManager
     # @raise [HttpError] Error raised when HTTP status from Zabbix Server response is not a 200 OK.
     # @return [Integer] Zabbix object id
     def delete(data)
-      result = @client.api_request(method: 'screen.delete', params: [data])
-      result.empty? ? nil : result['screenids'][0].to_i
+      result = @client.api_request(method: "screen.delete", params: [data])
+      result.empty? ? nil : result["screenids"][0].to_i
     end
 
     # Get or Create Screen object for Host using Zabbix API

data/lib/zabbix_manager/classes/scripts.rb

@@ -1,14 +1,16 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Scripts < Basic
     def method_name
-      'script'
+      "script"
     end
 
     # The id field name used for identifying specific Screen objects via Zabbix API
     #
     # @return [String]
     def identify
-      'name'
+      "name"
     end
 
     # Submits a request to the zabbix api
@@ -20,7 +22,7 @@ class ZabbixManager
     # Returns nothing
     def execute(data)
       @client.api_request(
-        method: 'script.execute',
+        method: "script.execute",
         params: {
           scriptid: data[:scriptid],
           hostid: data[:hostid]
@@ -29,7 +31,7 @@ class ZabbixManager
     end
 
     def getscriptsbyhost(data)
-      @client.api_request(method: 'script.getscriptsbyhosts', params: data)
+      @client.api_request(method: "script.getscriptsbyhosts", params: data)
     end
   end
 end

data/lib/zabbix_manager/classes/server.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Server
     # @return [String]
@@ -10,7 +12,7 @@ class ZabbixManager
     # @return [String] Zabbix API version number
     def initialize(client)
       @client = client
-      @version = @client.api_version()
+      @version = @client.api_version
     end
   end
 end

data/lib/zabbix_manager/classes/templates.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Templates < Basic
     # The method name used for interacting with Templates via Zabbix API
     #
     # @return [String]
     def method_name
-      'template'
+      "template"
     end
 
     # The id field name used for identifying specific Template objects via Zabbix API
     #
     # @return [String]
     def identify
-      'host'
+      "host"
     end
 
     # Delete Template object using Zabbix API
@@ -21,8 +23,8 @@ class ZabbixManager
     # @raise [HttpError] Error raised when HTTP status from Zabbix Server response is not a 200 OK.
     # @return [Integer] The Template object id that was deleted
     def delete(data)
-      result = @client.api_request(method: 'template.delete', params: [data])
-      result.empty? ? nil : result['templateids'][0].to_i
+      result = @client.api_request(method: "template.delete", params: [data])
+      result.empty? ? nil : result["templateids"][0].to_i
     end
 
     # Get Template ids for Host from Zabbix API
@@ -32,8 +34,8 @@ class ZabbixManager
     # @raise [HttpError] Error raised when HTTP status from Zabbix Server response is not a 200 OK.
     # @return [Array] Returns array of Template ids
     def get_ids_by_host(data)
-      @client.api_request(method: 'template.get', params: data).map do |tmpl|
-        tmpl['templateid']
+      @client.api_request(method: "template.get", params: data).map do |tmpl|
+        tmpl["templateid"]
       end
     end
 
@@ -58,9 +60,9 @@ class ZabbixManager
     # @return [Boolean]
     def mass_update(data)
       result = @client.api_request(
-        method: 'template.massUpdate',
+        method: "template.massUpdate",
         params: {
-          hosts:     data[:hosts_id].map { |t| { hostid: t } },
+          hosts: data[:hosts_id].map { |t| { hostid: t } },
           templates: data[:templates_id].map { |t| { templateid: t } }
         }
       )
@@ -75,9 +77,9 @@ class ZabbixManager
     # @return [Boolean]
     def mass_add(data)
       result = @client.api_request(
-        method: 'template.massAdd',
+        method: "template.massAdd",
         params: {
-          hosts:     data[:hosts_id].map { |t| { hostid: t } },
+          hosts: data[:hosts_id].map { |t| { hostid: t } },
           templates: data[:templates_id].map { |t| { templateid: t } }
         }
       )
@@ -92,12 +94,12 @@ class ZabbixManager
     # @return [Boolean]
     def mass_remove(data)
       result = @client.api_request(
-        method: 'template.massRemove',
+        method: "template.massRemove",
         params: {
-          hostids:     data[:hosts_id],
+          hostids: data[:hosts_id],
           templateids: data[:templates_id],
-          groupids:    data[:group_id],
-          force:       1
+          groupids: data[:group_id],
+          force: 1
         }
       )
       result.empty? ? false : true
@@ -107,14 +109,14 @@ class ZabbixManager
     def get_template_ids(data)
       # 直接调后端接口
       result = @client.api_request(
-        method: 'template.get',
+        method: "template.get",
         params: {
           output: "extend",
           filter: {
             host: [data].flatten
           }
         }
-      ).map { |temp| [templateid: temp['templateid']]}
+      ).map { |temp| [templateid: temp["templateid"]] }
 
       # 返回 template 模板数组对象
       result.empty? ? nil : result.flatten

data/lib/zabbix_manager/classes/triggers.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Triggers < Basic
     # The method name used for interacting with Triggers via Zabbix API
     #
     # @return [String]
     def method_name
-      'trigger'
+      "trigger"
     end
 
     # The id field name used for identifying specific Trigger objects via Zabbix API
     #
     # @return [String]
     def identify
-      'description'
+      "description"
     end
 
     # Dump Trigger object data by key from Zabbix API
@@ -24,14 +26,14 @@ class ZabbixManager
       log "[DEBUG] Call dump_by_id with parameters: #{data.inspect}"
 
       @client.api_request(
-        method: 'trigger.get',
+        method: "trigger.get",
         params: {
-          filter:           {
+          filter: {
             keys.to_sym => data[keys.to_sym]
           },
-          output:           'extend',
-          select_items:     'extend',
-          select_functions: 'extend'
+          output: "extend",
+          select_items: "extend",
+          select_functions: "extend"
         }
       )
     end
@@ -51,13 +53,13 @@ class ZabbixManager
         dump = symbolize_keys(item) if item[key].to_i == data[key.to_sym].to_i
       end
 
-      expression        = dump[:items][0][:key_] + '.' + dump[:functions][0][:function] + '(' + dump[:functions][0][:parameter] + ')'
+      expression        = "#{dump[:items][0][:key_]}.#{dump[:functions][0][:function]}(#{dump[:functions][0][:parameter]})"
       dump[:expression] = dump[:expression].gsub(/\{(\d*)\}/, "{#{expression}}") # TODO: ugly regexp
       dump.delete(:functions)
       dump.delete(:items)
 
       old_expression    = data[:expression]
-      data[:expression] = data[:expression].gsub(/\{.*\:/, '{') # TODO: ugly regexp
+      data[:expression] = data[:expression].gsub(/\{.*:/, "{") # TODO: ugly regexp
       data.delete(:templateid)
 
       log "[DEBUG] expression: #{dump[:expression]}\n data: #{data[:expression]}"
@@ -68,7 +70,10 @@ class ZabbixManager
       else
         data[:expression] = old_expression
         # disable old trigger
-        log '[DEBUG] disable :' + @client.api_request(method: "#{method_name}.update", params: [{ triggerid: data[:triggerid], status: '1' }]).inspect
+        log "[DEBUG] disable :" + @client.api_request(method: "#{method_name}.update",
+                                                      params: [{
+                                                        triggerid: data[:triggerid], status: "1"
+                                                      }]).inspect
         # create new trigger
         data.delete(:triggerid)
         create(data)
@@ -105,16 +110,15 @@ class ZabbixManager
     # 测试数据
     def mojo_data
       data = {
-        comments:            "MOJO1",
-        opdata:              "MOJO_OPDATA",
-        priority:            1,
-        description:         "MOJO1",
-        expression:          "{SZX1-ISP-SW7:net.if.in[ifHCInOctets.1].avg(15m)}>=450000000 or {SZX1-ISP-SW7:net.if.out[ifHCOutOctets.1].avg(15m)}>=450000000",
-        recovery_expression: "{SZX1-ISP-SW7:net.if.in[ifHCInOctets.1].avg(15m)}<=350000000 or {SZX1-ISP-SW7:net.if.out[ifHCOutOctets.1].avg(15m)}<=350000000",
+        comments: "MOJO1",
+        opdata: "MOJO_OPDATA",
+        priority: 1,
+        description: "MOJO1",
+        expression: "{SZX1-ISP-SW7:net.if.in[ifHCInOctets.1].avg(15m)}>=450000000 or {SZX1-ISP-SW7:net.if.out[ifHCOutOctets.1].avg(15m)}>=450000000",
+        recovery_expression: "{SZX1-ISP-SW7:net.if.in[ifHCInOctets.1].avg(15m)}<=350000000 or {SZX1-ISP-SW7:net.if.out[ifHCOutOctets.1].avg(15m)}<=350000000"
       }
 
       create_trigger data.stringify_keys!
-
     end
 
     # 设置触发器
@@ -122,17 +126,17 @@ class ZabbixManager
       data = data.with_indifferent_access
       # 请求生成触发器
       result = @client.api_request(
-        method: 'trigger.create',
+        method: "trigger.create",
         params: {
-          comments:            data["comments"],
-          priority:            data["priority"],
-          description:         data["description"],
-          expression:          data["expression"],
+          comments: data["comments"],
+          priority: data["priority"],
+          description: data["description"],
+          expression: data["expression"],
           recovery_expression: data["recovery_expression"],
-          opdata:              data["opdata"],
-          recovery_mode:       1,
-          type:                0,
-          manual_close:        1,
+          opdata: data["opdata"],
+          recovery_mode: 1,
+          type: 0,
+          manual_close: 1
         }
       )
       # 检查是是否存在
@@ -144,21 +148,21 @@ class ZabbixManager
       data = data.with_indifferent_access
       # 请求生成触发器
       result = @client.api_request(
-        method: 'trigger.update',
+        method: "trigger.update",
         params: {
-          triggerid:           triggerid.to_i,
-          comments:            data["comments"],
-          priority:            data["priority"],
-          description:         data["description"],
-          expression:          data["expression"],
+          triggerid: triggerid,
+          comments: data["comments"],
+          priority: data["priority"],
+          description: data["description"],
+          expression: data["expression"],
           recovery_expression: data["recovery_expression"],
-          opdata:              data["opdata"],
-          recovery_mode:       1,
-          type:                1,
-          manual_close:        1,
+          opdata: data["opdata"],
+          recovery_mode: 1,
+          type: 1,
+          manual_close: 1
         }
       )
-      ap result
+      # ap result
       # 检查是是否存在
       result.empty? ? nil : result["triggerids"]
     end

data/lib/zabbix_manager/classes/unusable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Triggers < Basic
     def create_or_update(data)

data/lib/zabbix_manager/classes/usergroups.rb

@@ -1,24 +1,26 @@
+# frozen_string_literal: true
+
 class ZabbixManager
   class Usergroups < Basic
     # The method name used for interacting with Usergroups via Zabbix API
     #
     # @return [String]
     def method_name
-      'usergroup'
+      "usergroup"
     end
 
     # The key field name used for Usergroup objects via Zabbix API
     #
     # @return [String]
     def key
-      'usrgrpid'
+      "usrgrpid"
     end
 
     # The id field name used for identifying specific Usergroup objects via Zabbix API
     #
     # @return [String]
     def identify
-      'name'
+      "name"
     end
 
     # Set permissions for usergroup using Zabbix API
@@ -30,13 +32,13 @@ class ZabbixManager
     def permissions(data)
       permission = data[:permission] || 2
       result = @client.api_request(
-        method: 'usergroup.update',
+        method: "usergroup.update",
         params: {
           usrgrpid: data[:usrgrpid],
           rights: data[:hostgroupids].map { |t| { permission: permission, id: t } }
         }
       )
-      result ? result['usrgrpids'][0].to_i : nil
+      result ? result["usrgrpids"][0].to_i : nil
     end
 
     # Add users to usergroup using Zabbix API
@@ -60,14 +62,14 @@ class ZabbixManager
       user_groups = data[:usrgrpids].map do |t|
         {
           usrgrpid: t,
-          userids: data[:userids],
+          userids: data[:userids]
         }
       end
       result = @client.api_request(
-        method: 'usergroup.update',
-        params: user_groups,
+        method: "usergroup.update",
+        params: user_groups
       )
-      result ? result['usrgrpids'][0].to_i : nil
+      result ? result["usrgrpids"][0].to_i : nil
     end
   end
 end