lib/puppet-lint/plugins/topscope_variable.rb in puppet-lint-topscope-variable-check-1.2.0 vs lib/puppet-lint/plugins/topscope_variable.rb in puppet-lint-topscope-variable-check-2.0.0
- old
+ new
@@ -3,17 +3,18 @@
class_list = (class_indexes + defined_type_indexes)
# do not check if the code is not part of a class
return if class_list.first.nil?
tokens.select { |x| x.type == :VARIABLE }.each do |token|
- next if token.value !~ /^::[a-z0-9_][a-zA-Z0-9_]+::/
+ next unless /^::[a-z0-9_][a-zA-Z0-9_]+::/.match?(token.value)
+
fixed = token.value.sub(/^::/, '')
notify(
:warning,
message: "use $#{fixed} instead of $#{token.value}",
line: token.line,
column: token.column,
- token: token
+ token: token,
)
end
end
def fix(problem)