slim 1.1.0 → 1.1.1

data/.travis.yml

@@ -1,16 +1,24 @@
 rvm:
   - 1.8.7
-  - 1.9.2
   - 1.9.3
   - ruby-head
   - jruby
-  - rbx-2.0
+  - rbx-18mode
 env:
   - "TASK=test"
   - "TASK=test TEMPLE=master"
   - "TASK=test:rails RAILS=master"
   - "TASK=test:rails RAILS=3.0.11"
   - "TASK=test:rails RAILS=3.1.3"
+matrix:
+  exclude:
+    # Test Rails master only on 1.9.3+ Rubies
+    - rvm: 1.8.7
+      env: "TASK=test:rails RAILS=master"
+    - rvm: jruby
+      env: "TASK=test:rails RAILS=master"
+    - rvm: rbx-18mode
+      env: "TASK=test:rails RAILS=master"
 script: "bundle exec rake test:ci"
 notifications:
   email: false

data/CHANGES

@@ -1,4 +1,9 @@
-master
+1.1.1
+
+  * Evaluating a html attribute now happens only once (#219)
+  * Code with trailing comma is treated as broken line (#226)
+  * Support option :remove_empty_attrs (default true)
+  * Require temple 0.4.0
 
 1.1.0
 

data/lib/slim/compiler.rb

@@ -79,7 +79,7 @@ module Slim
             if delimiter = options[:attr_delimiter][name]
               "#{tmp}.respond_to?(:join) ? #{tmp}.flatten.compact.join(#{delimiter.inspect}) : #{tmp}"
             else
-              code
+              tmp
             end
            ]]]]
       end

data/lib/slim/engine.rb

@@ -33,7 +33,8 @@ module Slim
     # Symbol      | :format            | :xhtml                        | HTML output format
     # String      | :attr_wrapper      | '"'                           | Character to wrap attributes in html (can be ' or ")
     # Hash        | :attr_delimiter    | {'class' => ' '}              | Joining character used if multiple html attributes are supplied (e.g. id1_id2)
-    # Symbol      | :sort_attrs        | true                          | Sort attributes by name
+    # Boolean     | :sort_attrs        | true                          | Sort attributes by name
+    # Boolean     | :remove_empty_attrs| true                          | Remove attributes with empty value
     # Boolean     | :pretty            | false                         | Pretty html indenting (This is slower!)
     # String      | :indent            | '  '                          | Indentation string
     # Boolean     | :streaming         | false (true in Rails > 3.1)   | Enable output streaming
@@ -61,8 +62,10 @@ module Slim
     use Slim::Sections, :sections, :dictionary, :dictionary_access
     use Slim::EndInserter
     use Slim::Compiler, :disable_capture, :attr_delimiter
-    use Temple::HTML::AttributeMerger, :attr_delimiter, :sort_attrs
-    use Temple::HTML::Pretty, :format, :attr_wrapper, :pretty, :indent
+    html :AttributeMerger, :attr_delimiter
+    html :AttributeSorter, :sort_attrs
+    html :AttributeRemover, :remove_empty_attrs
+    html :Pretty, :format, :attr_wrapper, :pretty, :indent
     filter :Escapable, :use_html_safe, :disable_escape
     filter :ControlFlow
     filter :MultiFlattener

data/lib/slim/parser.rb

@@ -224,7 +224,7 @@ module Slim
       end
     end
 
-    def parse_text_block(first_line = nil, text_indent = nil)
+    def parse_text_block(first_line = nil, text_indent = nil, in_tag = false)
       result = [:multi]
       if !first_line || first_line.empty?
         text_indent = nil
@@ -253,7 +253,11 @@ module Slim
           # The text block lines must be at least indented
           # as deep as the first line.
           offset = text_indent ? indent - text_indent : 0
-          syntax_error!('Unexpected text indentation') if offset < 0
+          if offset < 0
+            syntax_error!("Text line not indented deep enough.\n" +
+                          "The first text line defines the necessary text indentation." +
+                          (in_tag ? "\nAre you trying to nest a child tag in a tag containing text? Use | for the text block!" : ''))
+          end
 
           result << [:newline] << [:slim, :interpolate, (text_indent ? "\n" : '') + (' ' * offset) + @line]
 
@@ -267,7 +271,7 @@ module Slim
 
     def parse_broken_line
       broken_line = @line.strip
-      while broken_line[-1] == ?\\
+      while broken_line =~ /[,\\]\Z/
         next_line || syntax_error!('Unexpected end of file')
         broken_line << "\n" << @line.strip
       end
@@ -312,7 +316,7 @@ module Slim
         @stacks << content
       when /\A( ?)(.*)\Z/
         # Text content
-        tag << parse_text_block($2, @orig_line.size - @line.size + $1.size)
+        tag << parse_text_block($2, @orig_line.size - @line.size + $1.size, true)
       end
     end
 

data/lib/slim/version.rb

@@ -1,5 +1,5 @@
 module Slim
   # Slim version string
   # @api public
-  VERSION = '1.1.0'
+  VERSION = '1.1.1'
 end

data/slim.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = %w(lib)
 
-  s.add_runtime_dependency('temple', ['~> 0.3.5'])
+  s.add_runtime_dependency('temple', ['~> 0.4.0'])
   s.add_runtime_dependency('tilt', ['~> 1.3.2'])
 
   s.add_development_dependency('rake', ['>= 0.8.7'])

data/test/rails/test/test_slim.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../helper', __FILE__)
 
-class TestSlim < ActionController::IntegrationTest
+class TestSlim < ActionDispatch::IntegrationTest
   test "normal view" do
     get "slim/normal"
     assert_response :success
@@ -59,13 +59,7 @@ class TestSlim < ActionController::IntegrationTest
     post "parents", 'parent[name]' => "p1", 'parent[children_attributes][0][name]' => "c1"
     get "parents/1/edit"
 
-    assert_html '<form accept-charset="UTF-8" action="/parents/1" class="edit_parent" enctype="multipart/form-data" id="edit_parent_1" method="post">'+
-      '<div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /><input name="_method" type="hidden" value="put" />'+
-      '</div><h1>Parent</h1><input id="parent_name" name="parent[name]" size="30" type="text" value="p1" />'+
-      '<h2>Children</h2>'+
-      '<ul><li><input id="parent_children_attributes_0_name" name="parent[children_attributes][0][name]" size="30" type="text" value="c1" /></li>'+
-      '<input id="parent_children_attributes_0_id" name="parent[children_attributes][0][id]" type="hidden" value="1" /></ul>'+
-      '</form>'
+    assert_match(%r{<form accept-charset="UTF-8" action="/parents/1" class="edit_parent" enctype="multipart/form-data" id="edit_parent_1" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /><input name="_method" type="hidden" value="[^"]+" /></div><h1>Parent</h1><input id="parent_name" name="parent\[name\]" size="30" type="text" value="p1" /><h2>Children</h2><ul><li><input id="parent_children_attributes_0_name" name="parent\[children_attributes\]\[0\]\[name\]" size="30" type="text" value="c1" /></li><input id="parent_children_attributes_0_id" name="parent\[children_attributes\]\[0\]\[id\]" type="hidden" value="1" /></ul></form>}, @response.body)
   end
 
   protected

data/test/slim/helper.rb

@@ -9,7 +9,7 @@ MiniTest::Unit.autorun
 
 Slim::Engine.after  Slim::Parser, Temple::Filters::Validator, :grammar => Slim::Grammar
 Slim::Engine.before Slim::Compiler, Temple::Filters::Validator, :grammar => Slim::Grammar
-Slim::Engine.before Temple::HTML::Pretty, Temple::Filters::Validator
+Slim::Engine.before :Pretty, Temple::Filters::Validator
 
 class TestSlim < MiniTest::Unit::TestCase
   def setup
@@ -72,7 +72,7 @@ class TestSlim < MiniTest::Unit::TestCase
 end
 
 class Env
-  attr_reader :var
+  attr_reader :var, :x
 
   class ::HtmlSafeString < String
     def html_safe?
@@ -88,6 +88,7 @@ class Env
 
   def initialize
     @var = 'instance'
+    @x = 0
   end
 
   def id_helper
@@ -144,6 +145,11 @@ class Env
   def output_number
     1337
   end
+
+  def succ_x
+    @x = @x.succ
+  end
+
 end
 
 class ViewEnv

data/test/slim/test_chain_manipulation.rb

@@ -6,7 +6,7 @@ class TestSlimChainManipulation < TestSlim
 p Test
 }
     chain = proc do |engine|
-      engine.replace(Temple::HTML::Pretty, :ReplacementFilter) do |exp|
+      engine.replace(:Pretty, :ReplacementFilter) do |exp|
         [:dynamic, '1+1']
       end
     end

data/test/slim/test_code_output.rb

@@ -149,6 +149,15 @@ p = \
     assert_html '<p>Hello Ruby!</p>7', source
   end
 
+  def test_render_with_comma_end
+    source = %q{
+p = message("Hello",
+            "Ruby!")
+}
+
+    assert_html '<p>Hello Ruby!</p>', source
+  end
+
   def test_render_with_no_trailing_character
     source = %q{
 p

data/test/slim/test_html_structure.rb

@@ -459,4 +459,13 @@ html: body: .content
 }
     assert_html %{<html><body><div class=\"content\">Text</div></body></html>}, source
   end
+
+  def test_eval_attributes_once
+    source = %q{
+input[value=succ_x]
+input[value=succ_x]
+}
+    assert_html %{<input value="1" /><input value="2" />}, source
+  end
+
 end

data/test/slim/test_parser_errors.rb

@@ -26,7 +26,23 @@ p
    text
 }
 
-    assert_syntax_error "Unexpected text indentation\n  (__TEMPLATE__), Line 4\n    text\n    ^\n", source
+    assert_syntax_error "Text line not indented deep enough.\nThe first text line defines the necessary text indentation.\n  (__TEMPLATE__), Line 4\n    text\n    ^\n", source
+  end
+
+  def test_unexpected_text_indentation_in_tag
+    source = %q{
+ul
+  li List1
+    ul
+      li a
+      li b
+  li List2
+    ul
+      li a
+      li b
+}
+
+    assert_syntax_error "Text line not indented deep enough.\nThe first text line defines the necessary text indentation.\nAre you trying to nest a child tag in a tag containing text? Use | for the text block!\n  (__TEMPLATE__), Line 4\n    ul\n    ^\n", source
   end
 
   def test_malformed_indentation

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slim
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -11,22 +11,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-06 00:00:00.000000000 Z
+date: 2012-02-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: temple
-  requirement: &2152139880 !ruby/object:Gem::Requirement
+  requirement: &17637820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.3.5
+        version: 0.4.0
   type: :runtime
   prerelease: false
-  version_requirements: *2152139880
+  version_requirements: *17637820
 - !ruby/object:Gem::Dependency
   name: tilt
-  requirement: &2152135380 !ruby/object:Gem::Requirement
+  requirement: &17637240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -34,10 +34,10 @@ dependencies:
         version: 1.3.2
   type: :runtime
   prerelease: false
-  version_requirements: *2152135380
+  version_requirements: *17637240
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2152133280 !ruby/object:Gem::Requirement
+  requirement: &17636740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -45,10 +45,10 @@ dependencies:
         version: 0.8.7
   type: :development
   prerelease: false
-  version_requirements: *2152133280
+  version_requirements: *17636740
 - !ruby/object:Gem::Dependency
   name: sass
-  requirement: &2152132380 !ruby/object:Gem::Requirement
+  requirement: &17636160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -56,10 +56,10 @@ dependencies:
         version: 3.1.0
   type: :development
   prerelease: false
-  version_requirements: *2152132380
+  version_requirements: *17636160
 - !ruby/object:Gem::Dependency
   name: minitest
-  requirement: &2152131200 !ruby/object:Gem::Requirement
+  requirement: &17652060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -67,10 +67,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2152131200
+  version_requirements: *17652060
 - !ruby/object:Gem::Dependency
   name: kramdown
-  requirement: &2152129720 !ruby/object:Gem::Requirement
+  requirement: &17651560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -78,10 +78,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2152129720
+  version_requirements: *17651560
 - !ruby/object:Gem::Dependency
   name: creole
-  requirement: &2152128100 !ruby/object:Gem::Requirement
+  requirement: &17650940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -89,10 +89,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2152128100
+  version_requirements: *17650940
 - !ruby/object:Gem::Dependency
   name: builder
-  requirement: &2152125140 !ruby/object:Gem::Requirement
+  requirement: &17649640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -100,7 +100,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2152125140
+  version_requirements: *17649640
 description: Slim is a template language whose goal is reduce the syntax to the essential
   parts without becoming cryptic.
 email:
@@ -226,9 +226,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: slim
-rubygems_version: 1.8.10
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: Slim is a template language.
 test_files: []
-has_rdoc: