Merge pull request #5421 from betagouv/fix-haml-linter
fix haml linter declaration
This commit is contained in:
commit
7301b202aa
1 changed files with 40 additions and 38 deletions
|
@ -1,46 +1,48 @@
|
|||
module HamlLint
|
||||
class Linter::ApplicationNameLinter < Linter
|
||||
include LinterRegistry
|
||||
if defined?(HamlLint)
|
||||
module HamlLint
|
||||
class Linter::ApplicationNameLinter < Linter
|
||||
include LinterRegistry
|
||||
|
||||
FORBIDDEN = 'demarches-simplifiees.fr'
|
||||
REPLACEMENT = "APPLICATION_NAME"
|
||||
MSG = 'Hardcoding %s is forbidden, use %s instead'
|
||||
FORBIDDEN = 'demarches-simplifiees.fr'
|
||||
REPLACEMENT = "APPLICATION_NAME"
|
||||
MSG = 'Hardcoding %s is forbidden, use %s instead'
|
||||
|
||||
def visit_tag(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_script(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_silent_script(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_plain(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_comment(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_haml_comment(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def check(node)
|
||||
line = line_text_for_node(node)
|
||||
if line.downcase.include?(FORBIDDEN)
|
||||
record_lint(node, format(MSG, FORBIDDEN, REPLACEMENT))
|
||||
def visit_tag(node)
|
||||
check(node)
|
||||
end
|
||||
end
|
||||
|
||||
private
|
||||
def visit_script(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def line_text_for_node(node)
|
||||
document.source_lines[node.line - 1]
|
||||
def visit_silent_script(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_plain(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_comment(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def visit_haml_comment(node)
|
||||
check(node)
|
||||
end
|
||||
|
||||
def check(node)
|
||||
line = line_text_for_node(node)
|
||||
if line.downcase.include?(FORBIDDEN)
|
||||
record_lint(node, format(MSG, FORBIDDEN, REPLACEMENT))
|
||||
end
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def line_text_for_node(node)
|
||||
document.source_lines[node.line - 1]
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue