cells 4.0.0 → 4.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 22c7de3a30ac46478e9d4f4ef37c4b8fe1b49b17
-  data.tar.gz: 00b8a25633e2495ff4368ffe53e0c8f404ba7d8d
+  metadata.gz: 10248a40af9eca0d333368d6df675d88f77be026
+  data.tar.gz: b4fb151ae10270795f57f5a418d3e552424a2631
 SHA512:
-  metadata.gz: 991d17fcb0eae9e9261a6f978651601eb2e99053ac95dc5a3621054ccfaf6ebb0003fb628b8d6a974bc6f8af668ba82a9d7de2f66c80f33454f070301bf87efc
-  data.tar.gz: 9cc7f11fe2703b49721d3a222637d48f13ace7323d7399e876e04e3c43b688fff1419de2c586b01cc342addcedb16f638325590efd76378578114f3e56704ae9
+  metadata.gz: bc18a9d14431f421008e3b758d2ac9e567c09cde38330fb0be8264c33682f98e3d32b38f96d1efce0111ff0b3981e60e8262d02ade5d0c5ac83573b6fe675dbc
+  data.tar.gz: 3ba936fe9fa477efdf3504c6d2682f8251479b9cacf894bff4b5b95bac5025cc39c67bbc2fd005bf32febd6d519a833d366d299387336db9b281f996ae5cab8a

data/CHANGES.md

@@ -1,3 +1,7 @@
+## 4.0.1
+
+* Support forgery protection in `form_tag`.
+
 ## 4.0.0
 
 * **Rails Support:** Rails 4.0+ is fully supported, in older versions some form helpers do not work. Let us know how you fixed this.

data/README.md

@@ -46,7 +46,7 @@ Usually, you'd pass in one or more objects you want the cell to present. That ca
 A cell is a light-weight class with one or multiple methods that render views.
 
 ```ruby
-class Comment::Cell < Cell::ViewModel
+class CommentCell < Cell::ViewModel
   property :body
   property :author
 

data/lib/cell/caching/notification.rb

@@ -2,9 +2,9 @@ module Cell
   module Caching
     module Notifications
       def fetch_from_cache_for(key, options)
-        ActiveSupport::Notifications.instrument('read_fragment.cells', :key => key) do
+        ActiveSupport::Notifications.instrument('read_fragment.cells', key: key) do
           cache_store.fetch(key, options) do
-            ActiveSupport::Notifications.instrument('write_fragment.cells', :key => key) do
+            ActiveSupport::Notifications.instrument('write_fragment.cells', key: key) do
               yield
             end
           end

data/lib/cell/partial.rb

@@ -12,6 +12,6 @@ module Cell::ViewModel::Partial
     view     += ".#{options[:formats].first}" if options[:formats]
     prefixes  = self.class.view_paths.collect { |path| parts.unshift(path).join("/") }
 
-    options.merge!(:view => view, :prefixes => prefixes)
+    options.merge!(view: view, prefixes: prefixes)
   end
 end

data/lib/cell/rails.rb

@@ -15,7 +15,7 @@ module Cell
       # Returns the cell instance for +name+. You may pass arbitrary options to your
       # cell.
       #
-      #   = cell(:song, :title => "Creeping Out Sara").render(:show)
+      #   = cell(:song, title: "Creeping Out Sara").render(:show)
       def cell(name, *args, &block)
         controller.cell(name, *args, &block)
       end
@@ -39,10 +39,8 @@ module Cell
       included do
         extend Uber::Delegates
         delegates :parent_controller, :session, :params, :request, :config, :env, :url_options
-      end
-
-      def protect_against_forgery? # TODO: implement forgery protection with ActionController.
-        false
+        # forgery protection.
+        delegates :parent_controller, :protect_against_forgery?, :form_authenticity_token, :request_forgery_protection_token
       end
 
       def call(*)

data/lib/cell/version.rb

@@ -1,3 +1,3 @@
 module Cell
-  VERSION = "4.0.0"
+  VERSION = "4.0.1"
 end

data/lib/cell/view_model.rb

@@ -129,7 +129,7 @@ module Cell
       def with_layout(options, content)
         return content unless layout = options[:layout]
 
-        template = find_template(options.merge :view => layout) # we could also allow a different layout engine, etc.
+        template = find_template(options.merge view: layout) # we could also allow a different layout engine, etc.
 
         render_template(template, options) { content }
       end
@@ -200,9 +200,9 @@ module Cell
     def normalize_options(options, caller) # TODO: rename to #setup_options! to be inline with Trb.
       options = if options.is_a?(Hash)
         # TODO: speedup by not doing state_for_implicit_render.
-        {:view => state_for_implicit_render(caller)}.merge(options)
+        {view: state_for_implicit_render(caller)}.merge(options)
       else
-        {:view => options.to_s}
+        {view: options.to_s}
       end
 
       options[:prefixes] ||= _prefixes

data/test/builder_test.rb

@@ -45,7 +45,7 @@ class BuilderTest < MiniTest::Spec
   it do
     cell = SongCell.(Song.new("San Francisco"), evergreen: true)
     cell.must_be_instance_of EvergreenCell
-    cell.options.must_equal({:evergreen=>true})
+    cell.options.must_equal({evergreen:true})
   end
 
   # without arguments.

data/test/caching_test.rb

@@ -104,10 +104,10 @@ class CachingUnitTest < MiniTest::Spec
   #   end
 
   #   it "accept cache options" do
-  #     key = @class.state_cache_key(:tock, :volume => 9)
+  #     key = @class.state_cache_key(:tock, volume: 9)
   #     assert Cell::Rails.cache_store.write(key, 'ONE!')
 
-  #     MusicianController.new.expire_cell_state(DirectorCell, :tock, :volume => 9)
+  #     MusicianController.new.expire_cell_state(DirectorCell, :tock, volume: 9)
   #     assert_equal "1", @class.cache_store.read(@key)
   #     assert_not ::Cell::Rails.cache_store.read(key)
   #   end
@@ -183,7 +183,7 @@ class CachingTest < MiniTest::Spec
   # compute key with cell properties from #initialize.
   it do
     director_cell.class.cache :show do
-      @counter < 3 ? {:count => "<"} : {:count => ">"}
+      @counter < 3 ? {count: "<"} : {count: ">"}
     end
 
     director_cell(1).call.must_equal "1"
@@ -197,7 +197,7 @@ class CachingTest < MiniTest::Spec
     director_cell.class.cache :show, :version
     director_cell.class.class_eval do
       def version
-        @counter < 3 ? {:count => "<"} : {:count => ">"}
+        @counter < 3 ? {count: "<"} : {count: ">"}
       end
     end
 
@@ -261,7 +261,7 @@ class CachingTest < MiniTest::Spec
 
   # options are passed through to cache store.
   # :expires_in.
-  # :tags => lambda { |one, two, three| "#{one},#{two},#{three}" }
+  # tags: lambda { |one, two, three| "#{one},#{two},#{three}" }
   class CacheStore
     attr_reader :fetch_args
 
@@ -288,7 +288,7 @@ end
 
 class CachingInheritanceTest < CachingTest
   class DirectorCell < ::DirectorCell
-    cache :show, :expires_in => 10.minutes do
+    cache :show, expires_in: 10.minutes do
       "v1"
     end
   end
@@ -297,7 +297,7 @@ class CachingInheritanceTest < CachingTest
   end
 
   class DirectorsDaughterCell < ::DirectorCell
-    cache :show, :expires_in => 9.minutes do
+    cache :show, expires_in: 9.minutes do
       "v2"
     end
   end

data/test/concept_test.rb

@@ -16,7 +16,7 @@ module Record
     # cell(:song, concept: :record)
     class Song < self # cell("record/cell/song")
       def show
-        render :view => :song#, :layout => "layout"
+        render view: :song#, layout: "layout"
         # TODO: test layout: .. in ViewModel
       end
     end

data/test/public_test.rb

@@ -20,7 +20,7 @@ class PublicTest < MiniTest::Spec
   it { Cell::ViewModel.cell("public_test/song").must_be_instance_of SongCell }
 
   # ViewModel.cell passes options to cell.
-  it { Cell::ViewModel.cell("public_test/song", Object, genre: "Metal").initialize_args.must_equal [Object, {:genre=>"Metal"}] }
+  it { Cell::ViewModel.cell("public_test/song", Object, genre: "Metal").initialize_args.must_equal [Object, {genre:"Metal"}] }
 
   # ViewModel.cell(collection: []) renders cells.
   it { Cell::ViewModel.cell('public_test/song', collection: [Object, Module]).must_equal '[Object, {}][Module, {}]' }

data/test/render_test.rb

@@ -95,10 +95,4 @@ class RenderTest < MiniTest::Spec
 end
 
 # test inheritance
-
-# test view: :bla and :bla
-# with layout and locals.
-# with layout and :text
-
-# render with format (e.g. when using ERB for one view)
-# should we allow changing the format "per run", so a cell can do .js and .haml? or should that be configurable on class level?
+# with layout and locals.

data/test/test_helper.rb

@@ -22,19 +22,19 @@ end
 
 class MusicianController < ActionController::Base
   def view_with_concept_with_show
-    render :inline => %{<%= concept("view_extensions_test/cell", "Up For Breakfast", volume: 1).show %>} # TODO: concept doesn't need .call
+    render inline: %{<%= concept("view_extensions_test/cell", "Up For Breakfast", volume: 1).show %>} # TODO: concept doesn't need .call
   end
 
   def view_with_concept_without_call
-    render :inline => %{<%= concept("view_extensions_test/cell", "A Tale That Wasn't Right") %>} # this tests ViewModel#to_s.
+    render inline: %{<%= concept("view_extensions_test/cell", "A Tale That Wasn't Right") %>} # this tests ViewModel#to_s.
   end
 
   def view_with_concept_with_call
-    render :inline => %{<%= concept("view_extensions_test/cell", "A Tale That Wasn't Right").call %>}
+    render inline: %{<%= concept("view_extensions_test/cell", "A Tale That Wasn't Right").call %>}
   end
 
   def view_with_cell_with_call
-    render :inline => %{<%= cell("view_extensions_test/song", "A Tale That Wasn't Right").call %>}
+    render inline: %{<%= cell("view_extensions_test/song", "A Tale That Wasn't Right").call %>}
   end
 
   def action_with_concept_with_call

data/test/twin_test.rb

@@ -20,7 +20,7 @@
 #     end
 #   end
 
-#   let (:model) { OpenStruct.new(:title => "Kenny") }
+#   let (:model) { OpenStruct.new(title: "Kenny") }
 
 #   it { SongCell.new( model, :online? => true).call.must_equal "kenny is true" }
 # end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cells
 version: !ruby/object:Gem::Version
-  version: 4.0.0
+  version: 4.0.1
 platform: ruby
 authors:
 - Nick Sutterer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-09 00:00:00.000000000 Z
+date: 2015-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: uber