haml 3.1.8 → 3.2.0.alpha.2

data/vendor/sass/lib/sass/tree/visitors/to_css.rb

@@ -57,11 +57,21 @@ class Sass::Tree::Visitors::ToCss < Sass::Tree::Visitors::Base
 
   def visit_comment(node)
     return if node.invisible?
-    spaces = ('  ' * [@tabs - node.resolved_value[/^ */].size, 0].max)
+    spaces = ('  ' * [@tabs - node.value[/^ */].size, 0].max)
 
-    content = node.resolved_value.gsub(/^/, spaces).gsub(%r{^(\s*)//(.*)$}) do |md|
+    content = node.value.gsub(/^/, spaces).gsub(%r{^(\s*)//(.*)$}) do |md|
       "#{$1}/*#{$2} */"
     end
+    if content =~ /[^\\]\#\{.*\}/
+      Sass::Util.sass_warn <<MESSAGE
+WARNING:
+On line #{node.line}#{" of '#{node.filename}'" if node.filename}
+Comments will evaluate the contents of interpolations (\#{ ... }) in Sass 3.2.
+Please escape the interpolation by adding a backslash before the hash sign.
+MESSAGE
+    elsif content =~ /\\\#\{.*\}/
+      content.gsub!(/\\(\#\{.*\})/, '\1')
+    end
     content.gsub!(/\n +(\* *(?!\/))?/, ' ') if (node.style == :compact || node.style == :compressed) && !node.loud
     content
   end
@@ -112,9 +122,9 @@ class Sass::Tree::Visitors::ToCss < Sass::Tree::Visitors::Base
   def visit_prop(node)
     tab_str = '  ' * (@tabs + node.tabs)
     if node.style == :compressed
-      "#{tab_str}#{node.resolved_name}:#{node.resolved_value}"
+      "#{tab_str}#{node.resolved_name}:#{node.resolved_value}#{'!important' if node.important}"
     else
-      "#{tab_str}#{node.resolved_name}: #{node.resolved_value};"
+      "#{tab_str}#{node.resolved_name}: #{node.resolved_value}#{' !important' if node.important};"
     end
   end
 
@@ -146,10 +156,6 @@ class Sass::Tree::Visitors::ToCss < Sass::Tree::Visitors::Base
       if node.style != :compressed
         if node.options[:debug_info]
           to_return << visit(debug_info_rule(node.debug_info, node.options)) << "\n"
-        elsif node.options[:trace_selectors]
-          to_return << "#{old_spaces}/* "
-          to_return << node.stack_trace.join("\n   #{old_spaces}")
-          to_return << " */\n"
         elsif node.options[:line_comments]
           to_return << "#{old_spaces}/* line #{node.line}"
 
@@ -198,7 +204,7 @@ class Sass::Tree::Visitors::ToCss < Sass::Tree::Visitors::Base
                 [Sass::Selector::Element.new(k.to_s.gsub(/[^\w-]/, "\\\\\\0"), nil)])
             ])
         ])
-      prop = Sass::Tree::PropNode.new([""], Sass::Script::String.new(''), :new)
+      prop = Sass::Tree::PropNode.new([""], "", false, :new)
       prop.resolved_name = "font-family"
       prop.resolved_value = Sass::SCSS::RX.escape_ident(v.to_s)
       rule << prop

data/vendor/sass/lib/sass/tree/warn_node.rb

@@ -6,7 +6,7 @@ module Sass
     class WarnNode < Node
       # The expression to print.
       # @return [Script::Node]
-      attr_accessor :expr
+      attr_reader :expr
 
       # @param expr [Script::Node] The expression to print
       def initialize(expr)

data/vendor/sass/lib/sass/tree/while_node.rb

@@ -7,7 +7,7 @@ module Sass::Tree
   class WhileNode < Node
     # The parse tree for the continuation expression.
     # @return [Script::Node]
-    attr_accessor :expr
+    attr_reader :expr
 
     # @param expr [Script::Node] See \{#expr}
     def initialize(expr)

data/vendor/sass/lib/sass/util.rb

@@ -293,17 +293,19 @@ module Sass
     #
     # @yield A block in which no Sass warnings will be printed
     def silence_sass_warnings
-      old_level, Sass.logger.log_level = Sass.logger.log_level, :error
+      old_silence_warnings = @@silence_warnings
+      @@silence_warnings = true
       yield
     ensure
-      Sass.logger.log_level = old_level
+      @@silence_warnings = old_silence_warnings
     end
 
     # The same as `Kernel#warn`, but is silenced by \{#silence\_sass\_warnings}.
     #
     # @param msg [String]
     def sass_warn(msg)
-      Sass.logger.warn(msg)
+      return if @@silence_warnings
+      warn(msg)
     end
 
     ## Cross Rails Version Compatibility
@@ -467,8 +469,7 @@ MSG
       # We allow any printable ASCII characters but double quotes in the charset decl
       bin = str.dup.force_encoding("BINARY")
       encoding = Sass::Util::ENCODINGS_TO_CHECK.find do |enc|
-        re = Sass::Util::CHARSET_REGEXPS[enc]
-        re && bin =~ re
+        bin =~ Sass::Util::CHARSET_REGEXPS[enc]
       end
       charset, bom = $1, $2
       if charset
@@ -509,8 +510,6 @@ MSG
             Regexp.new(/\A(?:#{_enc("\uFEFF", e)})?#{
               _enc('@charset "', e)}(.*?)#{_enc('"', e)}|\A(#{
               _enc("\uFEFF", e)})/)
-          rescue Encoding::ConverterNotFound => _
-            nil # JRuby on Java 5 doesn't support UTF-32
           rescue
             # /\A@charset "(.*?)"/
             Regexp.new(/\A#{_enc('@charset "', e)}(.*?)#{_enc('"', e)}/)
@@ -615,57 +614,6 @@ MSG
       '"' + obj.gsub(/[\x00-\x7F]+/) {|s| s.inspect[1...-1]} + '"'
     end
 
-    # Extracts the non-string vlaues from an array containing both strings and non-strings.
-    # These values are replaced with escape sequences.
-    # This can be undone using \{#inject\_values}.
-    #
-    # This is useful e.g. when we want to do string manipulation
-    # on an interpolated string.
-    #
-    # The precise format of the resulting string is not guaranteed.
-    # However, it is guaranteed that newlines and whitespace won't be affected.
-    #
-    # @param arr [Array] The array from which values are extracted.
-    # @return [(String, Array)] The resulting string, and an array of extracted values.
-    def extract_values(arr)
-      values = []
-      return arr.map do |e|
-        next e.gsub('{', '{{') if e.is_a?(String)
-        values << e
-        next "{#{values.count - 1}}"
-      end.join, values
-    end
-
-    # Undoes \{#extract\_values} by transforming a string with escape sequences
-    # into an array of strings and non-string values.
-    #
-    # @param str [String] The string with escape sequences.
-    # @param values [Array] The array of values to inject.
-    # @return [Array] The array of strings and values.
-    def inject_values(str, values)
-      return [str.gsub('{{', '{')] if values.empty?
-      # Add an extra { so that we process the tail end of the string
-      result = (str + '{{').scan(/(.*?)(?:(\{\{)|\{(\d+)\})/m).map do |(pre, esc, n)|
-        [pre, esc ? '{' : '', n ? values[n.to_i] : '']
-      end.flatten(1)
-      result[-2] = '' # Get rid of the extra {
-      merge_adjacent_strings(result).reject {|s| s == ''}
-    end
-
-    # Allows modifications to be performed on the string form
-    # of an array containing both strings and non-strings.
-    #
-    # @param arr [Array] The array from which values are extracted.
-    # @yield [str] A block in which string manipulation can be done to the array.
-    # @yieldparam str [String] The string form of `arr`.
-    # @yieldreturn [String] The modified string.
-    # @return [Array] The modified, interpolated array.
-    def with_extracted_values(arr)
-      str, vals = extract_values(arr)
-      str = yield str
-      inject_values(str, vals)
-    end
-
     ## Static Method Stuff
 
     # The context in which the ERB for \{#def\_static\_method} will be run.

data/vendor/sass/sass.gemspec

@@ -18,12 +18,11 @@ SASS_GEMSPEC = Gem::Specification.new do |spec|
       command line tool or a web-framework plugin.
     END
 
-  spec.required_ruby_version = '>= 1.8.7'
   spec.add_development_dependency 'yard', '>= 0.5.3'
   spec.add_development_dependency 'maruku', '>= 0.5.9'
 
   readmes = Dir['*'].reject{ |x| x =~ /(^|[^.a-z])[a-z]+/ || x == "TODO" }
-  spec.executables = ['sass', 'sass-convert', 'scss']
+  spec.executables = ['sass', 'sass-convert']
   spec.files = Dir['rails/init.rb', 'lib/**/*', 'vendor/**/*',
     'bin/*', 'test/**/*', 'extra/**/*', 'Rakefile', 'init.rb',
     '.yardopts'] + readmes

data/vendor/sass/test/sass/cache_test.rb

@@ -1,6 +1,5 @@
 #!/usr/bin/env ruby
 require File.dirname(__FILE__) + '/../test_helper'
-require File.dirname(__FILE__) + '/test_helper'
 require 'sass/engine'
 
 class CacheTest < Test::Unit::TestCase
@@ -12,7 +11,6 @@ class CacheTest < Test::Unit::TestCase
 
   def teardown
     FileUtils.rm_rf @@cache_dir
-    clean_up_sassc
   end
 
   def test_file_cache_writes_a_file
@@ -66,19 +64,6 @@ class CacheTest < Test::Unit::TestCase
     assert_equal an_object, cache.retrieve("an_object", "")
   end
 
-  class Unmarshalable
-    def _dump(_)
-      raise 'Unmarshalable'
-    end
-  end
-
-  def test_cache_node_with_unmarshalable_option
-    engine = Sass::Engine.new("foo {a: b + c}",
-      :syntax => :scss, :object => Unmarshalable.new, :filename => 'file.scss',
-      :importer => Sass::Importers::Filesystem.new(absolutize('templates')))
-    engine.to_tree
-  end
-
   private
   def root_node
     Sass::Engine.new(<<-SCSS, :syntax => :scss).to_tree

data/vendor/sass/test/sass/conversion_test.rb

@@ -1131,13 +1131,17 @@ div
 SASS
   end
 
-   def test_loud_comment_conversion
+  def test_loud_comment_conversion
     assert_renders(<<SASS, <<SCSS)
 /*! \#{"interpolated"}
+/*!
+ *  \#{"also interpolated"}
 SASS
 /*! \#{"interpolated"} */
+/*!
+ *  \#{"also interpolated"} */
 SCSS
-    silence_warnings {assert_renders(<<SASS, <<SCSS)}
+    assert_renders(<<SASS, <<SCSS)
 //! \#{"interpolated"}
 //!
 //! \#{"also interpolated"}

data/vendor/sass/test/sass/css2sass_test.rb

@@ -268,15 +268,6 @@ SASS
 CSS
   end
 
-  def test_double_comma
-    assert_equal(<<SASS, css2sass(<<CSS))
-foo, bar
-  a: b
-SASS
-foo, , bar { a: b }
-CSS
-  end
-
   # Error reporting
 
   def test_error_reporting

data/vendor/sass/test/sass/engine_test.rb

@@ -57,12 +57,16 @@ MSG
     "& a\n  :b c" => ["Base-level rules cannot contain the parent-selector-referencing character '&'.", 1],
     "a\n  :b\n    c" => "Illegal nesting: Only properties may be nested beneath properties.",
     "$a: b\n  :c d\n" => "Illegal nesting: Nothing may be nested beneath variable declarations.",
+    "@import foo.sass" => "File to import not found or unreadable: foo.sass.",
     "$a: b\n  :c d\n" => "Illegal nesting: Nothing may be nested beneath variable declarations.",
+    "@import foo.sass" => <<MSG,
+File to import not found or unreadable: foo.sass.
+Load path: .
+MSG
     "@import templates/basic\n  foo" => "Illegal nesting: Nothing may be nested beneath import directives.",
     "foo\n  @import foo.css" => "CSS import directives may only be used at the root of a document.",
     "@if true\n  @import foo" => "Import directives may not be used within control directives or mixins.",
     "@mixin foo\n  @import foo" => "Import directives may not be used within control directives or mixins.",
-    "@import foo;" => "Invalid @import: expected end of line, was \";\".",
     '$foo: "bar" "baz" !' => %Q{Invalid CSS after ""bar" "baz" ": expected expression (e.g. 1px, bold), was "!"},
     '$foo: "bar" "baz" $' => %Q{Invalid CSS after ""bar" "baz" ": expected expression (e.g. 1px, bold), was "$"},
     "=foo\n  :color red\n.bar\n  +bang" => "Undefined mixin 'bang'.",
@@ -259,7 +263,7 @@ SASS
   end
 
   def test_imported_exception
-    [1, 2, 3, 4, 5].each do |i|
+    [1, 2, 3, 4].each do |i|
       begin
         Sass::Engine.new("@import bork#{i}", :load_paths => [File.dirname(__FILE__) + '/templates/']).render
       rescue Sass::SyntaxError => err
@@ -281,7 +285,7 @@ SASS
   end
 
   def test_double_imported_exception
-    [1, 2, 3, 4, 5].each do |i|
+    [1, 2, 3, 4].each do |i|
       begin
         Sass::Engine.new("@import nested_bork#{i}", :load_paths => [File.dirname(__FILE__) + '/templates/']).render
       rescue Sass::SyntaxError => err
@@ -306,23 +310,6 @@ SASS
     end
   end
 
-  def test_selector_tracing
-    actual_css = render(<<-SCSS, :syntax => :scss, :trace_selectors => true)
-      @mixin mixed {
-        .mixed { color: red; }
-      }
-      .context {
-        @include mixed;
-      }
-    SCSS
-    assert_equal(<<CSS,actual_css)
-/* on line 2 of test_selector_tracing_inline.scss, in `mixed'
-   from line 5 of test_selector_tracing_inline.scss */
-.context .mixed {
-  color: red; }
-CSS
-  end
-
   def test_mixin_exception
     render(<<SASS)
 =error-mixin($a)
@@ -592,21 +579,12 @@ CSS
     assert File.exists?(sassc_file)
   end
 
-  def test_nonexistent_import
-    assert_raise_message(Sass::SyntaxError, <<ERR.rstrip) do
-File to import not found or unreadable: nonexistent.sass.
-Load path: #{Dir.pwd}
-ERR
-      render("@import nonexistent.sass")
-    end
-  end
-
   def test_nonexistent_extensionless_import
     assert_raise_message(Sass::SyntaxError, <<ERR.rstrip) do
 File to import not found or unreadable: nonexistent.
-Load path: #{Dir.pwd}
+Load path: .
 ERR
-      render("@import nonexistent")
+      assert_equal("@import url(nonexistent.css);\n", render("@import nonexistent"))
     end
   end
 
@@ -1203,18 +1181,6 @@ bar
 SASS
   end
 
-  def test_control_directive_in_nested_property
-    assert_equal(<<CSS, render(<<SASS))
-foo {
-  a-b: c; }
-CSS
-foo
-  a:
-    @if true
-      b: c
-SASS
-  end
-
   def test_interpolation
     assert_equal("a-1 {\n  b-2-3: c-3; }\n", render(<<SASS))
 $a: 1
@@ -1582,11 +1548,11 @@ foo
    */
 SASS
   end
-
   def test_loud_comment_in_silent_comment
-    silence_warnings {assert_equal <<CSS, render(<<SASS, :style => :compressed)}
+    assert_equal <<CSS, render(<<SASS, :style => :compressed)
 foo{color:blue;/* foo */
 /* bar */
+/* */
 /* bip */
 /* baz */}
 CSS
@@ -1602,7 +1568,8 @@ SASS
 
   def test_loud_comment_is_evaluated
     assert_equal <<CSS, render(<<SASS)
-/* Hue: 327.216deg */
+/*
+ * Hue: 327.216deg */
 CSS
 /*!
   Hue: \#{hue(#f836a0)}
@@ -1805,22 +1772,22 @@ SASS
 
   def test_interpolation_doesnt_deep_unquote_strings
     assert_equal(<<CSS, render(<<SASS))
-.foo {
-  a: "bar" "baz"; }
+.foo- "bar" "baz" {
+  a: b; }
 CSS
-.foo
-  a: \#{"bar" "baz"}
+.foo-\#{"bar" "baz"}
+  a: b
 SASS
   end
 
   def test_warn_directive
   expected_warning = <<EXPECTATION
 WARNING: this is a warning
-         on line 4 of test_warn_directive_inline.sass
+        on line 4 of test_warn_directive_inline.sass
 
 WARNING: this is a mixin warning
-         on line 2 of test_warn_directive_inline.sass, in `foo'
-         from line 7 of test_warn_directive_inline.sass
+        on line 2 of test_warn_directive_inline.sass, in `foo'
+        from line 7 of test_warn_directive_inline.sass
 EXPECTATION
     assert_warning expected_warning do
       assert_equal <<CSS, render(<<SASS)
@@ -1850,15 +1817,15 @@ SASS
   def test_warn_with_imports
     expected_warning = <<WARN
 WARNING: In the main file
-         on line 1 of #{File.dirname(__FILE__)}/templates/warn.sass
+        on line 1 of #{File.dirname(__FILE__)}/templates/warn.sass
 
 WARNING: Imported
-         on line 1 of #{File.dirname(__FILE__)}/templates/warn_imported.sass
-         from line 2 of #{File.dirname(__FILE__)}/templates/warn.sass
+        on line 1 of #{File.dirname(__FILE__)}/templates/warn_imported.sass
+        from line 2 of #{File.dirname(__FILE__)}/templates/warn.sass
 
 WARNING: In an imported mixin
-         on line 4 of #{File.dirname(__FILE__)}/templates/warn_imported.sass, in `emits-a-warning'
-         from line 3 of #{File.dirname(__FILE__)}/templates/warn.sass
+        on line 4 of #{File.dirname(__FILE__)}/templates/warn_imported.sass, in `emits-a-warning'
+        from line 3 of #{File.dirname(__FILE__)}/templates/warn.sass
 WARN
     assert_warning expected_warning do
       renders_correctly "warn", :style => :compact, :load_paths => [File.dirname(__FILE__) + "/templates"]
@@ -2041,31 +2008,6 @@ CSS
 
   # Regression tests
 
-  def test_interpolated_comment_in_mixin
-    assert_equal <<CSS, render(<<SASS)
-/* color: red */
-.foo {
-  color: red; }
-
-/* color: blue */
-.foo {
-  color: blue; }
-
-/* color: green */
-.foo {
-  color: green; }
-CSS
-=foo($var)
-  /*! color: \#{$var}
-  .foo
-    color: $var
-
-+foo(red)
-+foo(blue)
-+foo(green)
-SASS
-  end
-
   def test_parens_in_mixins
     assert_equal(<<CSS, render(<<SASS))
 .foo {
@@ -2206,16 +2148,6 @@ CSS
 SASS
   end
 
-  def test_silent_comment_in_prop_val_after_important
-    assert_equal(<<CSS, render(<<SASS))
-.advanced {
-  display: none !important; }
-CSS
-.advanced
-  display: none !important // yeah, yeah. it's not really a style anyway.
-SASS
-  end
-
   def test_mixin_with_keyword_args
     assert_equal <<CSS, render(<<SASS)
 .mixed {
@@ -2344,16 +2276,7 @@ SASS
 WARNING:
 On line 1 of 'test_comment_interpolation_warning_inline.sass'
 Comments will evaluate the contents of interpolations (\#{ ... }) in Sass 3.2.
-Please escape the interpolation by adding a backslash before the `#`.
-END
-  end
-
-  def test_loud_silent_comment_warning
-    assert_warning(<<END) {render("//! \#{foo}")}
-WARNING:
-On line 1 of 'test_loud_silent_comment_warning_inline.sass'
-`//` comments will no longer be allowed to use the `!` flag in Sass 3.2.
-Please change to `/*` comments.
+Please escape the interpolation by adding a backslash before the hash sign.
 END
   end
 
@@ -2497,27 +2420,6 @@ SASS
     assert_equal original_filename, importer.engine("imported").options[:original_filename]
   end
 
-  def test_deprecated_PRECISION
-    assert_warning(<<END) {assert_equal 1000.0, Sass::Script::Number::PRECISION}
-Sass::Script::Number::PRECISION is deprecated and will be removed in a future release. Use Sass::Script::Number.precision_factor instead.
-END
-  end
-  def test_changing_precision
-    begin
-      Sass::Script::Number.precision = 8
-      assert_equal <<CSS, render(<<SASS)
-div {
-  maximum: 1.00000001;
-  too-much: 1.0; }
-CSS
-div
-  maximum : 1.00000001
-  too-much: 1.000000001
-SASS
-    ensure
-      Sass::Script::Number.precision = 3
-    end
-  end
 
   private