mongoid-embedded-errors 1.1.0 → 2.0.0

data/Gemfile.lock

@@ -1,16 +1,16 @@
 PATH
   remote: .
   specs:
-    mongoid-embedded-errors (1.1.0)
+    mongoid-embedded-errors (2.0.0)
       mongoid (>= 3.0.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (3.2.9)
-      activesupport (= 3.2.9)
+    activemodel (3.2.12)
+      activesupport (= 3.2.12)
       builder (~> 3.0.0)
-    activesupport (3.2.9)
+    activesupport (3.2.12)
       i18n (~> 0.6)
       multi_json (~> 1.0)
     builder (3.0.4)
@@ -22,9 +22,9 @@ GEM
       moped (~> 1.1)
       origin (~> 1.0)
       tzinfo (~> 0.3.22)
-    moped (1.2.7)
-    multi_json (1.4.0)
-    origin (1.0.9)
+    moped (1.4.2)
+    multi_json (1.6.1)
+    origin (1.0.11)
     rspec (2.11.0)
       rspec-core (~> 2.11.0)
       rspec-expectations (~> 2.11.0)

data/README.md

@@ -84,7 +84,12 @@ article = Article.new(pages: [Page.new(sections: [Section.new])])
 article.valid? # => false
 
 article.error.messages
-# => {:name=>["can't be blank"], :summary=>["can't be blank"], :pages=>[{0=>{:title=>["can't be blank"], :sections=>[{0=>{:header=>["can't be blank"]}}]}}]}
+{
+  :name => ["can't be blank"], 
+  :summary => ["can't be blank"], 
+  :"pages[0].title" => ["can't be blank"], 
+  :"pages[0].sections[0].header" => ["can't be blank"]
+}
 ```
 
 Now, isn't that much nicer? Yeah, I think so to.

data/lib/mongoid-embedded-errors.rb

@@ -15,10 +15,22 @@ module Mongoid
     def errors_with_embedded_errors
       errs = errors_without_embedded_errors
       self.embedded_relations.each do |name, metadata|
+        # name is something like pages or sections
+        # if there is an 'is invalid' message for the relation then let's work it:
         if errs[name]
+          # first delete the unless 'is invalid' error for the relation
           errs.delete(name.to_sym)
+          # next, loop through each of the relations (pages, sections, etc...)
           self.send(name).each_with_index do |rel, i|
-            errs[name] = {i => rel.errors.messages} if rel.errors.any?
+            # get each of their individual message and add them to the parent's errors:
+            if rel.errors.any?
+              rel.errors.messages.each do |k, v|
+                key = "#{name}[#{i}].#{k}".to_sym
+                errs.delete(key)
+                errs[key] = v
+                errs[key].flatten!
+              end
+            end
           end
         end
       end

data/lib/mongoid-embedded-errors/version.rb

@@ -1,5 +1,5 @@
 module Mongoid
   module EmbeddedErrors
-    VERSION = "1.1.0"
+    VERSION = "2.0.0"
   end
 end

data/spec/embedded_errors_spec.rb

@@ -17,16 +17,8 @@ describe Mongoid::EmbeddedErrors do
       article.errors.messages.should eql({
         name: ["can't be blank"],
         summary: ["can't be blank"],
-        pages: [{
-          0 => {
-            title: ["can't be blank"],
-            sections: [{
-              1 => {
-                header: ["can't be blank"]
-              }
-            }]
-          }
-        }]
+        :"pages[0].title" => ["can't be blank"],
+        :"pages[0].sections[1].header" => ["can't be blank"]
       })
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-embedded-errors
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 2.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-03 00:00:00.000000000 Z
+date: 2013-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mongoid
@@ -68,7 +68,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Easily bubble up errors from embedded documents in Mongoid.