slim 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1a7b030e5d1e59697c83e105cd3fabced713e020
-  data.tar.gz: da7d8ed9b48d1e8a014334395b076d36160ed5cd
+  metadata.gz: eeabe6b2a2349dba5997fc11b9f8308b8c059b64
+  data.tar.gz: bf7ecd8073af2e89a0ee3cf413ac8db8bdf2392f
 SHA512:
-  metadata.gz: c82fb688b72606a09154716304290a05ee4f44d98e868d6a13c194f84a2061e612b9012b7cc262de14410c5db1ec0a1de549503f0b92d78aee66f43fac485406
-  data.tar.gz: a253b0c4833acdba6452540810784770227ed1dcad883d4f499f8a1e6c4329edf80cb4f9c98402075a5bd3ff7d097a82b509ad30789ef047d69decd4f014d3f7
+  metadata.gz: e84616653fbc224fbff5e4ce111a38d38afaa75a20a70f6f7c0428e65a225ca23228a4b0bf9c26d01784b3ee919d476cd369aaffdfe602d13a2cc2c8aa04cb1e
+  data.tar.gz: 122cfa59ce81ad34e11d67b9988768b69fa7a9cdb18bacd02a0a4d1f722631b7cfa5a22985dcde14c743fa99cb8726dda8a8a8b9d264c47066d55b8eb3fef289

data/.travis.yml

@@ -22,6 +22,7 @@ env:
   - "TASK=test:rails RAILS=3.2.21 TILT=1.3.4"
   - "TASK=test:rails RAILS=4.0.12"
   - "TASK=test:rails RAILS=4.1.8"
+  - "TASK=test:rails RAILS=4.2.0"
   - "TASK=test:sinatra SINATRA=master"
   - "TASK=test:sinatra SINATRA=1.3.6"
   - "TASK=test:sinatra SINATRA=1.4.5"

data/CHANGES

@@ -1,3 +1,8 @@
+3.0.1
+
+  * Allow more special characters in html attribute names (See https://html.spec.whatwg.org/multipage/syntax.html#attributes-2), #567
+  * Fix: Code attributes mutate their argument (#571)
+
 3.0.0
 
   * Drop 1.8.7 support

data/README.jp.md

@@ -199,14 +199,14 @@ body
 
 行末がカンマ `,` で終わる場合 (例 関数呼び出し) には行末にバックスラッシュを追加する必要はありません。行末スペースを追加するために修飾子の `>` や `<` もサポートします。
 
-* `=>` は末尾のスペースを伴った出力をします。 末尾のスペースが追加されることを除いて, 単一の等合 (`=`) と同じです。古い構文の `='` も同様にサポートされます。
+* `=>` は末尾のスペースを伴った出力をします。 末尾のスペースが追加されることを除いて, 単一の等合 (`=`) と同じです。
 * `=<` は先頭のスペースを伴った出力をします。先頭のスペースが追加されることを除いて, 単一の等号 (`=`) と同じです。
 
 ### HTML エスケープを伴わない出力 `==`
 
 単一のイコール (`=`) と同じですが, `escape_html` メソッドを経由しません。 末尾や先頭のスペースを追加するための修飾子 `>` と `<` はサポートされています。
 
-* `==>` は HTML エスケープを行わず末尾のスペースを伴った出力をします。末尾のスペースが追加されることを除いて, 二重等号 (`==`) と同じです。 古い構文の `=='` もサポートされます。
+* `==>` は HTML エスケープを行わず末尾のスペースを伴った出力をします。末尾のスペースが追加されることを除いて, 二重等号 (`==`) と同じです。
 * `==<` は HTML エスケープを行わず先頭のスペースを伴った出力をします。先頭のスペースが追加されることを除いて, 二重等号 (`==`) と同じです。
 
 ### コードコメント `/`
@@ -870,10 +870,7 @@ Slim を設定する方法はコンパイル機構に少し依存します。(Ra
 ### デフォルトオプション
 
 ~~~ ruby
-# デバック用に html をきれいにインデントし属性をソートしない (Ruby 1.8)
-Slim::Engine.set_options pretty: true. sort_attrs: false
-
-# デバック用に html をきれいにインデントし属性をソートしない (Ruby 1.9)
+# デバック用に html をきれいにインデントし属性をソートしない
 Slim::Engine.set_options pretty: true, sort_attrs: false
 ~~~
 
@@ -944,7 +941,7 @@ end
 | シンボル | :js_wrapper | nil | :comment,  :cdata や :both で JavaScript をラップします。:guess を指定することで :format オプションに基いて設定することもできます |
 | 真偽値 | :pretty | false | 綺麗な html インデント <b>(遅くなります!)</b> |
 | 文字列 | :indent | '  ' | インデントに使用される文字列 |
-| 真偽値 | :streaming | false (Rails では true) | ストリーミング出力の有効化 |
+| 真偽値 | :streaming | false (Rails では true, 無効化するにはストリーミングを参照) | ストリーミング出力の有効化, 体感的なパフォーマンスの向上 |
 | Class | :generator | Temple::Generators::ArrayBuffer/ RailsOutputBuffer | Temple コードジェネレータ (デフォルトのジェネレータは配列バッファを生成します) |
 | 文字列 | :buffer | '_buf' (Rails では '@output_buffer') | バッファに使用される変数 |
 
@@ -1015,12 +1012,17 @@ html
 ### Rails
 
 Rails のジェネレータは [slim-rails](https://github.com/slim-template/slim-rails) によって提供されます。
-slim-rails は Rails で Slim を使用する場合に必須ではありません。Slim をインストールし Gemfile に `gem 'slim'` を追加するだけです。
+slim-rails は Rails で Slim を使用する場合に必須ではありません。Slim をインストールし Gemfile に `gem 'slim-rails'` を追加するだけです。
 後は .slim 拡張子を使えば Rails で使用できます。
 
-#### Streaming
+#### ストリーミング
+
+HTTP ストリーミングは Rails がそれをサポートしているバージョンであればデフォルトで有効化されています。しかし, ストリーミングが体感的なパフォーマンスを改善していることを知る必要があります。
+レンダリング時間は増加するでしょう。ストリーミングを無効化したい場合の設定です:
 
-HTTP ストリーミングは Rails がそれをサポートしているバージョンであればデフォルトで有効化されています。
+~~~ ruby
+Slim::RailsTemplate.set_options streaming: false
+~~~
 
 ## ツール
 
@@ -1113,7 +1115,7 @@ Travis-CI は継続的インテグレーションテストに利用されてい
 
 Slim はすべての主要な Ruby 実装で動作します:
 
-* Ruby 1.8.7, 1.9.3, 2.0.0 および 2.1.0
+* Ruby 1.9.3, 2.0.0 および 2.1.0
 * Ruby EE
 * JRuby 1.9 mode
 * Rubinius 2.0

data/README.md

@@ -1,6 +1,6 @@
 # Slim
 
-[![Gem Version](https://badge.fury.io/rb/slim.png)](http://rubygems.org/gems/slim) [![Build Status](https://secure.travis-ci.org/slim-template/slim.png?branch=master)](http://travis-ci.org/slim-template/slim) [![Dependency Status](https://gemnasium.com/slim-template/slim.png?travis)](https://gemnasium.com/slim-template/slim) [![Code Climate](https://codeclimate.com/github/slim-template/slim.png)](https://codeclimate.com/github/slim-template/slim) [![Gittip donate button](http://img.shields.io/gittip/bevry.png)](https://www.gittip.com/min4d/ "Donate weekly to this project using Gittip")
+[![Gem Version](https://img.shields.io/gem/v/slim.svg)](http://rubygems.org/gems/slim) [![Build Status](https://img.shields.io/travis/slim-template/slim.svg?branch=master)](http://travis-ci.org/slim-template/slim) [![Dependency Status](https://img.shields.io/gemnasium/slim-template/slim.svg?travis)](https://gemnasium.com/slim-template/slim) [![Code Climate](https://img.shields.io/codeclimate/github/slim-template/slim.svg)](https://codeclimate.com/github/slim-template/slim) [![Gittip donate button](https://img.shields.io/gratipay/bevry.svg)](https://www.gittip.com/min4d/ "Donate weekly to this project using Gittip")
 [![Flattr donate button](https://raw.github.com/balupton/flattr-buttons/master/badge-89x18.gif)](https://flattr.com/submit/auto?user_id=min4d&url=http%3A%2F%2Fslim-lang.org%2F "Donate monthly to this project using Flattr")
 
 Slim is a template language whose goal is to reduce the view syntax to the essential parts without becoming cryptic. It started as an exercise to see how much could be removed from a standard html template (<, >, closing tags, etc...). As more people took an interest in Slim, the functionality grew and so did the flexibility of the syntax.
@@ -1148,7 +1148,7 @@ Slim is released under the [MIT license](http://www.opensource.org/licenses/MIT)
 
 If you want to support this project please visit the Gittip and Flattr pages.
 
-[![Gittip donate button](http://img.shields.io/gittip/bevry.png)](https://www.gittip.com/min4d/ "Donate weekly to this project using Gittip")
+[![Gittip donate button](https://img.shields.io/gratipay/bevry.svg)](https://www.gittip.com/min4d/ "Donate weekly to this project using Gittip")
 [![Flattr donate button](https://raw.github.com/balupton/flattr-buttons/master/badge-89x18.gif)](https://flattr.com/submit/auto?user_id=min4d&url=http%3A%2F%2Fslim-lang.org%2F "Donate monthly to this project using Flattr")
 
 Currently the donations will be used to cover the hosting costs (domain name etc).

data/lib/slim/code_attributes.rb

@@ -54,7 +54,7 @@ module Slim
          [:code, "#{tmp} = #{code}"],
          [:if, "Array === #{tmp}",
           [:multi,
-           [:code, "#{tmp}.flatten!"],
+           [:code, "#{tmp} = #{tmp}.flatten"],
            [:code, "#{tmp}.map!(&:to_s)"],
            [:code, "#{tmp}.reject!(&:empty?)"],
            [:escape, escape, [:dynamic, "#{tmp}.join(#{delimiter.inspect})"]]],

data/lib/slim/interpolation.rb

@@ -18,9 +18,9 @@ module Slim
           # Escaped interpolation
           block << [:static, '#{']
           string = $'
-        when /\A#\{/
+        when /\A#\{((?>[^{}]|(\{(?>[^{}]|\g<1>)*\}))*)\}/
           # Interpolation
-          string, code = parse_expression($')
+          string, code = $', $1
           escape = code !~ /\A\{.*\}\Z/
           block << [:slim, :output, escape, escape ? code : code[1..-2], [:multi]]
         when /\A([#\\]?[^#\\]*([#\\][^\\#\{][^#\\]*)*)/
@@ -31,23 +31,5 @@ module Slim
       end until string.empty?
       block
     end
-
-    protected
-
-    def parse_expression(string)
-      count, i = 1, 0
-      while i < string.size && count != 0
-        if string[i] == ?{
-          count += 1
-        elsif string[i] == ?}
-          count -= 1
-        end
-        i += 1
-      end
-
-      raise(Temple::FilterError, "Text interpolation: Expected closing }") if count != 0
-
-      return string[i..-1], string[0, i-1]
-    end
   end
 end

data/lib/slim/parser.rb

@@ -72,8 +72,10 @@ module Slim
       @code_attr_delims_re = /\A[#{keys}]/
       keys = Regexp.escape @attr_list_delims.keys.join
       @attr_list_delims_re = /\A\s*([#{keys}])/
-      # Access available engine keys to allow nicer one-line syntax
       @embedded_re = /\A(#{Regexp.union(Embedded.engines.keys.map(&:to_s))}):(\s*)/
+      @attr_name = "\\A\\s*([^\0\"'><\/=\s#{(@attr_list_delims.flatten + @code_attr_delims.flatten).map {|x| Regexp.escape(x) }.join}]+)"
+      @quoted_attr_re = /#{@attr_name}\s*=(=?)\s*("|')/
+      @code_attr_re = /#{@attr_name}\s*=(=?)\s*/
     end
 
     # Compile string to Temple expression
@@ -92,10 +94,6 @@ module Slim
 
     protected
 
-    ATTR_NAME = '\\A\\s*(\p{Word}(?:\p{Word}|:|-)*)'
-    QUOTED_ATTR_RE = /#{ATTR_NAME}\s*=(=?)\s*("|')/
-    CODE_ATTR_RE = /#{ATTR_NAME}\s*=(=?)\s*/
-
     def reset(lines = nil, stacks = nil)
       # Since you can indent however you like in Slim, we need to keep a list
       # of how deeply indented you are. For instance, in a template like this:
@@ -235,7 +233,7 @@ module Slim
       when @embedded_re
         # Embedded template detected. It is treated as block.
         @stacks.last << [:slim, :embedded, $1, parse_text_block($', @orig_line.size - $'.size + $2.size)]
-      when /\Adoctype\s+/
+      when /\Adoctype\b/
         # Found doctype declaration
         @stacks.last << [:html, :doctype, $'.strip]
       when @tag_re
@@ -398,7 +396,7 @@ module Slim
       end
 
       if delimiter
-        boolean_attr_re = /#{ATTR_NAME}(?=(\s|#{Regexp.escape delimiter}|\Z))/
+        boolean_attr_re = /#{@attr_name}(?=(\s|#{Regexp.escape delimiter}|\Z))/
         end_re = /\A\s*#{Regexp.escape delimiter}/
       end
 
@@ -408,12 +406,12 @@ module Slim
           # Splat attribute
           @line = $'
           attributes << [:slim, :splat, parse_ruby_code(delimiter)]
-        when QUOTED_ATTR_RE
+        when @quoted_attr_re
           # Value is quoted (static)
           @line = $'
           attributes << [:html, :attr, $1,
                          [:escape, $2.empty?, [:slim, :interpolate, parse_quoted_attribute($3)]]]
-        when CODE_ATTR_RE
+        when @code_attr_re
           # Value is ruby code
           @line = $'
           name = $1
@@ -492,9 +490,7 @@ module Slim
         end
       end
 
-      syntax_error!("Expected closing brace }") if count != 0
       @line.slice!(0)
-
       value
     end
 

data/lib/slim/version.rb

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

data/test/core/test_code_evaluation.rb

@@ -162,6 +162,14 @@ p id=(1 + 1)*5 Test it
     assert_html '<p id="10">Test it</p>', source
   end
 
+  def test_code_attribute_does_not_modify_argument
+    template = 'span class=attribute'
+    model = OpenStruct.new(attribute: [:a, :b, [:c, :d]])
+    output = Slim::Template.new { template }.render(model)
+    assert_equal('<span class="a b c d"></span>', output)
+    assert_equal([:a, :b, [:c, :d]], model.attribute)
+  end
+
   def test_number_type_interpolation
     source = %q{
 p = output_number

data/test/core/test_html_attributes.rb

@@ -256,4 +256,13 @@ p(id="marvin" class=nil nonempty=("".to_s) data-info="Illudium Q-36")= output_nu
 
     assert_html '<p data-info="Illudium Q-36" id="marvin" nonempty="">1337</p>', source
   end
+
+  def test_weird_attribute
+    source = %q{
+p
+  img(src='img.png' whatsthis?!)
+  img src='img.png' whatsthis?!="wtf"
+}
+    assert_html '<p><img src="img.png" whatsthis?!="" /><img src="img.png" whatsthis?!="wtf" /></p>', source
+  end
 end

data/test/core/test_parser_errors.rb

@@ -77,15 +77,6 @@ p
     assert_syntax_error "Expected closing delimiter ]\n  (__TEMPLATE__), Line 3, Column 28\n    img src=[hash[1] + hash[2]\n                              ^\n", source
   end
 
-  def test_expected_attribute
-    source = %q{
-p
-  img(src='img.png' whatsthis?!)
-}
-
-    assert_syntax_error "Expected attribute\n  (__TEMPLATE__), Line 3, Column 20\n    img(src='img.png' whatsthis?!)\n                      ^\n", source
-  end
-
   def test_invalid_empty_attribute
     source = %q{
 p

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slim
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - Daniel Mendler
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-07 00:00:00.000000000 Z
+date: 2014-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: temple
@@ -201,4 +201,3 @@ signing_key:
 specification_version: 4
 summary: Slim is a template language.
 test_files: []
-has_rdoc: