Put test result in variable to satisfy RuboCop
This commit is contained in:
@@ -18,12 +18,12 @@ Facter.add(:java_major_version) do
|
||||
java_version = Facter.value(:java_version)
|
||||
unless java_version.nil?
|
||||
# First part > 1, use the first part as major version
|
||||
java_version_test = java_version.strip.split('.')[0]
|
||||
if java_version_test.to_i > 1
|
||||
java_major_version = java_version_test
|
||||
else
|
||||
java_major_version = java_version.strip.split('_')[0].split('.')[1]
|
||||
end
|
||||
java_version_first_number = java_version.strip.split('.')[0]
|
||||
java_major_version = if java_version_first_number.to_i > 1
|
||||
java_version_first_number
|
||||
else
|
||||
java_version.strip.split('_')[0].split('.')[1]
|
||||
end
|
||||
end
|
||||
end
|
||||
java_major_version
|
||||
|
||||
@@ -16,14 +16,13 @@ Facter.add(:java_patch_level) do
|
||||
java_version = Facter.value(:java_version)
|
||||
unless java_version.nil?
|
||||
# First part > 1, use . as seperator to get patch level
|
||||
java_version_test = java_version.strip.split('.')[0]
|
||||
if java_version_test.to_i > 1
|
||||
java_patch_level = java_version.strip.split('.')[2]
|
||||
else
|
||||
java_patch_level = java_version.strip.split('_')[1]
|
||||
end
|
||||
java_version_first_number = java_version.strip.split('.')[0]
|
||||
java_patch_level = if java_version_first_number.to_i > 1
|
||||
java_version.strip.split('.')[2]
|
||||
else
|
||||
java_version.strip.split('_')[1]
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
java_patch_level
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user