From 1816b7474c19d4f035c7d9924329a68482ea2bba Mon Sep 17 00:00:00 2001
From: Mauricio Giacomini Girardello <mauriciogiacomini4@gmail.com>
Date: Mon, 18 Jan 2016 18:10:53 -0200
Subject: [PATCH] fixing build attachment

---
 .../learning_object/attachment_builder.rb         | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/app/builders/learning_object/attachment_builder.rb b/app/builders/learning_object/attachment_builder.rb
index 39a9fed4..e905a145 100644
--- a/app/builders/learning_object/attachment_builder.rb
+++ b/app/builders/learning_object/attachment_builder.rb
@@ -7,10 +7,8 @@ class LearningObject::AttachmentBuilder < ::Builder
   end
 
   def self.build_from_orientdb(args = {})
-    args['retrieve_link'] = args.delete('retrieveLink')
-    args['mime_type'] = args.delete('mimeType')
     build_from_cache args do
-      ::LearningObject::Attachment.new(args)
+      ::LearningObject::Attachment.new(sanitize_args(args))
     end
   end
 
@@ -27,11 +25,14 @@ class LearningObject::AttachmentBuilder < ::Builder
     object
   end
 
-  def self.permitted_args(args={})
-    permitted = [:id, :name, :link, :retrieve_link, :description, :format, :mime_type, :size]
-    args.delete_if do |key, value|
-      !permitted.include? key.to_sym
+  def self.sanitize_args(args={})
+    args['retrieve_link'] = args.delete('retrieveLink')
+    args['mime_type'] = args.delete('mimeType')
+    args['size'] = args.delete('sizeBytes')
+    ["handle", "type", "expand", "bundleName", "parentObject", "checkSum", "sequenceId", "policies"].each do |arg|
+      args.delete(arg)
     end
+    args
   end
 
 end
\ No newline at end of file
-- 
GitLab