forest_liana 5.4.4 → 6.0.0.pre.beta.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6535dcd725511fc2509e63c4c1fb1a5ac3f412f857471a0820baaf67da7a2f86
-  data.tar.gz: bbbcf484b173bb2e96baffbc99b9bd42c64f3530b30801ce7179e4c2a5c69fad
+  metadata.gz: 8a534b6abdb1a47d2690f785ef13dff74d97117fa7e1871b82659644ac9c1a53
+  data.tar.gz: 9d7c3281edc36a9aabf47f74851c3ba191d1e31c7bd29a0a4c7d9e8b6f3522f7
 SHA512:
-  metadata.gz: a9d3683defa8ed58627ce6e1c194076ae8af34ca137357ae15db8c4959811f015f9030acb207d8977485845891d92ce85ac5c55c950aded3e9d1547e432f4021
-  data.tar.gz: 15de6a5729539bd88d05e182f16f9329e1f36ad58e078e1598437febeb8c8b049a81f948f681e0037236089a49fce8559824255db4004bad9114878f6552539a
+  metadata.gz: c18f213548f7584a4dc7aabe121442d0e50270f29860c54164348bd45fca77810d5402284147b9a114d04f17b5314ee8d7f64daae1887e6f258cfd7663ba6c1f
+  data.tar.gz: 6edfe33f160b65de926013b45ef0ba4b7ae374a4446faf64ca6ffbd3a5e606845c89c9e30756fe8e26d4d420786ab09a36b38b0077d0c975a4d695b0aaa0c5ef

data/app/controllers/forest_liana/actions_controller.rb

@@ -1,112 +1,7 @@
 module ForestLiana
   class ActionsController < ForestLiana::BaseController
-
     def values
       render serializer: nil, json: {}, status: :ok
     end
-
-    def get_collection(collection_name)
-      ForestLiana.apimap.find { |collection| collection.name.to_s == collection_name }
-    end
-
-    def get_action(collection_name)
-      collection = get_collection(collection_name)
-      begin
-         collection.actions.find {|action| ActiveSupport::Inflector.parameterize(action.name) == params[:action_name]}
-      rescue => error
-        FOREST_LOGGER.error "Smart Action get action retrieval error: #{error}"
-        nil
-      end
-    end
-    
-    def get_record
-      model = ForestLiana::SchemaUtils.find_model_from_collection_name(params[:collectionName])
-      redord_getter = ForestLiana::ResourceGetter.new(model, {:id => params[:recordIds][0]})
-      redord_getter.perform
-      redord_getter.record
-    end
-
-    def get_smart_action_load_ctx(fields)
-      fields = fields.reduce({}) do |p, c|
-        ForestLiana::WidgetsHelper.set_field_widget(c)
-        p.update(c[:field] => c.merge!(value: nil))
-      end
-      {:record => get_record, :fields => fields}
-    end
-
-    def get_smart_action_change_ctx(fields)
-      fields = fields.reduce({}) do |p, c|
-        field = c.permit!.to_h.symbolize_keys
-        ForestLiana::WidgetsHelper.set_field_widget(field)
-        p.update(c[:field] => field)
-      end
-      {:record => get_record, :fields => fields}
-    end
-
-    def handle_result(result, formatted_fields, action)
-      if result.nil? || !result.is_a?(Hash)
-        return render status: 500, json: { error: 'Error in smart action load hook: hook must return an object' }
-      end
-      is_same_data_structure = ForestLiana::IsSameDataStructureHelper::Analyser.new(formatted_fields, result, 1)
-      unless is_same_data_structure.perform
-        return render status: 500, json: { error: 'Error in smart action hook: fields must be unchanged (no addition nor deletion allowed)' }
-      end
-
-      # Apply result on fields (transform the object back to an array), preserve order.
-      fields = action.fields.map do |field|
-        updated_field = result[field[:field]]
-
-        # Reset `value` when not present in `enums` (which means `enums` has changed).
-        if updated_field[:enums].is_a?(Array)
-          # `value` can be an array if the type of fields is `[x]`
-          if updated_field[:type].is_a?(Array) && updated_field[:value].is_a?(Array) && !(updated_field[:value] - updated_field[:enums]).empty?
-            updated_field[:value] = nil
-          end
-
-          # `value` can be any other value
-          if !updated_field[:type].is_a?(Array) && !updated_field[:enums].include?(updated_field[:value])
-            updated_field[:value] = nil
-          end
-        end
-
-        updated_field
-      end
-
-      render serializer: nil, json: { fields: fields}, status: :ok
-    end
-
-    def load
-      action = get_action(params[:collectionName])
-
-      if !action
-        render status: 500, json: {error: 'Error in smart action load hook: cannot retrieve action from collection'}
-      else
-        # Transform fields from array to an object to ease usage in hook, adds null value.
-        context = get_smart_action_load_ctx(action.fields)
-        formatted_fields = context[:fields].clone # clone for following test on is_same_data_structure
-
-        # Call the user-defined load hook.
-        result = action.hooks[:load].(context)
-
-        handle_result(result, formatted_fields, action)
-      end
-    end
-
-    def change
-      action = get_action(params[:collectionName])
-
-      if !action
-        render status: 500, json: {error: 'Error in smart action change hook: cannot retrieve action from collection'}
-      else
-        # Transform fields from array to an object to ease usage in hook.
-        context = get_smart_action_change_ctx(params[:fields])
-        formatted_fields = context[:fields].clone # clone for following test on is_same_data_structure
-
-        # Call the user-defined change hook.
-        result = action.hooks[:change][params[:changedField]].(context)
-
-        handle_result(result, formatted_fields, action)
-      end
-    end
   end
 end

data/app/controllers/forest_liana/resources_controller.rb

@@ -16,15 +16,18 @@ module ForestLiana
     def index
       begin
         if request.format == 'csv'
-          checker = ForestLiana::PermissionsChecker.new(@resource, 'exportEnabled', @rendering_id, user_id: forest_user['id'])
+          checker = ForestLiana::PermissionsChecker.new(@resource, 'export', @rendering_id)
+          return head :forbidden unless checker.is_authorized?
+        elsif params.has_key?(:searchToEdit)
+          checker = ForestLiana::PermissionsChecker.new(@resource, 'searchToEdit', @rendering_id)
           return head :forbidden unless checker.is_authorized?
         else
           checker = ForestLiana::PermissionsChecker.new(
             @resource,
-            'browseEnabled',
+            'list',
             @rendering_id,
-            user_id: forest_user['id'],
-            collection_list_parameters: get_collection_list_permission_info(forest_user, request)
+            nil,
+            get_collection_list_permission_info(forest_user, request)
           )
           return head :forbidden unless checker.is_authorized?
         end
@@ -56,10 +59,10 @@ module ForestLiana
       begin
         checker = ForestLiana::PermissionsChecker.new(
           @resource,
-          'browseEnabled',
+          'list',
           @rendering_id,
-          user_id: forest_user['id'],
-          collection_list_parameters: get_collection_list_permission_info(forest_user, request)
+          nil,
+          get_collection_list_permission_info(forest_user, request)
         )
         return head :forbidden unless checker.is_authorized?
 
@@ -86,7 +89,7 @@ module ForestLiana
 
     def show
       begin
-        checker = ForestLiana::PermissionsChecker.new(@resource, 'readEnabled', @rendering_id, user_id: forest_user['id'])
+        checker = ForestLiana::PermissionsChecker.new(@resource, 'show', @rendering_id)
         return head :forbidden unless checker.is_authorized?
 
         getter = ForestLiana::ResourceGetter.new(@resource, params)
@@ -101,7 +104,7 @@ module ForestLiana
 
     def create
       begin
-        checker = ForestLiana::PermissionsChecker.new(@resource, 'addEnabled', @rendering_id, user_id: forest_user['id'])
+        checker = ForestLiana::PermissionsChecker.new(@resource, 'create', @rendering_id)
         return head :forbidden unless checker.is_authorized?
 
         creator = ForestLiana::ResourceCreator.new(@resource, params)
@@ -124,7 +127,7 @@ module ForestLiana
 
     def update
       begin
-        checker = ForestLiana::PermissionsChecker.new(@resource, 'editEnabled', @rendering_id, user_id: forest_user['id'])
+        checker = ForestLiana::PermissionsChecker.new(@resource, 'update', @rendering_id)
         return head :forbidden unless checker.is_authorized?
 
         updater = ForestLiana::ResourceUpdater.new(@resource, params)
@@ -146,7 +149,7 @@ module ForestLiana
     end
 
     def destroy
-      checker = ForestLiana::PermissionsChecker.new(@resource, 'deleteEnabled', @rendering_id, user_id: forest_user['id'])
+      checker = ForestLiana::PermissionsChecker.new(@resource, 'delete', @rendering_id)
       return head :forbidden unless checker.is_authorized?
 
       @resource.destroy(params[:id]) if @resource.exists?(params[:id])
@@ -158,7 +161,7 @@ module ForestLiana
     end
 
     def destroy_bulk
-      checker = ForestLiana::PermissionsChecker.new(@resource, 'deleteEnabled', @rendering_id, user_id: forest_user['id'])
+      checker = ForestLiana::PermissionsChecker.new(@resource, 'delete', @rendering_id)
       return head :forbidden unless checker.is_authorized?
 
       ids = ForestLiana::ResourcesGetter.get_ids_from_request(params)
@@ -242,8 +245,8 @@ module ForestLiana
       @collection ||= ForestLiana.apimap.find { |collection| collection.name.to_s == collection_name }
     end
 
-    # NOTICE: Return a formatted object containing the request condition filters and
-    #         the user id used by the scope validator class to validate if scope is
+    # NOTICE: Return a formatted object containing the request condition filters and 
+    #         the user id used by the scope validator class to validate if scope is 
     #         in request
     def get_collection_list_permission_info(user, collection_list_request)
       {

data/app/controllers/forest_liana/smart_actions_controller.rb

@@ -19,15 +19,14 @@ module ForestLiana
 
     def check_permission_for_smart_route
       begin
-
+        
         smart_action_request = get_smart_action_request
         if !smart_action_request.nil? && smart_action_request.has_key?(:smart_action_id)
           checker = ForestLiana::PermissionsChecker.new(
             find_resource(smart_action_request[:collection_name]),
             'actions',
             @rendering_id,
-            user_id: forest_user['id'],
-            smart_action_request_info: get_smart_action_request_info
+            get_smart_action_permission_info(forest_user, smart_action_request)
           )
           return head :forbidden unless checker.is_authorized?
         else
@@ -55,14 +54,10 @@ module ForestLiana
       end
     end
 
-    # smart action permissions are retrieved from the action's endpoint and http_method
-    def get_smart_action_request_info
-      endpoint = request.fullpath
-      # Trim starting '/'
-      endpoint[0] = '' if endpoint[0] == '/'
+    def get_smart_action_permission_info(user, smart_action_request)
       {
-        endpoint: endpoint,
-        http_method: request.request_method
+        user_id: user['id'],
+        action_id: smart_action_request[:smart_action_id],
       }
     end
   end

data/app/models/forest_liana/model/action.rb

@@ -5,7 +5,7 @@ class ForestLiana::Model::Action
   extend ActiveModel::Naming
 
   attr_accessor :id, :name, :base_url, :endpoint, :http_method, :fields, :redirect,
-    :type, :download, :hooks
+    :type, :download
 
   def initialize(attributes = {})
     if attributes.key?(:global)
@@ -66,7 +66,6 @@ class ForestLiana::Model::Action
     @base_url ||= nil
     @type ||= "bulk"
     @download ||= false
-    @hooks = !@hooks.nil? ? @hooks.symbolize_keys : nil
   end
 
   def persisted?

data/app/serializers/forest_liana/stripe_invoice_serializer.rb

@@ -3,20 +3,20 @@ module ForestLiana
     include JSONAPI::Serializer
 
     attribute :amount_due
-    attribute :amount_paid
-    attribute :amount_remaining
-    attribute :application_fee_amount
     attribute :attempt_count
     attribute :attempted
+    attribute :closed
     attribute :currency
-    attribute :due_date
+    attribute :date
+    attribute :forgiven
     attribute :paid
     attribute :period_end
     attribute :period_start
-    attribute :status
     attribute :subtotal
     attribute :total
+    attribute :application_fee
     attribute :tax
+    attribute :tax_percent
 
     has_one :customer
 

data/app/services/forest_liana/apimap_sorter.rb

@@ -39,7 +39,6 @@ module ForestLiana
       'redirect',
       'download',
       'fields',
-      'hooks',
     ]
     KEYS_ACTION_FIELD = [
       'field',

data/app/services/forest_liana/permissions_checker.rb

@@ -1,162 +1,100 @@
 module ForestLiana
   class PermissionsChecker
-    @@permissions_cached = Hash.new
-    @@scopes_cached = Hash.new
-    @@roles_acl_activated = false
-    # TODO: handle cache scopes per rendering
+    @@permissions_per_rendering = Hash.new
     @@expiration_in_seconds = (ENV['FOREST_PERMISSIONS_EXPIRATION_IN_SECONDS'] || 3600).to_i
 
-    def initialize(resource, permission_name, rendering_id, user_id:, smart_action_request_info: nil, collection_list_parameters: nil)
-      @user_id = user_id
+    def initialize(resource, permission_name, rendering_id, smart_action_parameters = nil, collection_list_parameters = nil)
       @collection_name = ForestLiana.name_for(resource)
       @permission_name = permission_name
       @rendering_id = rendering_id
-      @smart_action_request_info = smart_action_request_info
+      @smart_action_parameters = smart_action_parameters
       @collection_list_parameters = collection_list_parameters
     end
 
     def is_authorized?
-      # User is still authorized if he already was and the permission has not expire
-      # if !have_permissions_expired && is_allowed
-      return true unless have_permissions_expired? || !is_allowed
-
-      fetch_permissions
-      is_allowed
+      (is_permission_expired? || !is_allowed?) ? retrieve_permissions_and_check_allowed : true
     end
 
     private
 
-    def fetch_permissions
-      permissions = ForestLiana::PermissionsGetter::get_permissions_for_rendering(@rendering_id)
-      @@roles_acl_activated = permissions['meta']['rolesACLActivated']
-      permissions['last_fetch'] = Time.now
-      if @@roles_acl_activated
-        @@permissions_cached = permissions
-      else
-        permissions['data'] = ForestLiana::PermissionsFormatter.convert_to_new_format(permissions['data'], @rendering_id)
-        @@permissions_cached[@rendering_id] = permissions
-      end
-      add_scopes_to_cache(permissions)
+    def get_permissions
+      @@permissions_per_rendering &&
+        @@permissions_per_rendering[@rendering_id] &&
+        @@permissions_per_rendering[@rendering_id]['data']
+    end
+
+    def get_last_retrieve
+      @@permissions_per_rendering &&
+        @@permissions_per_rendering[@rendering_id] &&
+        @@permissions_per_rendering[@rendering_id]['last_retrieve']
     end
 
-    def add_scopes_to_cache(permissions)
-      permissions['data']['renderings'].keys.each { |rendering_id|
-        @@scopes_cached[rendering_id] = permissions['data']['renderings'][rendering_id]
-        @@scopes_cached[rendering_id]['last_fetch'] = Time.now
-      } if permissions['data']['renderings']
+    def smart_action_allowed?(smart_actions_permissions) 
+      if !@smart_action_parameters||
+          !@smart_action_parameters[:user_id] ||
+          !@smart_action_parameters[:action_id] ||
+          !smart_actions_permissions ||
+          !smart_actions_permissions[@smart_action_parameters[:action_id]]
+        return false
+      end
+
+      @user_id = @smart_action_parameters[:user_id]
+      @action_id = @smart_action_parameters[:action_id]
+      @smart_action_permissions = smart_actions_permissions[@action_id]
+      @allowed = @smart_action_permissions['allowed']
+      @users = @smart_action_permissions['users']
+
+      return @allowed && (@users.nil?|| @users.include?(@user_id.to_i));
     end
 
-    def is_allowed
-      permissions = get_permissions_content
+    def collection_list_allowed?(scope_permissions)
+      return ForestLiana::ScopeValidator.new(
+        scope_permissions['filter'],
+        scope_permissions['dynamicScopesValues']['users']
+      ).is_scope_in_request?(@collection_list_parameters)
+    end
 
+    def is_allowed?
+      permissions = get_permissions
       if permissions && permissions[@collection_name] &&
         permissions[@collection_name]['collection']
         if @permission_name === 'actions'
           return smart_action_allowed?(permissions[@collection_name]['actions'])
+        # NOTICE: Permissions[@collection_name]['scope'] will either contains conditions filter and
+        #         dynamic user values definition, or null for collection that does not use scopes
+        elsif @permission_name === 'list' and permissions[@collection_name]['scope']
+          return collection_list_allowed?(permissions[@collection_name]['scope'])
         else
-          if @permission_name === 'browseEnabled'
-            refresh_scope_cache if scope_cache_expired?
-            scope_permissions = get_scope_in_permissions
-            if scope_permissions
-              # NOTICE: current_scope will either contains conditions filter and
-              #         dynamic user values definition, or null for collection that does not use scopes
-              return false unless are_scopes_valid?(scope_permissions)
-            end
-          end
-          return is_user_allowed(permissions[@collection_name]['collection'][@permission_name])
+          return permissions[@collection_name]['collection'][@permission_name]
         end
       else
         false
       end
     end
 
-    def get_scope_in_permissions
-      @@scopes_cached[@rendering_id] &&
-      @@scopes_cached[@rendering_id][@collection_name] &&
-      @@scopes_cached[@rendering_id][@collection_name]['scope']
-    end
-
-    def scope_cache_expired?
-      return true unless @@scopes_cached[@rendering_id] && @@scopes_cached[@rendering_id]['last_fetch']
-
-      elapsed_seconds = date_difference_in_seconds(Time.now, @@scopes_cached[@rendering_id]['last_fetch'])
-      elapsed_seconds >= @@expiration_in_seconds
-    end
-
-    # This will happen only on rolesACLActivated (as scope cache will always be up to date on disabled)
-    def refresh_scope_cache
-      permissions = ForestLiana::PermissionsGetter::get_permissions_for_rendering(@rendering_id, rendering_specific_only: true)
-      add_scopes_to_cache(permissions)
-    end
-
-    # When acl disabled permissions are stored and retrieved by rendering
-    def get_permissions
-      @@roles_acl_activated ? @@permissions_cached : @@permissions_cached[@rendering_id]
-    end
-
-    def get_permissions_content
-      permissions = get_permissions
-      permissions && permissions['data'] && permissions['data']['collections']
-    end
-
-    def get_last_fetch
-      permissions = get_permissions
-      permissions && permissions['last_fetch']
-    end
-
-    def get_smart_action_permissions(smart_actions_permissions)
-      endpoint = @smart_action_request_info[:endpoint]
-      http_method = @smart_action_request_info[:http_method]
-
-      return nil unless endpoint && http_method
-
-      schema_smart_action = ForestLiana::Utils::BetaSchemaUtils.find_action_from_endpoint(@collection_name, endpoint, http_method)
-
-      schema_smart_action &&
-        schema_smart_action.name &&
-        smart_actions_permissions &&
-        smart_actions_permissions[schema_smart_action.name]
-    end
-
-    def is_user_allowed(permission_value)
-      return false if permission_value.nil?
-      return permission_value if permission_value.in? [true, false]
-      permission_value.include?(@user_id.to_i)
-    end
-
-    def smart_action_allowed?(smart_actions_permissions)
-      smart_action_permissions = get_smart_action_permissions(smart_actions_permissions)
-
-      return false unless smart_action_permissions
-
-      is_user_allowed(smart_action_permissions['triggerEnabled'])
-    end
-
-    def are_scopes_valid?(scope_permissions)
-      return ForestLiana::ScopeValidator.new(
-        scope_permissions['filter'],
-        scope_permissions['dynamicScopesValues']['users']
-      ).is_scope_in_request?(@collection_list_parameters)
+    def retrieve_permissions
+      @@permissions_per_rendering[@rendering_id] = Hash.new
+      permissions = ForestLiana::PermissionsGetter.new(@rendering_id).perform()
+      @@permissions_per_rendering[@rendering_id]['data'] = permissions
+      @@permissions_per_rendering[@rendering_id]['last_retrieve'] = Time.now
     end
 
     def date_difference_in_seconds(date1, date2)
       (date1 - date2).to_i
     end
 
-    def have_permissions_expired?
-      last_fetch = get_last_fetch
-      return true unless last_fetch
+    def is_permission_expired?
+      last_retrieve = get_last_retrieve
+
+      return true if last_retrieve.nil?
 
-      elapsed_seconds = date_difference_in_seconds(Time.now, last_fetch)
+      elapsed_seconds = date_difference_in_seconds(Time.now, last_retrieve)
       elapsed_seconds >= @@expiration_in_seconds
     end
 
-    # Used only for testing purpose
-    def self.empty_cache
-      @@permissions_cached = Hash.new
-      @@scopes_cached = Hash.new
-      @@roles_acl_activated = false
-      @@expiration_in_seconds = (ENV['FOREST_PERMISSIONS_EXPIRATION_IN_SECONDS'] || 3600).to_i
+    def retrieve_permissions_and_check_allowed
+      retrieve_permissions
+      is_allowed?
     end
   end
 end

data/app/services/forest_liana/permissions_getter.rb

@@ -1,60 +1,25 @@
 module ForestLiana
   class PermissionsGetter
-    class << PermissionsGetter
-      def get_permissions_api_route
-        '/liana/v3/permissions'
-      end
-
-      # Permission format example:
-      # collections => {
-      #   {model_name} => {
-      #     collection => {
-      #       browseEnabled => true,
-      #       readEnabled => true,
-      #       editEnabled => true,
-      #       addEnabled => true,
-      #       deleteEnabled => true,
-      #       exportEnabled => true,
-      #     },
-      #     actions => {
-      #       {action_name} => {
-      #         triggerEnabled => true,
-      #       },
-      #     },
-      #   },
-      # },
-      # rederings => {
-      #   {rendering_id} => {
-      #       {collection_id} => {
-      #         scope => {
-      #           dynamicScopesValues => {},
-      #           filter => {}
-      #         }
-      #       }
-      #     }
-      #   }
-      # }
-      # With `rendering_specific_only` this returns only the permissions related data specific to the provided rendering
-      # For now this only includes scopes
-      def get_permissions_for_rendering(rendering_id, rendering_specific_only: false)
-        begin
-          query_parameters = { 'renderingId' => rendering_id }
-          query_parameters['renderingSpecificOnly'] = rendering_specific_only if rendering_specific_only
+    def initialize(rendering_id)
+      @route = "/liana/v2/permissions"
+      @rendering_id = rendering_id
+    end
 
-          api_route = get_permissions_api_route
-          response = ForestLiana::ForestApiRequester.get(api_route, query: query_parameters)
+    def perform
+      begin
+        query_parameters = { 'renderingId' => @rendering_id }
+        response = ForestLiana::ForestApiRequester.get(@route, query: query_parameters)
 
-          if response.is_a?(Net::HTTPOK)
-            JSON.parse(response.body)
-          else
-            raise "Forest API returned an #{ForestLiana::Errors::HTTPErrorHelper.format(response)}"
-          end
-        rescue => exception
-          FOREST_LOGGER.error 'Cannot retrieve the permissions from the Forest server.'
-          FOREST_LOGGER.error 'Which was caused by:'
-          ForestLiana::Errors::ExceptionHelper.recursively_print(exception, margin: ' ', is_error: true)
-          nil
+        if response.is_a?(Net::HTTPOK)
+          JSON.parse(response.body)
+        else
+          raise "Forest API returned an #{ForestLiana::Errors::HTTPErrorHelper.format(response)}"
         end
+      rescue => exception
+        FOREST_LOGGER.error 'Cannot retrieve the permissions from the Forest server.'
+        FOREST_LOGGER.error 'Which was caused by:'
+        ForestLiana::Errors::ExceptionHelper.recursively_print(exception, margin: ' ', is_error: true)
+        nil
       end
     end
   end

data/app/services/forest_liana/resource_creator.rb

@@ -53,7 +53,7 @@ module ForestLiana
     end
 
     def has_strong_parameter
-      Rails::VERSION::MAJOR > 5 || @resource.instance_method(:update!).arity == 1
+      Rails::VERSION::MAJOR > 5 || @resource.instance_method(:update_attributes!).arity == 1
     end
   end
 end

data/app/services/forest_liana/resource_updater.rb

@@ -14,9 +14,9 @@ module ForestLiana
         @record = @resource.find(@params[:id])
 
         if has_strong_parameter
-          @record.update(resource_params)
+          @record.update_attributes(resource_params)
         else
-          @record.update(resource_params, without_protection: true)
+          @record.update_attributes(resource_params, without_protection: true)
         end
       rescue ActiveRecord::StatementInvalid => exception
         # NOTICE: SQL request cannot be executed properly
@@ -33,7 +33,7 @@ module ForestLiana
     end
 
     def has_strong_parameter
-      Rails::VERSION::MAJOR > 5 || @resource.instance_method(:update!).arity == 1
+      Rails::VERSION::MAJOR > 5 || @resource.instance_method(:update_attributes!).arity == 1
     end
   end
 end

data/app/services/forest_liana/resources_getter.rb

@@ -12,11 +12,11 @@ module ForestLiana
       @collection = get_collection(@collection_name)
       @fields_to_serialize = get_fields_to_serialize
       @field_names_requested = field_names_requested
-      get_segment
-      compute_includes
+      get_segment()
+      compute_includes()
       @search_query_builder = SearchQueryBuilder.new(@params, @includes, @collection)
 
-      prepare_query
+      prepare_query()
     end
 
     def self.get_ids_from_request(params)

data/app/services/forest_liana/schema_utils.rb

@@ -2,14 +2,9 @@ module ForestLiana
   class SchemaUtils
 
     def self.associations(active_record_class)
-      active_record_class.reflect_on_all_associations.select do |association|
-        begin
-          !polymorphic?(association) && !is_active_type?(association.klass)
-        rescue
-          FOREST_LOGGER.warn "Unknown association #{association.name} on class #{active_record_class.name}"
-          false
-        end
-      end
+      active_record_class
+        .reflect_on_all_associations
+        .select { |association| !polymorphic?(association) && !is_active_type?(association.klass) }
     end
 
     def self.one_associations(active_record_class)