jbuilder 2.0.8 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c6836a60ee9c7dd9709201026f7a6b472cc2019e
-  data.tar.gz: 4a8069c61889a929e3ffa10b3d46b799399202a0
+  metadata.gz: a389ec3a309016718e77b531bb60b89aa18b5734
+  data.tar.gz: 0b749ea3555366e70c5f99d7697072e9f9badb27
 SHA512:
-  metadata.gz: 774ae98b86547a5fd8261a3452e5ceb188924b9e84cf52d7c1f9c6e5f7ebfe6fb93816ef04b9f5f2aeb1f65d2136ed4d13b466957bd0cf65b3ead3a80515856f
-  data.tar.gz: 4aeb49c248572ceedfac7fab0bfc3d763607ead19be5dda7362d93e1bd87ddb0a1ea8b47f846747c61afb13b358c7ddc772e8d22542863ddbb948a0781fc61dd
+  metadata.gz: a807a3a810d46fa3c6b16a1d171d6f94f765aee7dfb93da2a435160a26a4ac3e921a34783e80738cfc5b30eadfd3fffcbf074ce2e6cba801c2f7332c373c75a6
+  data.tar.gz: 3409ea860fbff36986ba86572d9db4690f1328d83d3876b19fb6c97744c15b1df7cd80f953fa4f2a231325c6dfe6656dfdee9f64619e917fb608248c096efb35

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+2.1.0
+-----
+* [Blocks and their extract! shortcuts are additive by default](https://github.com/rails/jbuilder/commit/a49390736c5f6e2d7a31111df6531bc28dba9fb1)
+
 2.0.8
 -----
 * [Eliminate circular dependencies](https://github.com/rails/jbuilder/commit/0879484dc74e7be93b695f66e3708ba48cdb1be3)

data/jbuilder.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name     = 'jbuilder'
-  s.version  = '2.0.8'
+  s.version  = '2.1.0'
   s.authors  = ['David Heinemeier Hansson', 'Pavel Pravosud']
   s.email    = ['david@37signals.com', 'pavel@pravosud.com']
   s.summary  = 'Create JSON structures via a Builder-style DSL'

data/lib/jbuilder.rb

@@ -32,7 +32,7 @@ class Jbuilder
       else
         # json.comments { ... }
         # { "comments": ... }
-        _scope{ yield self }
+        _merge_block(key){ yield self }
       end
     elsif args.empty?
       if ::Jbuilder === value
@@ -52,7 +52,7 @@ class Jbuilder
     else
       # json.author @post.creator, :name, :email_address
       # { "author": { "name": "David", "email_address": "david@loudthinking.com" } }
-      _scope{ extract! value, *args }
+      _merge_block(key){ extract! value, *args }
     end
 
     _set_value key, result
@@ -174,13 +174,15 @@ class Jbuilder
   #
   #   [1,2,3]
   def array!(collection = [], *attributes, &block)
-    @attributes = if block
+    array = if block
       _map_collection(collection, &block)
     elsif attributes.any?
       _map_collection(collection) { |element| extract! element, *attributes }
     else
       collection
     end
+
+    merge! array
   end
 
   # Extracts the mentioned attributes or hash elements from the passed object and turns them into attributes of the JSON.
@@ -230,12 +232,7 @@ class Jbuilder
 
   # Merges hash or array into current builder.
   def merge!(hash_or_array)
-    if ::Array === hash_or_array
-      @attributes = [] unless ::Array === @attributes
-      @attributes.concat hash_or_array
-    else
-      @attributes.update hash_or_array
-    end
+    @attributes = _merge_values(@attributes, hash_or_array)
   end
 
   # Encodes the current builder as JSON.
@@ -253,12 +250,29 @@ class Jbuilder
     attributes.each{ |key| _set_value key, object.public_send(key) }
   end
 
+  def _merge_block(key, &block)
+    current_value = _read(key, {})
+    raise NullError.build(key) if current_value.nil?
+    value = _scope{ yield self }
+    value.nil? ? value : _merge_values(current_value, value)
+  end
+
+  def _read(key, default = nil)
+    @attributes.fetch(_key(key)){ default }
+  end
+
+  def _write(key, value)
+    @attributes[_key(key)] = value
+  end
+
+  def _key(key)
+    @key_formatter.format(key)
+  end
+
   def _set_value(key, value)
     raise NullError.build(key) if @attributes.nil?
     return if @ignore_nil && value.nil?
-
-    key = @key_formatter.format(key)
-    @attributes[key] = value
+    _write key, value
   end
 
   def _map_collection(collection)
@@ -281,6 +295,19 @@ class Jbuilder
   def _mapable_arguments?(value, *args)
     value.respond_to?(:map)
   end
+
+  def _merge_values(attributes, hash_or_array)
+    attributes = attributes.dup
+
+    if ::Array === hash_or_array
+      attributes = [] unless ::Array === attributes
+      attributes.concat hash_or_array
+    else
+      attributes.update hash_or_array
+    end
+
+    attributes
+  end
 end
 
 require 'jbuilder/jbuilder_template' if defined?(ActionView::Template)

data/test/jbuilder_test.rb

@@ -121,6 +121,42 @@ class JbuilderTest < ActiveSupport::TestCase
     assert_equal 32, result['author']['age']
   end
 
+  test 'blocks are additive' do
+    result = jbuild do |json|
+      json.author do
+        json.name 'David'
+      end
+
+      json.author do
+        json.age  32
+      end
+    end
+
+    assert_equal 'David', result['author']['name']
+    assert_equal 32, result['author']['age']
+  end
+
+  test 'blocks are additive via extract syntax' do
+    person = Person.new('Pavel', 27)
+
+    result = jbuild do |json|
+      json.author person, :age
+      json.author person, :name
+    end
+
+    assert_equal 'Pavel', result['author']['name']
+    assert_equal 27, result['author']['age']
+  end
+
+  test 'arrays are additive' do
+    result = jbuild do |json|
+      json.array! %w[foo]
+      json.array! %w[bar]
+    end
+
+    assert_equal %w[foo bar], result
+  end
+
   test 'nesting multiple children with block' do
     result = jbuild do |json|
       json.comments do
@@ -531,9 +567,40 @@ class JbuilderTest < ActiveSupport::TestCase
     assert_nil result
   end
 
-  test 'throws meaningfull error when on trying to add properties to null' do
+  test 'null! in a block' do
+    result = jbuild do |json|
+      json.author do
+        json.name 'David'
+      end
+
+      json.author do
+        json.null!
+      end
+    end
+
+    assert result.key?('author')
+    assert_nil result['author']
+  end
+
+  test 'throws NullError when trying to add properties to null' do
     json = Jbuilder.new
     json.null!
-    assert_raise(Jbuilder::NullError) { json.foo 'bar' }
+    assert_raise Jbuilder::NullError do
+      json.foo 'bar'
+    end
+  end
+
+  test 'throws NullError when trying to add properties to null using block syntax' do
+    assert_raise Jbuilder::NullError do
+      jbuild do |json|
+        json.author do
+          json.null!
+        end
+
+        json.author do
+          json.name "Pavel"
+        end
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jbuilder
 version: !ruby/object:Gem::Version
-  version: 2.0.8
+  version: 2.1.0
 platform: ruby
 authors:
 - David Heinemeier Hansson
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-06 00:00:00.000000000 Z
+date: 2014-06-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport