files.com 1.1.768 → 1.1.770

data/lib/files.com/models/expectation_incident.rb

@@ -127,14 +127,19 @@ module Files
     #   per_page - int64 - Number of records to show per page.  (Max: 10000, 1,000 or less is recommended).
     #   sort_by - object - If set, sort records by the specified field in either `asc` or `desc` direction. Valid fields are `workspace_id`, `created_at` or `expectation_id`.
     #   filter - object - If set, return records where the specified field is equal to the supplied value. Valid fields are `expectation_id` and `workspace_id`. Valid field combinations are `[ workspace_id, expectation_id ]`.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::List of Files::ExpectationIncident objects.
     def self.list(params = {}, options = {})
       raise InvalidParameterError.new("Bad parameter: cursor must be an String") if params[:cursor] and !params[:cursor].is_a?(String)
       raise InvalidParameterError.new("Bad parameter: per_page must be an Integer") if params[:per_page] and !params[:per_page].is_a?(Integer)
       raise InvalidParameterError.new("Bad parameter: sort_by must be an Hash") if params[:sort_by] and !params[:sort_by].is_a?(Hash)
       raise InvalidParameterError.new("Bad parameter: filter must be an Hash") if params[:filter] and !params[:filter].is_a?(Hash)
 
-      List.new(ExpectationIncident, params) do
-        Api.send_request("/expectation_incidents", :get, params, options)
+      List.new(ExpectationIncident, params) do |page_params, &check_page|
+        Api.send_request("/expectation_incidents", :get, page_params, options, &check_page)
       end
     end
 
@@ -144,13 +149,18 @@ module Files
 
     # Parameters:
     #   id (required) - int64 - Expectation Incident ID.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::ExpectationIncident.
     def self.find(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
       raise InvalidParameterError.new("Bad parameter: id must be an Integer") if params[:id] and !params[:id].is_a?(Integer)
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
 
-      response, options = Api.send_request("/expectation_incidents/#{params[:id]}", :get, params, options)
+      response, options = Api.send_request("/expectation_incidents/#{params[:id]}", :get, params, options) { |reply| reply.require_object("ExpectationIncident") }
       ExpectationIncident.new(response.data, options)
     end
 
@@ -159,13 +169,18 @@ module Files
     end
 
     # Resolve an expectation incident
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::ExpectationIncident.
     def self.resolve(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
       raise InvalidParameterError.new("Bad parameter: id must be an Integer") if params[:id] and !params[:id].is_a?(Integer)
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
 
-      response, options = Api.send_request("/expectation_incidents/#{params[:id]}/resolve", :post, params, options)
+      response, options = Api.send_request("/expectation_incidents/#{params[:id]}/resolve", :post, params, options) { |reply| reply.require_object("ExpectationIncident") }
       ExpectationIncident.new(response.data, options)
     end
 
@@ -173,6 +188,11 @@ module Files
     #
     # Parameters:
     #   snoozed_until (required) - string - Time until which the incident should remain snoozed.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::ExpectationIncident.
     def self.snooze(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
@@ -181,18 +201,23 @@ module Files
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
       raise MissingParameterError.new("Parameter missing: snoozed_until") unless params[:snoozed_until]
 
-      response, options = Api.send_request("/expectation_incidents/#{params[:id]}/snooze", :post, params, options)
+      response, options = Api.send_request("/expectation_incidents/#{params[:id]}/snooze", :post, params, options) { |reply| reply.require_object("ExpectationIncident") }
       ExpectationIncident.new(response.data, options)
     end
 
     # Acknowledge an expectation incident
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::ExpectationIncident.
     def self.acknowledge(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
       raise InvalidParameterError.new("Bad parameter: id must be an Integer") if params[:id] and !params[:id].is_a?(Integer)
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
 
-      response, options = Api.send_request("/expectation_incidents/#{params[:id]}/acknowledge", :post, params, options)
+      response, options = Api.send_request("/expectation_incidents/#{params[:id]}/acknowledge", :post, params, options) { |reply| reply.require_object("ExpectationIncident") }
       ExpectationIncident.new(response.data, options)
     end
   end

data/lib/files.com/models/external_event.rb

@@ -64,9 +64,9 @@ module Files
         raise NotImplementedError.new("The ExternalEvent object doesn't support updates.")
       else
         new_obj = ExternalEvent.create(@attributes, @options)
+        @attributes = new_obj.attributes
       end
 
-      @attributes = new_obj.attributes
       true
     end
 
@@ -79,6 +79,11 @@ module Files
     #   filter_gteq - object - If set, return records where the specified field is greater than or equal the supplied value. Valid fields are `created_at`.
     #   filter_lt - object - If set, return records where the specified field is less than the supplied value. Valid fields are `created_at`.
     #   filter_lteq - object - If set, return records where the specified field is less than or equal the supplied value. Valid fields are `created_at`.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::List of Files::ExternalEvent objects.
     def self.list(params = {}, options = {})
       raise InvalidParameterError.new("Bad parameter: cursor must be an String") if params[:cursor] and !params[:cursor].is_a?(String)
       raise InvalidParameterError.new("Bad parameter: per_page must be an Integer") if params[:per_page] and !params[:per_page].is_a?(Integer)
@@ -89,8 +94,8 @@ module Files
       raise InvalidParameterError.new("Bad parameter: filter_lt must be an Hash") if params[:filter_lt] and !params[:filter_lt].is_a?(Hash)
       raise InvalidParameterError.new("Bad parameter: filter_lteq must be an Hash") if params[:filter_lteq] and !params[:filter_lteq].is_a?(Hash)
 
-      List.new(ExternalEvent, params) do
-        Api.send_request("/external_events", :get, params, options)
+      List.new(ExternalEvent, params) do |page_params, &check_page|
+        Api.send_request("/external_events", :get, page_params, options, &check_page)
       end
     end
 
@@ -100,13 +105,18 @@ module Files
 
     # Parameters:
     #   id (required) - int64 - External Event ID.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::ExternalEvent.
     def self.find(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
       raise InvalidParameterError.new("Bad parameter: id must be an Integer") if params[:id] and !params[:id].is_a?(Integer)
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
 
-      response, options = Api.send_request("/external_events/#{params[:id]}", :get, params, options)
+      response, options = Api.send_request("/external_events/#{params[:id]}", :get, params, options) { |reply| reply.require_object("ExternalEvent") }
       ExternalEvent.new(response.data, options)
     end
 
@@ -117,13 +127,18 @@ module Files
     # Parameters:
     #   status (required) - string - Status of event.
     #   body (required) - string - Event body
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::ExternalEvent.
     def self.create(params = {}, options = {})
       raise InvalidParameterError.new("Bad parameter: status must be an String") if params[:status] and !params[:status].is_a?(String)
       raise InvalidParameterError.new("Bad parameter: body must be an String") if params[:body] and !params[:body].is_a?(String)
       raise MissingParameterError.new("Parameter missing: status") unless params[:status]
       raise MissingParameterError.new("Parameter missing: body") unless params[:body]
 
-      response, options = Api.send_request("/external_events", :post, params, options)
+      response, options = Api.send_request("/external_events", :post, params, options) { |reply| reply.require_object("ExternalEvent") }
       ExternalEvent.new(response.data, options)
     end
   end

data/lib/files.com/models/file.rb

@@ -218,7 +218,7 @@ module Files
 
       create(destination, params, options)
     ensure
-      local_file.close
+      local_file&.close
     end
 
     def self.write(*_args)
@@ -455,10 +455,16 @@ module Files
         r, w = SizableIO.pipe
         Thread.new do
           download_content(w, **options)
+        rescue StandardError => e
+          # The reader raises this instead of seeing a clean end of file, even when the download never started.
+          w.do_set_error(e)
         ensure
           w.close
         end
         r.wait!(5)
+      rescue StandardError
+        r&.close
+        raise
       end
     end
 
@@ -1103,7 +1109,9 @@ module Files
       raise InvalidParameterError.new("Bad parameter: priority_color must be an String") if params[:priority_color] and !params[:priority_color].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      Api.send_request("/files/#{@attributes[:path]}", :patch, params, @options)
+      response, options = Api.send_request("/files/#{@attributes[:path]}", :patch, params, @options) { |reply| reply.require_object("File") }
+      @attributes = response.data
+      [ response, options ]
     end
 
     # Parameters:
@@ -1306,6 +1314,11 @@ module Files
     #   with_previews - boolean - Include file preview information?
     #   with_priority_color - boolean - Include file priority color information?
     #   with_direct_connection_info - boolean - Include optional direct connection information for a direct Agent transfer attempt?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::File.
     def self.download(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1314,7 +1327,7 @@ module Files
       raise InvalidParameterError.new("Bad parameter: preview_size must be an String") if params[:preview_size] and !params[:preview_size].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/files/#{params[:path]}", :get, params, options)
+      response, options = Api.send_request("/files/#{params[:path]}", :get, params, options) { |reply| reply.require_object("File") }
       File.new(response.data, options)
     end
 
@@ -1337,6 +1350,11 @@ module Files
     #   with_rename - boolean - Allow file rename instead of overwrite?
     #   buffered_upload - boolean - If true, and the path refers to a destination not stored on Files.com (such as a remote server mount), the upload will be uploaded first to Files.com before being sent to the remote server mount. This can allow clients to upload using parallel parts to a remote server destination that does not offer parallel parts support natively.
     #   with_direct_connection_info - boolean - Include optional direct connection information for a direct Agent transfer attempt?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::File.
     def self.create(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1353,7 +1371,7 @@ module Files
       raise InvalidParameterError.new("Bad parameter: structure must be an String") if params[:structure] and !params[:structure].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/files/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/files/#{params[:path]}", :post, params, options) { |reply| reply.require_object("File") }
       File.new(response.data, options)
     end
 
@@ -1361,6 +1379,11 @@ module Files
     #   custom_metadata - object - Custom metadata map of keys and values. Limited to 32 keys, 256 characters per key and 1024 characters per value.
     #   provided_mtime - string - Modified time of file.
     #   priority_color - string - Priority/Bookmark color of file.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::File.
     def self.update(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1370,12 +1393,17 @@ module Files
       raise InvalidParameterError.new("Bad parameter: priority_color must be an String") if params[:priority_color] and !params[:priority_color].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/files/#{params[:path]}", :patch, params, options)
+      response, options = Api.send_request("/files/#{params[:path]}", :patch, params, options) { |reply| reply.require_object("File") }
       File.new(response.data, options)
     end
 
     # Parameters:
     #   recursive - boolean - If true, will recursively delete folders.  Otherwise, will error on non-empty folders.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns nil.
     def self.delete(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1396,6 +1424,11 @@ module Files
     #   preview_size - string - Request a preview size.  Can be `small` (default), `large`, `xlarge`, or `pdf`.
     #   with_previews - boolean - Include file preview information?
     #   with_priority_color - boolean - Include file priority color information?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::File.
     def self.find(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1403,7 +1436,7 @@ module Files
       raise InvalidParameterError.new("Bad parameter: preview_size must be an String") if params[:preview_size] and !params[:preview_size].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/file_actions/metadata/#{params[:path]}", :get, params, options)
+      response, options = Api.send_request("/file_actions/metadata/#{params[:path]}", :get, params, options) { |reply| reply.require_object("File") }
       File.new(response.data, options)
     end
 
@@ -1412,13 +1445,18 @@ module Files
     end
 
     # List the contents of a ZIP file
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns an Array of Files::ZipListEntry objects.
     def self.zip_list_contents(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
       raise InvalidParameterError.new("Bad parameter: path must be an String") if params[:path] and !params[:path].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/file_actions/zip_list/#{params[:path]}", :get, params, options)
+      response, options = Api.send_request("/file_actions/zip_list/#{params[:path]}", :get, params, options) { |reply| reply.require_list("ZipListEntry") }
       response.data.map do |entity_data|
         ZipListEntry.new(entity_data, options)
       end
@@ -1431,6 +1469,11 @@ module Files
     #   copy_behaviors - boolean - If copying a folder, also copy supported behaviors, email notification subscriptions, and per-folder branding to the destination folder tree?
     #   structure - boolean - Copy structure only?
     #   overwrite - boolean - Overwrite existing file(s) in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.copy(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1439,7 +1482,7 @@ module Files
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
       raise MissingParameterError.new("Parameter missing: destination") unless params[:destination]
 
-      response, options = Api.send_request("/file_actions/copy/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/file_actions/copy/#{params[:path]}", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1448,6 +1491,11 @@ module Files
     # Parameters:
     #   destination (required) - string - Move destination path.
     #   overwrite - boolean - Overwrite existing file(s) in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.move(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1456,7 +1504,7 @@ module Files
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
       raise MissingParameterError.new("Parameter missing: destination") unless params[:destination]
 
-      response, options = Api.send_request("/file_actions/move/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/file_actions/move/#{params[:path]}", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1470,6 +1518,11 @@ module Files
     #   width - int64 - Maximum output width for image_convert.
     #   height - int64 - Maximum output height for image_convert.
     #   overwrite - boolean - Overwrite existing file in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.transform(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1485,7 +1538,7 @@ module Files
       raise MissingParameterError.new("Parameter missing: transform_type") unless params[:transform_type]
       raise MissingParameterError.new("Parameter missing: target_format") unless params[:target_format]
 
-      response, options = Api.send_request("/file_actions/transform/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/file_actions/transform/#{params[:path]}", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1498,6 +1551,11 @@ module Files
     #   use_all_private_keys - boolean - Use every accessible private GPG key in the source workspace for decryption.
     #   ignore_mdc_error - boolean - Ignore errors from the MDC (modification detection code) check.
     #   overwrite - boolean - Overwrite existing file in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.gpg_decrypt(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1508,7 +1566,7 @@ module Files
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
       raise MissingParameterError.new("Parameter missing: destination") unless params[:destination]
 
-      response, options = Api.send_request("/file_actions/gpg_decrypt/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/file_actions/gpg_decrypt/#{params[:path]}", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1521,6 +1579,11 @@ module Files
     #   signing_key_id - int64 - Optional GPG Key ID to sign with.
     #   armor - boolean - Output ASCII-armored encrypted data.
     #   overwrite - boolean - Overwrite existing file in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.gpg_encrypt(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1532,7 +1595,7 @@ module Files
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
       raise MissingParameterError.new("Parameter missing: destination") unless params[:destination]
 
-      response, options = Api.send_request("/file_actions/gpg_encrypt/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/file_actions/gpg_encrypt/#{params[:path]}", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1542,6 +1605,11 @@ module Files
     #   destination (required) - string - Destination folder path for extracted files.
     #   filename - string - Optional single entry filename to extract.
     #   overwrite - boolean - Overwrite existing files in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.unzip(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1551,7 +1619,7 @@ module Files
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
       raise MissingParameterError.new("Parameter missing: destination") unless params[:destination]
 
-      response, options = Api.send_request("/file_actions/unzip", :post, params, options)
+      response, options = Api.send_request("/file_actions/unzip", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1559,13 +1627,18 @@ module Files
     #   paths (required) - array(string) - Paths to include in the ZIP.
     #   destination (required) - string - Destination file path for the ZIP.
     #   overwrite - boolean - Overwrite existing file in the destination?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileAction.
     def self.zip(params = {}, options = {})
       raise InvalidParameterError.new("Bad parameter: paths must be an Array") if params[:paths] and !params[:paths].is_a?(Array)
       raise InvalidParameterError.new("Bad parameter: destination must be an String") if params[:destination] and !params[:destination].is_a?(String)
       raise MissingParameterError.new("Parameter missing: paths") unless params[:paths]
       raise MissingParameterError.new("Parameter missing: destination") unless params[:destination]
 
-      response, options = Api.send_request("/file_actions/zip", :post, params, options)
+      response, options = Api.send_request("/file_actions/zip", :post, params, options) { |reply| reply.require_object("FileAction") }
       FileAction.new(response.data, options)
     end
 
@@ -1581,6 +1654,11 @@ module Files
     #   with_rename - boolean - Allow file rename instead of overwrite?
     #   buffered_upload - boolean - If true, and the path refers to a destination not stored on Files.com (such as a remote server mount), the upload will be uploaded first to Files.com before being sent to the remote server mount. This can allow clients to upload using parallel parts to a remote server destination that does not offer parallel parts support natively.
     #   with_direct_connection_info - boolean - Include optional direct connection information for a direct Agent transfer attempt?
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns an Array of Files::FileUploadPart objects.
     def self.begin_upload(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -1592,7 +1670,7 @@ module Files
       raise InvalidParameterError.new("Bad parameter: size must be an Integer") if params[:size] and !params[:size].is_a?(Integer)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/file_actions/begin_upload/#{params[:path]}", :post, params, options)
+      response, options = Api.send_request("/file_actions/begin_upload/#{params[:path]}", :post, params, options) { |reply| reply.require_list("FileUploadPart") }
       response.data.map do |entity_data|
         FileUploadPart.new(entity_data, options)
       end

data/lib/files.com/models/file_comment.rb

@@ -56,7 +56,9 @@ module Files
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
       raise MissingParameterError.new("Parameter missing: body") unless params[:body]
 
-      Api.send_request("/file_comments/#{@attributes[:id]}", :patch, params, @options)
+      response, options = Api.send_request("/file_comments/#{@attributes[:id]}", :patch, params, @options) { |reply| reply.require_object("FileComment") }
+      @attributes = response.data
+      [ response, options ]
     end
 
     def delete(params = {})
@@ -76,12 +78,12 @@ module Files
 
     def save
       if @attributes[:id]
-        new_obj = update(@attributes)
+        update(@attributes)
       else
         new_obj = FileComment.create(@attributes, @options)
+        @attributes = new_obj.attributes
       end
 
-      @attributes = new_obj.attributes
       true
     end
 
@@ -89,6 +91,11 @@ module Files
     #   cursor - string - Used for pagination.  When a list request has more records available, cursors are provided in the response headers `X-Files-Cursor-Next` and `X-Files-Cursor-Prev`.  Send one of those cursor value here to resume an existing list from the next available record.  Note: many of our SDKs have iterator methods that will automatically handle cursor-based pagination.
     #   per_page - int64 - Number of records to show per page.  (Max: 10000, 1,000 or less is recommended).
     #   path (required) - string - Path to operate on.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::List of Files::FileComment objects.
     def self.list_for(path, params = {}, options = {})
       params ||= {}
       params[:path] = path
@@ -97,26 +104,36 @@ module Files
       raise InvalidParameterError.new("Bad parameter: path must be an String") if params[:path] and !params[:path].is_a?(String)
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      List.new(FileComment, params) do
-        Api.send_request("/file_comments/files/#{params[:path]}", :get, params, options)
+      List.new(FileComment, params) do |page_params, &check_page|
+        Api.send_request("/file_comments/files/#{page_params[:path]}", :get, page_params, options, &check_page)
       end
     end
 
     # Parameters:
     #   body (required) - string - Comment body.
     #   path (required) - string - File path.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileComment.
     def self.create(params = {}, options = {})
       raise InvalidParameterError.new("Bad parameter: body must be an String") if params[:body] and !params[:body].is_a?(String)
       raise InvalidParameterError.new("Bad parameter: path must be an String") if params[:path] and !params[:path].is_a?(String)
       raise MissingParameterError.new("Parameter missing: body") unless params[:body]
       raise MissingParameterError.new("Parameter missing: path") unless params[:path]
 
-      response, options = Api.send_request("/file_comments", :post, params, options)
+      response, options = Api.send_request("/file_comments", :post, params, options) { |reply| reply.require_object("FileComment") }
       FileComment.new(response.data, options)
     end
 
     # Parameters:
     #   body (required) - string - Comment body.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileComment.
     def self.update(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
@@ -125,10 +142,14 @@ module Files
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
       raise MissingParameterError.new("Parameter missing: body") unless params[:body]
 
-      response, options = Api.send_request("/file_comments/#{params[:id]}", :patch, params, options)
+      response, options = Api.send_request("/file_comments/#{params[:id]}", :patch, params, options) { |reply| reply.require_object("FileComment") }
       FileComment.new(response.data, options)
     end
 
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns nil.
     def self.delete(id, params = {}, options = {})
       params ||= {}
       params[:id] = id

data/lib/files.com/models/file_comment_reaction.rb

@@ -65,9 +65,9 @@ module Files
         raise NotImplementedError.new("The FileCommentReaction object doesn't support updates.")
       else
         new_obj = FileCommentReaction.create(@attributes, @options)
+        @attributes = new_obj.attributes
       end
 
-      @attributes = new_obj.attributes
       true
     end
 
@@ -75,6 +75,11 @@ module Files
     #   user_id - int64 - User ID.  Provide a value of `0` to operate the current session's user.
     #   file_comment_id (required) - int64 - ID of file comment to attach reaction to.
     #   emoji (required) - string - Emoji to react with.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileCommentReaction.
     def self.create(params = {}, options = {})
       raise InvalidParameterError.new("Bad parameter: user_id must be an Integer") if params[:user_id] and !params[:user_id].is_a?(Integer)
       raise InvalidParameterError.new("Bad parameter: file_comment_id must be an Integer") if params[:file_comment_id] and !params[:file_comment_id].is_a?(Integer)
@@ -82,10 +87,14 @@ module Files
       raise MissingParameterError.new("Parameter missing: file_comment_id") unless params[:file_comment_id]
       raise MissingParameterError.new("Parameter missing: emoji") unless params[:emoji]
 
-      response, options = Api.send_request("/file_comment_reactions", :post, params, options)
+      response, options = Api.send_request("/file_comment_reactions", :post, params, options) { |reply| reply.require_object("FileCommentReaction") }
       FileCommentReaction.new(response.data, options)
     end
 
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns nil.
     def self.delete(id, params = {}, options = {})
       params ||= {}
       params[:id] = id

data/lib/files.com/models/file_migration.rb

@@ -61,13 +61,18 @@ module Files
 
     # Parameters:
     #   id (required) - int64 - File Migration ID.
+    #
+    # params:: Hash of API operation parameter values.
+    # options:: Hash of optional request configuration passed to Files::Api.send_request.
+    #
+    # Returns a Files::FileMigration.
     def self.find(id, params = {}, options = {})
       params ||= {}
       params[:id] = id
       raise InvalidParameterError.new("Bad parameter: id must be an Integer") if params[:id] and !params[:id].is_a?(Integer)
       raise MissingParameterError.new("Parameter missing: id") unless params[:id]
 
-      response, options = Api.send_request("/file_migrations/#{params[:id]}", :get, params, options)
+      response, options = Api.send_request("/file_migrations/#{params[:id]}", :get, params, options) { |reply| reply.require_object("FileMigration") }
       FileMigration.new(response.data, options)
     end