lib/immoscout/models/actions/attachment.rb in immoscout-1.3.1 vs lib/immoscout/models/actions/attachment.rb in immoscout-1.3.2

@@ -1,8 +1,12 @@
+# frozen_string_literal: true
+
 require 'json'
 require_relative '../concerns/modelable'
 
+# rubocop:disable Metrics/BlockLength because this is how an ActiveSupport
+#   concern looks like
 module Immoscout
   module Models
     module Actions
       module Attachment
         extend ActiveSupport::Concern
@@ -10,15 +14,18 @@
         included do
           include Immoscout::Models::Concerns::Modelable
 
           self.unpack_collection = proc do |hash|
             hash
-              .fetch("common.attachments")
+              .fetch('common.attachments')
               .first
-              .fetch("attachment")
+              .fetch('attachment')
           end
 
+          # rubocop:disable Metrics/AbcSize because this is the
+          #   bare minimum logic
+          # rubocop:disable Metrics/MethodLength dito
           def save
             attachable_id = attachable.try(:id) || attachable
             response = api.post(
               "user/#{api.user_name}/realestate/#{attachable_id}/attachment",
               nil,
@@ -31,15 +38,19 @@
             )
             handle_response(response)
             self.id = id_from_response(response)
             self
           end
+          # rubocop:enable Metrics/AbcSize
+          # rubocop:enable Metrics/MethodLength
 
           def destroy
             attachable_id = attachable.try(:id) || attachable
             response = api.delete(
-              "user/#{api.user_name}/realestate/#{attachable_id}/attachment/#{id}",
+              "user/#{api.user_name}/" \
+              "realestate/#{attachable_id}/" \
+              "attachment/#{id}"
             )
             handle_response(response)
             self
           end
 
@@ -59,28 +70,31 @@
         end
 
         class_methods do
           def content_type_from_extension(ext)
             {
-              ".jpg"  => "image/jpeg",
-              ".jpeg" => "image/jpeg",
-              ".gif"  => "image/gif",
-              ".png"  => "image/png",
-              ".pdf"  => "application/pdf"
+              '.jpg' => 'image/jpeg',
+              '.jpeg' => 'image/jpeg',
+              '.gif' => 'image/gif',
+              '.png' => 'image/png',
+              '.pdf' => 'application/pdf'
             }.fetch(ext)
           end
 
+          # rubocop:disable Metrics/AbcSize because of the mapping logic
           def all(real_estate_id)
             response = api.get(
               "user/#{api.user_name}/realestate/#{real_estate_id}/attachment"
             )
             handle_response(response)
             objects = unpack_collection.call(response.body)
             objects
               .map { |object| new(object) }
               .select { |object| object.type =~ /#{name.demodulize}/i }
           end
+          # rubocop:enable Metrics/AbcSize
         end
       end
     end
   end
 end
+# rubocop:enable Metrics/BlockLength