toast 0.8.5 → 0.8.6

data/app/controller/toast_controller.rb

@@ -17,7 +17,6 @@ class ToastController < ApplicationController
       return head(:not_found)
 
     rescue Toast::PayloadInvalid => e
-      raise e if Rails.env == "test"
       return render :text => e.message, :status => :forbidden
 
     rescue Toast::PayloadFormatError => e

data/lib/toast/association.rb

@@ -88,15 +88,10 @@ module Toast
         raise PayloadFormatError
       end
 
-      # ignore:
-      # * all exposed readable, but not writable fields
-      # * all associations
-      # * self
-      ((@associate_config_in.readables - @associate_config_in.writables) +
-       @associate_config_in.exposed_associations + ["self"]).each do |rof|
-        payload.delete(rof)
-      end
-
+      payload.delete_if {|key,value|
+        !@associate_config_in.writables.include?(key) or
+        @associate_config_in.exposed_associations.include?(key)
+      }
 
       begin
         reflection = @model.reflect_on_association(@assoc.to_sym)

data/lib/toast/collection.rb

@@ -91,11 +91,10 @@ module Toast
         raise PayloadFormatError
       end
 
-      # silently ignore all exposed readable, but not writable fields
-      ((@config_in.readables - @config_in.writables) +  @config_in.exposed_associations + ["self"]).each do |rof|
-        payload.delete(rof)
-      end
-
+      payload.delete_if {|key,value|
+        !@config_in.writables.include?(key) or
+        @config_in.exposed_associations.include?(key)
+      }
 
       begin
 

data/lib/toast/record.rb

@@ -34,13 +34,11 @@ module Toast
         raise PayloadFormatError
       end
 
-      # ignore:
-      # * all exposed readable, but not writable fields
-      # * all associations
-      # * self
-      ((@config_in.readables - @config_in.writables) + @config_in.exposed_associations + ["self"]).each do |rof|
-        payload.delete(rof)
-      end
+      # ignore all not explicitly writable attribute
+      payload.delete_if {|key,value|
+        !@config_in.writables.include?(key) or
+        @config_in.exposed_associations.include?(key)
+      }
 
       # set the virtual attributes
       (@config_in.writables - @record.attribute_names -  @config_in.exposed_associations).each do |vattr|
@@ -53,7 +51,7 @@ module Toast
       end
 
       # mass-update for the rest
-      @record.update_attributes payload
+      @record.update_attributes! payload
       {
         :json => @record.represent( @config_out.exposed_attributes,
                                     @config_out.exposed_associations,
@@ -63,6 +61,10 @@ module Toast
         :location => self.base_uri + @record.uri_path,
         :content_type => @config_out.media_type
       }
+
+    rescue ActiveRecord::RecordInvalid => e
+      # model validation failed
+      raise PayloadInvalid.new(e.message)
     end
 
     def get

data/lib/toast/version.rb

@@ -1,3 +1,3 @@
 module Toast
-  VERSION = '0.8.5'
+  VERSION = '0.8.6'
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: toast
 version: !ruby/object:Gem::Version 
-  hash: 53
+  hash: 51
   prerelease: 
   segments: 
   - 0
   - 8
-  - 5
-  version: 0.8.5
+  - 6
+  version: 0.8.6
 platform: ruby
 authors: 
 - "robokopp (Robert Anni\xC3\xA9s)"
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-09-20 00:00:00 Z
+date: 2012-09-21 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rails