active_report 2.2.0 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0d88da8e3d323ba7fabbcfc4c1507f37686a3fe2
-  data.tar.gz: 497454172ccddee6ea4716ded9253927f5f1d31e
+  metadata.gz: ebf90bb311957add7e00dc356a4155ab6d0eff5a
+  data.tar.gz: 4c85f87f49123255f67dae03405dd221cb40d8a4
 SHA512:
-  metadata.gz: 4116f0a34255612cd0b811757835225bedbc6ef07ed041ea6de53c97ab21ae1a933334f285d66dd0552858a11538ce4468c4e539e7a80a9095999acdd8e11dd1
-  data.tar.gz: 271e32852cc2544244eba761b5b38817c71c14996546c137a9511c89ec094ab704e792ebd528d9d269457c47dda189156b3ba13181b97b529ac11c6116ce6f83
+  metadata.gz: fd84dc9f25ac4ec72ff2df3f40eaab3fbc9fc5cea8e75efd5d41bd19f56e873c3e28d0427286b56db00c8c7384c918d4cad70649ce491147f719dc2e80fba2af
+  data.tar.gz: e89cbb07d4bab09d75ed055c5f4e16ee9acd09a795d565bd5d10939e70206be140c4adea39230139458a2f333faff39f54ea354d22f3b71d7cf1ebac1531c219

data/lib/active_report.rb

@@ -1,6 +1,6 @@
-require "csv"
-require "active_object"
-require "active_report/version"
-require "active_report/array"
-require "active_report/hash"
-require "active_report/record"
+require 'csv'
+require 'active_object'
+require 'active_report/version'
+require 'active_report/array'
+require 'active_report/hash'
+require 'active_report/record'

data/lib/active_report/array.rb

@@ -21,16 +21,14 @@ class ActiveReport::Array
 
     CSV.generate(@options) do |csv|
       csv << @headers unless @headers.nil?
-      @datum.each { |data| csv << data }
+      @datum.lazy.each { |d| csv << d }
     end
   end
 
   def import
     processed_datum = [].push(@headers).compact
-    CSV.foreach(@datum, @options) do |data|
-      processed_datum.push(data)
-    end
-    return(processed_datum.size < 2 ? processed_datum.flatten : processed_datum)
+    CSV.foreach(@datum, @options) { |d| processed_datum.push(d) }
+    processed_datum.size < 2 ? processed_datum.flatten : processed_datum
   end
 
 end

data/lib/active_report/hash.rb

@@ -26,9 +26,9 @@ class ActiveReport::Hash
     CSV.generate(@options) do |csv|
       header = @datum.first.only(@only)     unless @only.empty?
       header = @datum.first.except(@except) unless @except.empty?
-      csv << (@headers || (header || @datum.first).keys.map { |k| k.to_s.gsub("_", " ").capitalize })
+      csv << (@headers || (header || @datum.first).keys.map { |k| k.to_s.gsub('_'.freeze, ' '.freeze).capitalize })
 
-      @datum.each do |data|
+      @datum.lazy.each do |data|
         cell = data.only(@only)     unless @only.empty?
         cell = data.except(@except) unless @except.empty?
         csv << (cell || data).values
@@ -41,14 +41,12 @@ class ActiveReport::Hash
     @except = [].push(@except).compact unless @except.is_a?(Array)
 
     processed_datum = []
-    CSV.foreach(@datum, @options).each_with_index do |data, line|
+    CSV.foreach(@datum, @options).lazy.each_with_index do |data, line|
       if @headers.nil? && line.zero?
         @headers = data
       else
         processed_data = {}
-        @headers.each_with_index do |v, i|
-          processed_data.store(v.to_s, data.fetch(i, nil) )
-        end
+        @headers.lazy.each_with_index { |v, i| processed_data.store(v.to_s, data.fetch(i, nil)) }
 
         processed_data.only!(@only)     unless @only.empty?
         processed_data.except!(@except) unless @except.empty?

data/lib/active_report/record.rb

@@ -1,4 +1,4 @@
-require "json"
+require 'json'
 class ActiveReport::Record
 
   attr_accessor :datum, :model, :only, :except, :headers, :options
@@ -28,9 +28,9 @@ class ActiveReport::Record
     CSV.generate(@options) do |csv|
       header = @datum.first.only(@only)     unless @only.empty?
       header = @datum.first.except(@except) unless @except.empty?
-      csv << (@headers || (header || @datum.first).keys.map { |k| k.to_s.gsub("_", " ").capitalize })
+      csv << (@headers || (header || @datum.first).keys.map { |k| k.to_s.gsub('_'.freeze, ' '.freeze).capitalize })
 
-      @datum.each do |data|
+      @datum.lazy.each do |data|
         cell = data.only(@only)     unless @only.empty?
         cell = data.except(@except) unless @except.empty?
         csv << (cell || data).values
@@ -41,19 +41,19 @@ class ActiveReport::Record
   def import
     if @model.nil? || (@model.superclass != ActiveRecord::Base)
       raise ArgumentError,
-        "Model must be an ActiveRecord::Base object."
+        "Model must be an ActiveRecord::Base object.".freeze
     end
 
     @only   = [].push(@only).compact   unless @only.is_a?(Array)
     @except = [].push(@except).compact unless @except.is_a?(Array)
 
     @datum = ActiveReport::Hash.import(@datum, headers: @headers, options: @options)
-    @datum.each do |data|
-      data.transform_keys! { |k| k.to_s.downcase.gsub(" ", "_").to_sym }
-      data.except!(:id)
+    @datum.lazy.each do |data|
+      data.transform_keys! { |k| k.to_s.downcase.gsub(' '.freeze, '_'.freeze).to_sym }
 
       data.only!(@only)     unless @only.empty?
       data.except!(@except) unless @except.empty?
+      data.except!(:id)
 
       @model.create(data)
     end

data/lib/active_report/version.rb

@@ -1,3 +1,3 @@
 module ActiveReport
-  VERSION = "2.2.0"
+  VERSION = "2.3.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_report
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.3.0
 platform: ruby
 authors:
 - Juan Gomez
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-08-22 00:00:00.000000000 Z
+date: 2015-10-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord