diff options
| author | Tim Smith <tsmith@chef.io> | 2020-01-20 09:56:57 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-01-20 09:56:57 -0800 |
| commit | 920971fa294615914b7542a6aec772ee1cd394af (patch) | |
| tree | 0d18bd77ff46aadfcb7136f791b2d0d16013a786 | |
| parent | e25edd2b208d4b7a433c42eb1d367c07af9c63f3 (diff) | |
| parent | f2b0a4d68b50a2d8906043cb7483f2b288c169e3 (diff) | |
| download | ohai-920971fa294615914b7542a6aec772ee1cd394af.tar.gz | |
Merge pull request #1423 from chef/dmi
Use is_a? to check the class in the DMI plugin
| -rw-r--r-- | lib/ohai/common/dmi.rb | 4 | ||||
| -rw-r--r-- | lib/ohai/plugins/solaris2/dmi.rb | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/lib/ohai/common/dmi.rb b/lib/ohai/common/dmi.rb index c7f8a5e1..cc9f4c10 100644 --- a/lib/ohai/common/dmi.rb +++ b/lib/ohai/common/dmi.rb @@ -117,12 +117,12 @@ module Ohai def convenience_keys(dmi) dmi.each do |type, records| in_common = Mash.new - next unless records.class.to_s == "Mash" + next unless records.is_a?(Mash) next unless records.key?("all_records") records[:all_records].each do |record| record.each do |field, value| - next if value.class.to_s == "Mash" + next if value.is_a?(Mash) next if field.to_s == "application_identifier" next if field.to_s == "size" next if field.to_s == "record_id" diff --git a/lib/ohai/plugins/solaris2/dmi.rb b/lib/ohai/plugins/solaris2/dmi.rb index 823d51cf..d4b91a3b 100644 --- a/lib/ohai/plugins/solaris2/dmi.rb +++ b/lib/ohai/plugins/solaris2/dmi.rb @@ -24,7 +24,7 @@ Ohai.plugin(:DMI) do # if we already have a "dmi" with keys (presumably from dmidecode), don't try smbios # note that a single key just means dmidecode exited with its version - if (dmi.class.to_s == "Mash") && (dmi.keys.length > 1) + if dmi.is_a?(Mash) && dmi.keys.length > 1 logger.trace("Plugin DMI: skipping smbios output, since DMI information has already been provided") return end @@ -177,7 +177,7 @@ Ohai.plugin(:DMI) do next end # overwrite "raw" value with a new Mash - dmi[dmi_record[:type]][:all_records][dmi_record[:position]][field] = Mash.new unless dmi[dmi_record[:type]][:all_records][dmi_record[:position]][field].class.to_s == "Mash" + dmi[dmi_record[:type]][:all_records][dmi_record[:position]][field] = Mash.new unless dmi[dmi_record[:type]][:all_records][dmi_record[:position]][field].is_a?(Mash) dmi[dmi_record[:type]][:all_records][dmi_record[:position]][field][extended_data[1]] = extended_data[2] else |