magerecord 0.1.2 → 0.1.3

data/VERSION

@@ -1 +1 @@
-0.1.2
+0.1.3

data/lib/magerecord/eav_record.rb

@@ -24,11 +24,29 @@ module MageRecord
 
 
     def method_missing(meth, *args, &block)
+      eav = get_eav_records(meth)
+
+      if eav.count > 0
+        # return value of Magento EAV attribute if available
+        eav.first
+      else
+        # call superclass's method_missing method
+        # or risk breaking Ruby's method lookup
+        super
+      end
+    end
+
+
+
+
+    private
+
+    def get_eav_records(attrib_sym)
       conn = self.class.connection
       eav_prefix = self.class.eav_prefix
 
       # note: select_values returns just one value from a single db column
-      eav = conn.select_values <<-RAWSQL.strip_heredoc
+      conn.select_values <<-RAWSQL.strip_heredoc
         SELECT
           CASE ea.backend_type
             WHEN 'varchar'  THEN entity_varchar.value
@@ -61,18 +79,9 @@ module MageRecord
         LEFT JOIN #{eav_prefix}_entity_decimal  AS entity_decimal ON entity.entity_id = entity_decimal.entity_id AND ea.attribute_id = entity_decimal.attribute_id AND ea.backend_type = 'decimal'
         LEFT JOIN #{eav_prefix}_entity_datetime AS entity_datetime ON entity.entity_id = entity_datetime.entity_id AND ea.attribute_id = entity_datetime.attribute_id AND ea.backend_type = 'datetime'
 
-        WHERE ea.attribute_code = #{conn.quote(meth)}
+        WHERE ea.attribute_code = #{conn.quote(attrib_sym)}
           AND entity.entity_id = #{id}
       RAWSQL
-
-      if eav.count > 0
-        # return value of Magento EAV attribute if available
-        eav.first
-      else
-        # call superclass's method_missing method
-        # or risk breaking Ruby's method lookup
-        super
-      end
     end
   end
 end

data/lib/magerecord/order_item.rb

@@ -15,7 +15,12 @@ module MageRecord
 
     # call associated product's method
     def method_missing(meth, *args, &block)
-      product.nil? ? super : product.send(meth)
+      super || (product && product.send(meth))
+    end
+
+
+    def respond_to?(meth, include_private = false)
+      super || (product && product.respond_to?(meth))
     end
   end
 end

data/magerecord.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "magerecord"
-  s.version = "0.1.2"
+  s.version = "0.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Marc"]
-  s.date = "2014-04-01"
+  s.date = "2014-05-09"
   s.description = "A simple ActiveRecord wrapper for various Magento models"
   s.email = "xerokill@gmail.com"
   s.extra_rdoc_files = [

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: magerecord
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-01 00:00:00.000000000 Z
+date: 2014-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -170,7 +170,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4499826596874981208
+      hash: -2096501154630265700
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: