Add configurable error_limit option

-[x] Update README.md
-[x] Whitelist new :error_limit option
-[x] Set default :error_limit to 5
-[x] sends option[:error_limit] instead of hardcoded 5 from dsl
-[x] import result checks for option[:error_limit]

Author: Pair

README.md

@@ -1,7 +1,7 @@
-# ActiveAdminImport 
+# ActiveAdminImport
 [The most fastest and efficient CSV import for Active Admin
 with support of validations, bulk inserts and encodings handling](http://activeadmin-plugins.github.io/active_admin_import/)
- 
+
 
 
 [![Build Status](https://img.shields.io/travis/activeadmin-plugins/active_admin_import.svg)](https://travis-ci.org/activeadmin-plugins/active_admin_import)
@@ -46,7 +46,7 @@ And then execute:
   <li> and more...</li>
 </ol>
 
-   
+
 
 #### Basic usage
 
@@ -60,24 +60,25 @@ end
 #### Options
 Tool                    | Description
 ---------------------   | -----------
-:back					|resource action to redirect after processing
-:csv_options			|hash with column separator, row separator, etc 
-:validate				|bool means perform validations or not
-:batch_size				|integer value of max  record count inserted by 1 query/transaction
+:back                   |resource action to redirect after processing
+:csv_options            |hash with column separator, row separator, etc
+:validate               |bool means perform validations or not
+:batch_size             |integer value of max  record count inserted by 1 query/transaction
 :batch_transaction    |bool (false by default), if transaction is used when batch importing and works when :validate is set to true
-:before_import			|proc for before import action, hook called with  importer object
-:after_import			|proc for after import action, hook called with  importer object
-:before_batch_import	|proc for before each batch action, called with  importer object
-:after_batch_import		|proc for after each batch action, called with  importer object
+:before_import          |proc for before import action, hook called with  importer object
+:after_import           |proc for after import action, hook called with  importer object
+:before_batch_import    |proc for before each batch action, called with  importer object
+:after_batch_import     |proc for after each batch action, called with  importer object
 :on_duplicate_key_update|an Array or Hash, tells activerecord-import to use MySQL's ON DUPLICATE KEY UPDATE or Postgres 9.5+ ON CONFLICT DO UPDATE ability.
-:timestamps				|bool, tells activerecord-import to not add timestamps (if false) even if record timestamps is disabled in ActiveRecord::Base
-:ignore					|bool, tells activerecord-import to use MySQL's INSERT IGNORE ability
-:template				|custom template rendering
-:template_object		|object passing to view
-:resource_class			|resource class name
-:resource_label			|resource label value
-:plural_resource_label	|pluralized resource label value (default config.plural_resource_label)
-:headers_rewrites		|hash with key (csv header) - value (db column name) rows mapping
+:timestamps             |bool, tells activerecord-import to not add timestamps (if false) even if record timestamps is disabled in ActiveRecord::Base
+:ignore                 |bool, tells activerecord-import to use MySQL's INSERT IGNORE ability
+:template               |custom template rendering
+:template_object        |object passing to view
+:resource_class         |resource class name
+:resource_label         |resource label value
+:plural_resource_label  |pluralized resource label value (default config.plural_resource_label)
+:error_limit            |Limit the number of errors reported (default `5`, set to `nil` for all)
+:headers_rewrites       |hash with key (csv header) - value (db column name) rows mapping
 
 
 
@@ -103,11 +104,3 @@ Tool                  | Description
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
-
-
-
-
-
-
-
-

lib/active_admin_import/dsl.rb

@@ -37,7 +37,7 @@ module DSL
             count: result.failed.count,
             model: model_name,
             plural_model: plural_model_name,
-            message: result.failed_message(limit: 5))
+            message: result.failed_message(limit: options[:error_limit]))
           return if options[:batch_transaction]
         end
         if result.imported?

lib/active_admin_import/import_result.rb

@@ -30,7 +30,7 @@ def empty?
     end
 
     def failed_message(options = {})
-      limit = options.fetch(:limit, failed.count)
+      limit = options[:limit] || failed.count
       failed.first(limit).map do |record|
         errors = record.errors
         (errors.full_messages.zip errors.keys.map { |k| record.send k }).map { |ms| ms.join(' - ') }.join(', ')

lib/active_admin_import/options.rb

@@ -19,6 +19,7 @@ module Options
       :resource_class,
       :resource_label,
       :plural_resource_label,
+      :error_limit,
       :headers_rewrites
     ].freeze
 
@@ -34,6 +35,7 @@ def self.options_for(config, options = {})
         resource_class: config.resource_class,
         resource_label: config.resource_label,
         plural_resource_label: config.plural_resource_label,
+        error_limit: 5,
         headers_rewrites: {}
       }.deep_merge(options)
     end