haml 3.0.0.beta.3 → 3.0.0.rc.1

data/lib/sass/tree/rule_node.rb

@@ -7,7 +7,6 @@ module Sass::Tree
   # @see Sass::Tree
   class RuleNode < Node
     # The character used to include the parent selector
-    # @private
     PARENT = '&'
 
     # The CSS selector for this rule,
@@ -18,44 +17,19 @@ module Sass::Tree
     # @return [Array<String, Sass::Script::Node>]
     attr_accessor :rule
 
-    # The CSS selectors for this rule,
-    # parsed for commas and parent-references.
+    # The CSS selector for this rule,
+    # without any unresolved interpolation
+    # but with parent references still intact.
     # It's only set once {Tree::Node#perform} has been called.
     #
-    # It's an array of arrays.
-    # The first level of arrays comma-separated selectors;
-    # the second represents structure within those selectors,
-    # currently only parent-refs (represented by `:parent`).
-    # Newlines are represented as literal `\n` characters in the strings.
-    # For example,
-    #
-    #     &.foo, bar, baz,
-    #     bip, &.bop, bup
-    #
-    # would be
-    #
-    #     [[:parent, ".foo"], ["bar"], ["baz"],
-    #      ["\nbip"], [:parent, ".bop"], ["bup"]]
-    #
-    # @return [Array<Array<String, Symbol>>]
+    # @return [Selector::CommaSequence]
     attr_accessor :parsed_rules
 
-    # The CSS selectors for this rule,
-    # with all nesting and parent references resolved.
+    # The CSS selector for this rule,
+    # without any unresolved interpolation or parent references.
     # It's only set once {Tree::Node#cssize} has been called.
     #
-    # Each element is a distinct selector, separated by commas.
-    # Newlines are represented as literal `\n` characters in the strings.
-    # For example,
-    #
-    #     foo bar, baz,
-    #     bang, bip bop, blip
-    #
-    # would be
-    #
-    #     ["foo bar", "baz", "\nbang", "bip bop", "blip"]
-    #
-    # @return [Array<String>]
+    # @return [Selector::CommaSequence]
     attr_accessor :resolved_rules
 
     # How deep this rule is indented
@@ -121,6 +95,7 @@ module Sass::Tree
       name.gsub(/^/, '  ' * tabs) + children_to_src(tabs, opts, :sass)
     end
 
+    # @see Node#to_scss
     def to_scss(tabs, opts = {})
       name = rule.map {|r| r.is_a?(String) ? r : "\#{#{r.to_sass(opts)}}"}.
         join.gsub(/^[ \t]*/, '  ' * tabs)
@@ -135,6 +110,15 @@ module Sass::Tree
       res
     end
 
+    # Extends this Rule's selector with the given `extends`.
+    #
+    # @see Node#do_extend
+    def do_extend(extends)
+      node = dup
+      node.resolved_rules = resolved_rules.do_extend(extends)
+      node
+    end
+
     protected
 
     # Computes the CSS for the rule.
@@ -154,7 +138,9 @@ module Sass::Tree
       rule_indent = '  ' * (tabs - 1)
       per_rule_indent, total_indent = [:nested, :expanded].include?(style) ? [rule_indent, ''] : ['', rule_indent]
 
-      total_rule = total_indent + resolved_rules.join(rule_separator).split("\n").map do |line|
+      total_rule = total_indent + resolved_rules.members.
+        map {|seq| seq.to_a.join}.
+        join(rule_separator).split("\n").map do |line|
         per_rule_indent + line.strip
       end.join(line_separator)
 
@@ -199,21 +185,24 @@ module Sass::Tree
       to_return
     end
 
-    # Runs any SassScript that may be embedded in the rule,
-    # and parses the selectors for commas.
+    # Runs SassScript interpolation in the selector,
+    # and then parses the result into a {Sass::Selector::CommaSequence}.
     #
     # @param environment [Sass::Environment] The lexical environment containing
     #   variable and mixin values
     def perform!(environment)
-      @parsed_rules = parse_selector(run_interp(@rule, environment))
+      @parsed_rules = Sass::SCSS::StaticParser.new(run_interp(@rule, environment)).
+        parse_selector(self.line, self.filename)
       super
     end
 
     # Converts nested rules into a flat list of rules.
     #
+    # @param extends [Haml::Util::SubsetMap{Selector::Simple => Selector::Sequence}]
+    #   The extensions defined for this tree
     # @param parent [RuleNode, nil] The parent node of this node,
     #   or nil if the parent isn't a {RuleNode}
-    def _cssize(parent)
+    def _cssize(extends, parent)
       node = super
       rules = node.children.select {|c| c.is_a?(RuleNode)}
       props = node.children.reject {|c| c.is_a?(RuleNode) || c.invisible?}
@@ -232,14 +221,28 @@ module Sass::Tree
     # Resolves parent references and nested selectors,
     # and updates the indentation based on the parent's indentation.
     #
+    # @param extends [Haml::Util::SubsetMap{Selector::Simple => Selector::Sequence}]
+    #   The extensions defined for this tree
     # @param parent [RuleNode, nil] The parent node of this node,
     #   or nil if the parent isn't a {RuleNode}
     # @raise [Sass::SyntaxError] if the rule has no parents but uses `&`
-    def cssize!(parent)
-      self.resolved_rules = resolve_parent_refs(parent && parent.resolved_rules)
+    def cssize!(extends, parent)
+      self.resolved_rules = @parsed_rules.resolve_parent_refs(parent && parent.resolved_rules)
       super
     end
 
+    # Returns an error message if the given child node is invalid,
+    # and false otherwise.
+    #
+    # {ExtendNode}s are valid within {RuleNode}s.
+    #
+    # @param child [Tree::Node] A potential child nodecompact.
+    # @return [Boolean, String] Whether or not the child node is valid,
+    #   as well as the error message to display if it is invalid
+    def invalid_child?(child)
+      super unless child.is_a?(ExtendNode)
+    end
+
     # A hash that will be associated with this rule in the CSS document
     # if the {file:SASS_REFERENCE.md#debug_info-option `:debug_info` option} is enabled.
     # This data is used by e.g. [the FireSass Firebug extension](https://addons.mozilla.org/en-US/firefox/addon/103988).
@@ -252,70 +255,16 @@ module Sass::Tree
 
     private
 
-    def resolve_parent_refs(super_rules)
-      if super_rules.nil?
-        return @parsed_rules.map do |rule|
-          if rule.include?(:parent)
-            raise Sass::SyntaxError.new("Base-level rules cannot contain the parent-selector-referencing character '#{PARENT}'.")
-          end
-
-          rule.join
-        end
-      end
-
-      new_rules = []
-      super_rules.each do |super_rule|
-        @parsed_rules.each do |rule|
-          new_rules << []
-
-          # An initial newline of the child rule
-          # should be moved to the beginning of the entire rule
-          rule.first.slice!(0) if nl = (rule.first.is_a?(String) && rule.first[0] == ?\n)
-          rule = [nl ? "\n" : "", :parent, " ", *rule] unless rule.include?(:parent)
-
-          new_rules.last << rule.map do |segment|
-            next segment unless segment == :parent
-            super_rule
-          end.join
-        end
-      end
-      new_rules
-    end
-
-    def parse_selector(text)
-      scanner = StringScanner.new(text)
-      rules = [[]]
-
-      while scanner.rest?
-        rules.last << scanner.scan(/[^",&]*/)
-        case scanner.scan(/./)
-        when '&'; rules.last << :parent
-        when ','
-          scanner.scan(/\s*/)
-          if scanner.rest?
-            rules << []
-            rules.last << "\n" if scanner.matched.include?("\n")
-          end
-        when '"'
-          rules.last << '"' << scanner.scan(/([^"\\]|\\.)*/)
-          # We don't want to enforce that strings are closed,
-          # but we do want to consume quotes or trailing backslashes.
-          rules.last << scanner.scan(/./) if scanner.rest?
-        end
-      end
-
-      rules.map! do |l|
-        Haml::Util.merge_adjacent_strings(l).reject {|r| r.is_a?(String) && r.empty?}
-      end
-
-      rules
-    end
-
     def debug_info_rule
       node = DirectiveNode.new("@media -sass-debug-info")
       debug_info.map {|k, v| [k.to_s, v.to_s]}.sort.each do |k, v|
         rule = RuleNode.new([""])
-        rule.resolved_rules = [[k.to_s.gsub(/[^\w-]/, "\\\\\\0")]]
+        rule.resolved_rules = Sass::Selector::CommaSequence.new(
+          [Sass::Selector::Sequence.new(
+              [Sass::Selector::SimpleSequence.new(
+                  [Sass::Selector::Element.new(k.to_s.gsub(/[^\w-]/, "\\\\\\0"), nil)])
+              ])
+          ])
         prop = PropNode.new([""], "", :new)
         prop.resolved_name = "font-family"
         prop.resolved_value = Sass::SCSS::RX.escape_ident(v.to_s)

data/lib/sass/tree/variable_node.rb

@@ -16,6 +16,7 @@ module Sass
 
       protected
 
+      # @see Node#to_src
       def to_src(tabs, opts, fmt)
         "#{'  ' * tabs}$#{dasherize(@name, opts)}: #{@expr.to_sass(opts)}#{' !default' if @guarded}#{semi fmt}\n"
       end

data/lib/sass/tree/warn_node.rb

@@ -12,6 +12,7 @@ module Sass
 
       protected
 
+      # @see Node#to_src
       def to_src(tabs, opts, fmt)
         "#{'  ' * tabs}@warn #{@expr.to_sass(opts)}#{semi fmt}\n"
       end

data/lib/sass/tree/while_node.rb

@@ -13,6 +13,7 @@ module Sass::Tree
 
     protected
 
+    # @see Node#to_src
     def to_src(tabs, opts, fmt)
       "#{'  ' * tabs}@while #{@expr.to_sass(opts)}" + children_to_src(tabs, opts, fmt)
     end

data/test/haml/engine_test.rb

@@ -1348,17 +1348,199 @@ SASS
       render("%a{:a => 'b',\n:b => 'c'}(c='d'\nd='e') bar"))
   end
 
+  # Ruby Multiline
+
+  def test_silent_ruby_multiline
+    assert_equal(<<HTML, render(<<HAML))
+bar, baz, bang
+<p>foo</p>
+HTML
+- foo = ["bar",
+         "baz",
+         "bang"]
+= foo.join(", ")
+%p foo
+HAML
+  end
+
+  def test_loud_ruby_multiline
+    assert_equal(<<HTML, render(<<HAML))
+bar, baz, bang
+<p>foo</p>
+<p>bar</p>
+HTML
+= ["bar",
+   "baz",
+   "bang"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_escaped_loud_ruby_multiline
+    assert_equal(<<HTML, render(<<HAML))
+bar&lt;, baz, bang
+<p>foo</p>
+<p>bar</p>
+HTML
+&= ["bar<",
+    "baz",
+    "bang"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_unescaped_loud_ruby_multiline
+    assert_equal(<<HTML, render(<<HAML, :escape_html => true))
+bar<, baz, bang
+<p>foo</p>
+<p>bar</p>
+HTML
+!= ["bar<",
+    "baz",
+    "bang"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_flattened_loud_ruby_multiline
+    assert_equal(<<HTML, render(<<HAML))
+<pre>bar&#x000A;baz&#x000A;bang</pre>
+<p>foo</p>
+<p>bar</p>
+HTML
+~ "<pre>" + ["bar",
+             "baz",
+             "bang"].join("\\n") + "</pre>"
+%p foo
+%p bar
+HAML
+  end
+
+  def test_loud_ruby_multiline_with_block
+    assert_equal(<<HTML, render(<<HAML))
+farfazfang
+<p>foo</p>
+<p>bar</p>
+HTML
+= ["bar",
+   "baz",
+   "bang"].map do |str|
+  - str.gsub("ba",
+             "fa")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_silent_ruby_multiline_with_block
+    assert_equal(<<HTML, render(<<HAML))
+far
+faz
+fang
+<p>foo</p>
+<p>bar</p>
+HTML
+- ["bar",
+   "baz",
+   "bang"].map do |str|
+  = str.gsub("ba",
+             "fa")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_ruby_multiline_in_tag
+    assert_equal(<<HTML, render(<<HAML))
+<p>foo, bar, baz</p>
+<p>foo</p>
+<p>bar</p>
+HTML
+%p= ["foo",
+     "bar",
+     "baz"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_escaped_ruby_multiline_in_tag
+    assert_equal(<<HTML, render(<<HAML))
+<p>foo&lt;, bar, baz</p>
+<p>foo</p>
+<p>bar</p>
+HTML
+%p&= ["foo<",
+      "bar",
+      "baz"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_unescaped_ruby_multiline_in_tag
+    assert_equal(<<HTML, render(<<HAML, :escape_html => true))
+<p>foo<, bar, baz</p>
+<p>foo</p>
+<p>bar</p>
+HTML
+%p!= ["foo<",
+      "bar",
+      "baz"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_ruby_multiline_with_normal_multiline
+    assert_equal(<<HTML, render(<<HAML))
+foobarbar, baz, bang
+<p>foo</p>
+<p>bar</p>
+HTML
+= "foo" + |
+  "bar" + |
+  ["bar", |
+   "baz",
+   "bang"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
+  def test_ruby_multiline_after_filter
+    assert_equal(<<HTML, render(<<HAML))
+foo
+bar
+bar, baz, bang
+<p>foo</p>
+<p>bar</p>
+HTML
+:plain
+  foo
+  bar
+= ["bar",
+   "baz",
+   "bang"].join(", ")
+%p foo
+%p bar
+HAML
+  end
+
   # Encodings
 
   def test_utf_8_bom
-    assert_equal <<CSS, render(<<SCSS)
+    assert_equal <<HTML, render(<<HAML)
 <div class='foo'>
   <p>baz</p>
 </div>
-CSS
+HTML
 \xEF\xBB\xBF.foo
   %p baz
-SCSS
+HAML
   end
 
   unless Haml::Util.ruby1_8?