boxr 1.22.0 → 1.23.0

data/lib/boxr/metadata.rb

@@ -1,45 +1,46 @@
+# frozen_string_literal: true
+
 module Boxr
   class Client
-
     def create_metadata(file, metadata, scope: :global, template: :properties)
       file_id = ensure_id(file)
       uri = "#{FILE_METADATA_URI}/#{file_id}/metadata/#{scope}/#{template}"
-      metadata, response = post(uri, metadata, content_type: "application/json")
+      metadata, = post(uri, metadata, content_type: 'application/json')
       metadata
     end
 
     def create_folder_metadata(folder, metadata, scope, template)
       folder_id = ensure_id(folder)
       uri = "#{FOLDER_METADATA_URI}/#{folder_id}/metadata/#{scope}/#{template}"
-      metadata, response = post(uri, metadata, content_type: "application/json")
+      metadata, = post(uri, metadata, content_type: 'application/json')
       metadata
     end
 
     def metadata(file, scope: :global, template: :properties)
       file_id = ensure_id(file)
       uri = "#{FILE_METADATA_URI}/#{file_id}/metadata/#{scope}/#{template}"
-      metadata, response = get(uri)
+      metadata, = get(uri)
       metadata
     end
 
     def folder_metadata(folder, scope, template)
       folder_id = ensure_id(folder)
       uri = "#{FOLDER_METADATA_URI}/#{folder_id}/metadata/#{scope}/#{template}"
-      metadata, response = get(uri)
+      metadata, = get(uri)
       metadata
     end
 
     def all_folder_metadata(folder)
       folder_id = ensure_id(folder)
       uri = "#{FOLDER_METADATA_URI}/#{folder_id}/metadata"
-      all_metadata, response = get(uri)
+      all_metadata, = get(uri)
       all_metadata
     end
 
     def all_metadata(file)
       file_id = ensure_id(file)
       uri = "#{FILE_METADATA_URI}/#{file_id}/metadata"
-      all_metadata, response = get(uri)
+      all_metadata, = get(uri)
       all_metadata
     end
 
@@ -47,10 +48,10 @@ module Boxr
       file_id = ensure_id(file)
       uri = "#{FILE_METADATA_URI}/#{file_id}/metadata/#{scope}/#{template}"
 
-      #in the event just one update is specified ensure that it is packaged inside an array
+      # in the event just one update is specified ensure that it is packaged inside an array
       updates = [updates] unless updates.is_a? Array
 
-      metadata, response = put(uri, updates, content_type: "application/json-patch+json")
+      metadata, = put(uri, updates, content_type: 'application/json-patch+json')
       metadata
     end
 
@@ -58,65 +59,65 @@ module Boxr
       folder_id = ensure_id(folder)
       uri = "#{FOLDER_METADATA_URI}/#{folder_id}/metadata/#{scope}/#{template}"
 
-      #in the event just one update is specified ensure that it is packaged inside an array
+      # in the event just one update is specified ensure that it is packaged inside an array
       updates = [updates] unless updates.is_a? Array
 
-      metadata, response = put(uri, updates, content_type: "application/json-patch+json")
+      metadata, = put(uri, updates, content_type: 'application/json-patch+json')
       metadata
     end
 
     def delete_metadata(file, scope: :global, template: :properties)
       file_id = ensure_id(file)
       uri = "#{FILE_METADATA_URI}/#{file_id}/metadata/#{scope}/#{template}"
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
 
     def delete_folder_metadata(folder, scope, template)
       folder_id = ensure_id(folder)
       uri = "#{FOLDER_METADATA_URI}/#{folder_id}/metadata/#{scope}/#{template}"
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
 
     def enterprise_metadata
       uri = "#{METADATA_TEMPLATES_URI}/enterprise"
-      ent_metadata, response = get(uri)
+      ent_metadata, = get(uri)
       ent_metadata
     end
-    alias :get_enterprise_templates :enterprise_metadata
+    alias get_enterprise_templates enterprise_metadata
 
     def metadata_schema(scope, template_key)
       uri = "#{METADATA_TEMPLATES_URI}/#{scope}/#{template_key}/schema"
-      schema, response = get(uri)
+      schema, = get(uri)
       schema
     end
-    alias :get_metadata_template_by_name :metadata_schema
+    alias get_metadata_template_by_name metadata_schema
 
     def get_metadata_template_by_id(template_id)
       template_id = ensure_id(template_id)
       uri = "#{METADATA_TEMPLATES_URI}/#{template_id}"
-      schema, response = get(uri)
+      schema, = get(uri)
       schema
     end
 
     def create_metadata_template(display_name, template_key: nil, fields: [], hidden: nil)
       uri = "#{METADATA_TEMPLATES_URI}/schema"
       schema = {
-        scope: "enterprise",
-        displayName: display_name,
+        scope: 'enterprise',
+        displayName: display_name
       }
       schema[:templateKey] = template_key unless template_key.nil?
       schema[:hidden] = hidden unless hidden.nil?
       schema[:fields] = fields unless fields.empty?
 
-      metadata_template, response = post(uri, schema, content_type: "application/json")
+      metadata_template, = post(uri, schema, content_type: 'application/json')
       metadata_template
     end
 
     def delete_metadata_template(scope, template_key)
       uri = "#{METADATA_TEMPLATES_URI}/#{scope}/#{template_key}/schema"
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
   end

data/lib/boxr/search.rb

@@ -1,43 +1,50 @@
+# frozen_string_literal: true
+
 module Boxr
   class Client
-
-    def search( query=nil, scope: nil, file_extensions: [], 
-                created_at_range_from_date: nil, created_at_range_to_date: nil,
-                updated_at_range_from_date: nil, updated_at_range_to_date: nil,
-                size_range_lower_bound_bytes: nil, size_range_upper_bound_bytes: nil, 
-                owner_user_ids: [], ancestor_folder_ids: [], content_types: [], trash_content: nil, 
-                mdfilters: nil, type: nil, limit: 30, offset: 0)
-
-      
-      unless mdfilters.nil?
-        unless mdfilters.is_a? String   #if a string is passed in assume it is already formatted correctly
-          unless mdfilters.is_a? Array  
-            mdfilters = [mdfilters]     #if just one mdfilter is specified ensure that it is packaged inside an array
-          end
-          mdfilters = JSON.dump(mdfilters) 
+    def search(query = nil, scope: nil, file_extensions: [], fields: [],
+               created_at_range_from_date: nil, created_at_range_to_date: nil,
+               updated_at_range_from_date: nil, updated_at_range_to_date: nil,
+               size_range_lower_bound_bytes: nil, size_range_upper_bound_bytes: nil,
+               owner_user_ids: [], ancestor_folder_ids: [], content_types: [], trash_content: nil,
+               mdfilters: nil, type: nil, limit: 30, offset: 0)
+      # if a string is passed in assume it is already formatted correctly
+      if !mdfilters.nil? && !mdfilters.is_a?(String)
+        unless mdfilters.is_a? Array
+          # if just one mdfilter is specified ensure that it is packaged inside an array
+          mdfilters = [mdfilters]
         end
+        mdfilters = JSON.dump(mdfilters)
       end
 
-      #build range strings
-      created_at_range_string = build_date_range_field(created_at_range_from_date, created_at_range_to_date)
-      updated_at_range_string = build_date_range_field(updated_at_range_from_date, updated_at_range_to_date)
-      size_range_string = build_size_range_field(size_range_lower_bound_bytes, size_range_upper_bound_bytes)
+      # build range strings
+      created_at_range_string = build_date_range_field(created_at_range_from_date,
+                                                       created_at_range_to_date)
+      updated_at_range_string = build_date_range_field(updated_at_range_from_date,
+                                                       updated_at_range_to_date)
+      size_range_string = build_size_range_field(size_range_lower_bound_bytes,
+                                                 size_range_upper_bound_bytes)
 
-      #build comma separated strings
+      # build comma separated strings
       file_extensions_string = to_comma_separated_string(file_extensions)
       owner_user_ids_string = to_comma_separated_string(owner_user_ids)
       ancestor_folder_ids_string = to_comma_separated_string(ancestor_folder_ids)
       content_types_string = to_comma_separated_string(content_types)
+      fields_string = to_comma_separated_string(fields)
 
       search_query = {}
       search_query[:query] = query unless query.nil?
       search_query[:scope] = scope unless scope.nil?
       search_query[:file_extensions] = file_extensions_string unless file_extensions_string.nil?
+      search_query[:fields] = fields_string unless fields_string.nil?
       search_query[:created_at_range] = created_at_range_string unless created_at_range_string.nil?
       search_query[:updated_at_range] = updated_at_range_string unless updated_at_range_string.nil?
       search_query[:size_range] = size_range_string unless size_range_string.nil?
       search_query[:owner_user_ids] = owner_user_ids_string unless owner_user_ids_string.nil?
-      search_query[:ancestor_folder_ids] = ancestor_folder_ids_string unless ancestor_folder_ids_string.nil?
+      unless ancestor_folder_ids_string.nil?
+        search_query[:ancestor_folder_ids] =
+          ancestor_folder_ids_string
+      end
       search_query[:content_types] = content_types_string unless content_types_string.nil?
       search_query[:trash_content] = trash_content unless trash_content.nil?
       search_query[:mdfilters] = mdfilters unless mdfilters.nil?
@@ -45,23 +52,22 @@ module Boxr
       search_query[:limit] = limit unless limit.nil?
       search_query[:offset] = offset unless offset.nil?
 
-      results, response = get(SEARCH_URI, query: search_query)
+      results, = get(SEARCH_URI, query: search_query)
       results.entries
     end
 
     private
 
     def build_date_range_field(from, to)
-      from_string = from.nil? ? "" : from.to_datetime.rfc3339
-      to_string = to.nil? ? "" : to.to_datetime.rfc3339
+      from_string = from.nil? ? '' : from.to_datetime.rfc3339
+      to_string = to.nil? ? '' : to.to_datetime.rfc3339
       build_range_string(from_string, to_string)
     end
 
     def build_size_range_field(lower, upper)
-      lower_string = lower.nil? ? "" : lower.to_i
-      upper_string = upper.nil? ? "" : upper.to_i
+      lower_string = lower.nil? ? '' : lower.to_i
+      upper_string = upper.nil? ? '' : upper.to_i
       build_range_string(lower_string, upper_string)
     end
-
   end
-end
+end

data/lib/boxr/shared_items.rb

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
+
 module Boxr
   class Client
-
     def shared_item(shared_link, shared_link_password: nil)
       box_api_header = "shared_link=#{shared_link}"
-      box_api_header += "&shared_link_password=#{shared_link_password}" unless shared_link_password.nil?
+      unless shared_link_password.nil?
+        box_api_header += "&shared_link_password=#{shared_link_password}"
+      end
 
-      file_or_folder, response = get(SHARED_ITEMS_URI, box_api_header: box_api_header)
+      file_or_folder, = get(SHARED_ITEMS_URI, box_api_header: box_api_header)
       file_or_folder
     end
-
   end
-end
+end

data/lib/boxr/tasks.rb

@@ -1,33 +1,34 @@
+# frozen_string_literal: true
+
 module Boxr
   class Client
-
     def file_tasks(file, fields: [])
       file_id = ensure_id(file)
       uri = "#{FILES_URI}/#{file_id}/tasks"
       query = build_fields_query(fields, TASK_FIELDS_QUERY)
 
-      tasks, response = get(uri, query: query)
+      tasks, = get(uri, query: query)
       tasks.entries
     end
 
     def create_task(file, action: :review, message: nil, due_at: nil)
       file_id = ensure_id(file)
-      attributes = {item: {type: :file, id: file_id}}
+      attributes = { item: { type: :file, id: file_id } }
       attributes[:action] = action unless action.nil?
       attributes[:message] = message unless message.nil?
       attributes[:due_at] = due_at.to_datetime.rfc3339 unless due_at.nil?
 
-      new_task, response = post(TASKS_URI, attributes)
+      new_task, = post(TASKS_URI, attributes)
       new_task
     end
 
     def task_from_id(task_id)
       task_id = ensure_id(task_id)
       uri = "#{TASKS_URI}/#{task_id}"
-      task, response = get(uri)
+      task, = get(uri)
       task
     end
-    alias :task :task_from_id
+    alias task task_from_id
 
     def update_task(task, action: :review, message: nil, due_at: nil)
       task_id = ensure_id(task)
@@ -37,48 +38,48 @@ module Boxr
       attributes[:message] = message unless message.nil?
       attributes[:due_at] = due_at.to_datetime.rfc3339 unless due_at.nil?
 
-      task, response = put(uri, attributes)
+      task, = put(uri, attributes)
       task
     end
 
     def delete_task(task)
       task_id = ensure_id(task)
       uri = "#{TASKS_URI}/#{task_id}"
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
 
     def task_assignments(task)
       task_id = ensure_id(task)
       uri = "#{TASKS_URI}/#{task_id}/assignments"
-      assignments, response = get(uri)
+      assignments, = get(uri)
       assignments['entries']
     end
 
     def create_task_assignment(task, assign_to: nil, assign_to_login: nil)
       task_id = ensure_id(task)
       assign_to_id = ensure_id(assign_to)
-      attributes = {task: {type: :task, id: "#{task_id}"}}
-      
-      attributes[:assign_to] = {} 
+      attributes = { task: { type: :task, id: task_id.to_s } }
+
+      attributes[:assign_to] = {}
       attributes[:assign_to][:login] = assign_to_login unless assign_to_login.nil?
       attributes[:assign_to][:id] = assign_to_id unless assign_to_id.nil?
 
-      new_task_assignment, response = post(TASK_ASSIGNMENTS_URI, attributes)
+      new_task_assignment, = post(TASK_ASSIGNMENTS_URI, attributes)
       new_task_assignment
     end
 
     def task_assignment(task)
       task_id = ensure_id(task)
       uri = "#{TASK_ASSIGNMENTS_URI}/#{task_id}"
-      task_assignment, response = get(uri)
+      task_assignment, = get(uri)
       task_assignment
     end
 
     def delete_task_assignment(task)
       task_id = ensure_id(task)
       uri = "#{TASK_ASSIGNMENTS_URI}/#{task_id}"
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
 
@@ -89,9 +90,8 @@ module Boxr
       attributes[:message] = message unless message.nil?
       attributes[:resolution_state] = resolution_state unless resolution_state.nil?
 
-      updated_task, response = put(uri, attributes)
+      updated_task, = put(uri, attributes)
       updated_task
     end
-
   end
-end
+end

data/lib/boxr/users.rb

@@ -1,24 +1,25 @@
+# frozen_string_literal: true
+
 module Boxr
   class Client
-
     def current_user(fields: [])
       uri = "#{USERS_URI}/me"
       query = build_fields_query(fields, USER_FIELDS_QUERY)
 
-      user, response = get(uri, query: query)
+      user, = get(uri, query: query)
       user
     end
-    alias :me :current_user
+    alias me current_user
 
     def user_from_id(user_id, fields: [])
       user_id = ensure_id(user_id)
       uri = "#{USERS_URI}/#{user_id}"
       query = build_fields_query(fields, USER_FIELDS_QUERY)
 
-      user, response = get(uri, query: query)
+      user, = get(uri, query: query)
       user
     end
-    alias :user :user_from_id
+    alias user user_from_id
 
     def all_users(filter_term: nil, fields: [], offset: nil, limit: nil)
       uri = USERS_URI
@@ -26,25 +27,25 @@ module Boxr
       query[:filter_term] = filter_term unless filter_term.nil?
 
       if offset.nil? || limit.nil?
-        users = get_all_with_pagination(uri, query: query, offset: 0, limit: DEFAULT_LIMIT)
+        get_all_with_pagination(uri, query: query, offset: 0, limit: DEFAULT_LIMIT)
       else
         query[:offset] = offset
         query[:limit] = limit
 
-        users, response = get(uri, query: query)
+        users, = get(uri, query: query)
         users['entries']
       end
     end
 
     def create_user(name, login: nil, role: nil, language: nil, is_sync_enabled: nil, job_title: nil,
-                                 phone: nil, address: nil, space_amount: nil, tracking_codes: nil,
-                                 can_see_managed_users: nil, is_external_collab_restricted: nil, status: nil, timezone: nil,
-                                 is_exempt_from_device_limits: nil, is_exempt_from_login_verification: nil,
-                                 is_platform_access_only: nil)
-
+                    phone: nil, address: nil, space_amount: nil, tracking_codes: nil,
+                    can_see_managed_users: nil, is_external_collab_restricted: nil, status: nil, timezone: nil,
+                    is_exempt_from_device_limits: nil, is_exempt_from_login_verification: nil,
+                    is_platform_access_only: nil)
       uri = USERS_URI
-      attributes = {name: name}
-      attributes[:login] = login unless login.nil? #login is not required for platform users, so needed to make this optional
+      attributes = { name: name }
+      # login is not required for platform users, so needed to make this optional
+      attributes[:login] = login unless login.nil?
       attributes[:role] = role unless role.nil?
       attributes[:language] = language unless language.nil?
       attributes[:is_sync_enabled] = is_sync_enabled unless is_sync_enabled.nil?
@@ -54,28 +55,41 @@ module Boxr
       attributes[:space_amount] = space_amount unless space_amount.nil?
       attributes[:tracking_codes] = tracking_codes unless tracking_codes.nil?
       attributes[:can_see_managed_users] = can_see_managed_users unless can_see_managed_users.nil?
-      attributes[:is_external_collab_restricted] = is_external_collab_restricted unless is_external_collab_restricted.nil?
+      unless is_external_collab_restricted.nil?
+        attributes[:is_external_collab_restricted] =
+          is_external_collab_restricted
+      end
       attributes[:status] = status unless status.nil?
       attributes[:timezone] = timezone unless timezone.nil?
-      attributes[:is_exempt_from_device_limits] = is_exempt_from_device_limits unless is_exempt_from_device_limits.nil?
-      attributes[:is_exempt_from_login_verification] = is_exempt_from_login_verification unless is_exempt_from_login_verification.nil?
-      attributes[:is_platform_access_only] = is_platform_access_only unless is_platform_access_only.nil?
+      unless is_exempt_from_device_limits.nil?
+        attributes[:is_exempt_from_device_limits] =
+          is_exempt_from_device_limits
+      end
+      unless is_exempt_from_login_verification.nil?
+        attributes[:is_exempt_from_login_verification] =
+          is_exempt_from_login_verification
+      end
+      unless is_platform_access_only.nil?
+        attributes[:is_platform_access_only] =
+          is_platform_access_only
+      end
 
-      new_user, response = post(uri, attributes)
+      new_user, = post(uri, attributes)
       new_user
     end
 
     def update_user(user, notify: nil, enterprise: true, name: nil, role: nil, language: nil, is_sync_enabled: nil,
-                             job_title: nil, phone: nil, address: nil, space_amount: nil, tracking_codes: nil,
-                             can_see_managed_users: nil, status: nil, timezone: nil, is_exempt_from_device_limits: nil,
-                             is_exempt_from_login_verification: nil, is_exempt_from_reset_required: nil, is_external_collab_restricted: nil)
-
+                    job_title: nil, phone: nil, address: nil, space_amount: nil, tracking_codes: nil,
+                    can_see_managed_users: nil, status: nil, timezone: nil, is_exempt_from_device_limits: nil,
+                    is_exempt_from_login_verification: nil, is_exempt_from_reset_required: nil, is_external_collab_restricted: nil)
       user_id = ensure_id(user)
       uri = "#{USERS_URI}/#{user_id}"
-      query = {notify: notify} unless notify.nil?
+      query = {}
+      query[:notify] = notify unless notify.nil?
 
       attributes = {}
-      attributes[:enterprise] = nil if enterprise.nil? #this is a special condition where setting this to nil means to roll this user out of the enterprise
+      # this is a special condition where setting this to nil means to roll this user out of the enterprise
+      attributes[:enterprise] = nil if enterprise.nil?
       attributes[:name] = name unless name.nil?
       attributes[:role] = role unless role.nil?
       attributes[:language] = language unless language.nil?
@@ -88,12 +102,24 @@ module Boxr
       attributes[:can_see_managed_users] = can_see_managed_users unless can_see_managed_users.nil?
       attributes[:status] = status unless status.nil?
       attributes[:timezone] = timezone unless timezone.nil?
-      attributes[:is_exempt_from_device_limits] = is_exempt_from_device_limits unless is_exempt_from_device_limits.nil?
-      attributes[:is_exempt_from_login_verification] = is_exempt_from_login_verification unless is_exempt_from_login_verification.nil?
-      attributes[:is_exempt_from_reset_required] = is_exempt_from_reset_required unless is_exempt_from_reset_required.nil?
-      attributes[:is_external_collab_restricted] = is_external_collab_restricted unless is_external_collab_restricted.nil?
+      unless is_exempt_from_device_limits.nil?
+        attributes[:is_exempt_from_device_limits] =
+          is_exempt_from_device_limits
+      end
+      unless is_exempt_from_login_verification.nil?
+        attributes[:is_exempt_from_login_verification] =
+          is_exempt_from_login_verification
+      end
+      unless is_exempt_from_reset_required.nil?
+        attributes[:is_exempt_from_reset_required] =
+          is_exempt_from_reset_required
+      end
+      unless is_external_collab_restricted.nil?
+        attributes[:is_external_collab_restricted] =
+          is_external_collab_restricted
+      end
 
-      updated_user, response = put(uri, attributes, query: query)
+      updated_user, = put(uri, attributes, query: query)
       updated_user
     end
 
@@ -104,7 +130,7 @@ module Boxr
       query[:notify] = notify unless notify.nil?
       query[:force] = force unless force.nil?
 
-      result, response = delete(uri, query: query)
+      result, = delete(uri, query: query)
       result
     end
 
@@ -114,9 +140,9 @@ module Boxr
       destination_user_id = ensure_id(destination_user)
       source_folder_id = ensure_id(source_folder)
       uri = "#{USERS_URI}/#{user_id}/folders/#{source_folder_id}"
-      attributes = {owned_by: {id: destination_user_id}}
+      attributes = { owned_by: { id: destination_user_id } }
 
-      folder, response = put(uri, attributes)
+      folder, = put(uri, attributes)
       folder
     end
 
@@ -124,16 +150,16 @@ module Boxr
       user_id = ensure_id(user)
       uri = "#{USERS_URI}/#{user_id}/email_aliases"
 
-      aliases, response = get(uri)
+      aliases, = get(uri)
       aliases['entries']
     end
 
     def add_email_alias_for_user(user, email)
       user_id = ensure_id(user)
       uri = "#{USERS_URI}/#{user_id}/email_aliases"
-      attributes = {email: email}
+      attributes = { email: email }
 
-      updated_user, response = post(uri, attributes)
+      updated_user, = post(uri, attributes)
       updated_user
     end
 
@@ -142,9 +168,8 @@ module Boxr
       email_alias_id = ensure_id(email_alias)
       uri = "#{USERS_URI}/#{user_id}/email_aliases/#{email_alias_id}"
 
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
-
   end
 end

data/lib/boxr/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Boxr
-  VERSION = "1.22.0".freeze
+  VERSION = '1.23.0'
 end

data/lib/boxr/watermarking.rb

@@ -1,66 +1,59 @@
+# frozen_string_literal: true
+
 module Boxr
   class Client
-
     def get_watermark_on_file(file)
-
       file_id = ensure_id(file)
       uri = "#{FILES_URI}/#{file_id}/watermark"
 
-      file, response = get(uri)
+      file, = get(uri)
       file
     end
 
     def apply_watermark_on_file(file)
-
       file_id = ensure_id(file)
       uri = "#{FILES_URI}/#{file_id}/watermark"
 
       attributes = {}
-      attributes[:watermark] = {:imprint => "default"}
+      attributes[:watermark] = { imprint: 'default' }
 
-      file, response = put(uri, attributes, content_type: "application/json")
+      file, = put(uri, attributes, content_type: 'application/json')
       file
     end
 
     def remove_watermark_on_file(file)
-
       file_id = ensure_id(file)
       uri = "#{FILES_URI}/#{file_id}/watermark"
 
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
 
     def get_watermark_on_folder(folder)
-
       folder_id = ensure_id(folder)
       uri = "#{FOLDERS_URI}/#{folder_id}/watermark"
 
-      folder, response = get(uri)
+      folder, = get(uri)
       folder
     end
 
     def apply_watermark_on_folder(folder)
-
       folder_id = ensure_id(folder)
       uri = "#{FOLDERS_URI}/#{folder_id}/watermark"
 
       attributes = {}
-      attributes[:watermark] = {:imprint => "default"}
+      attributes[:watermark] = { imprint: 'default' }
 
-      folder, response = put(uri, attributes, content_type: "application/json")
+      folder, = put(uri, attributes, content_type: 'application/json')
       folder
     end
 
     def remove_watermark_on_folder(folder)
-
       folder_id = ensure_id(folder)
       uri = "#{FOLDERS_URI}/#{folder_id}/watermark"
 
-      result, response = delete(uri)
+      result, = delete(uri)
       result
     end
-
-
   end
 end