class SCSSLint::Linter::MergeableSelector

Checks for rule sets that can be merged with other rule sets.

Public Instance Methods

check_node(node) { || ... } click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 6
def check_node(node)
  node.children.each_with_object([]) do |child_node, seen_nodes|
    next unless child_node.is_a?(Sass::Tree::RuleNode)

    next if whitelist_contains(child_node)

    mergeable_node = find_mergeable_node(child_node, seen_nodes)
    seen_nodes << child_node
    next unless mergeable_node

    rule_text = node_rule(child_node).gsub(/(\r?\n)+/, ' ')

    add_lint child_node.line,
             "Merge rule `#{rule_text}` with rule "                   "on line #{mergeable_node.line}"
  end

  yield # Continue linting children
end
Also aliased as: visit_root, visit_rule
visit_root(node)
Alias for: check_node
visit_rule(node)
Alias for: check_node

Private Instance Methods

equal?(node1, node2) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 56
def equal?(node1, node2)
  node_rule(node1) == node_rule(node2)
end
find_mergeable_node(node, seen_nodes) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 31
def find_mergeable_node(node, seen_nodes)
  return if multiple_parent_references?(node)

  seen_nodes.find do |seen_node|
    equal?(node, seen_node) ||
      (config['force_nesting'] && nested?(node, seen_node))
  end
end
multiple_parent_references?(rule_node) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 40
def multiple_parent_references?(rule_node)
  return unless rules = rule_node.parsed_rules

  # Iterate over each sequence counting all parent references
  total_parent_references = rules.members.inject(0) do |sum, seq|
    sum + seq.members.inject(0) do |ssum, simple_seq|
      next ssum unless simple_seq.respond_to?(:members)
      ssum + simple_seq.members.count do |member|
        member.is_a?(Sass::Selector::Parent)
      end
    end
  end

  total_parent_references > 1
end
nested?(node1, node2) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 60
def nested?(node1, node2)
  return false unless single_rule?(node1) && single_rule?(node2)

  rule1 = node_rule(node1)
  rule2 = node_rule(node2)
  subrule?(rule1, rule2) || subrule?(rule2, rule1)
end
node_rule(node) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 68
def node_rule(node)
  node.rule.join
end
single_rule?(node) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 72
def single_rule?(node)
  return unless node.parsed_rules
  node.parsed_rules.members.count == 1
end
subrule?(rule1, rule2) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 77
def subrule?(rule1, rule2)
  rule1.to_s.start_with?("#{rule2} ", "#{rule2}.")
end
whitelist_contains(node) click to toggle source
# File lib/scss_lint/linter/mergeable_selector.rb, line 81
def whitelist_contains(node)
  if @whitelist.nil?
    @whitelist = config['whitelist'] || []
    @whitelist = [@whitelist] if @whitelist.is_a? String
  end

  @whitelist.include?(node_rule(node))
end