eav_presenter 0.0.1 → 0.2

data/lib/eav_presenter.rb

@@ -17,6 +17,7 @@ module EavPresenter
   end
 
   def load_eav_attr_values_if_not_loaded
+    return [] if self.new_record?
     entity_foreign_key = self.class.instance_variable_get("@eav_entity_foreign_key")
     @eav_attr_values ||= self.class.instance_variable_get("@eav_value_class").send("find_all_by_#{entity_foreign_key}",self.id)
   end
@@ -24,12 +25,12 @@ module EavPresenter
   def save_eav_attr_values
     self.class.instance_variable_get("@eav_attrs").each { |at|
       eavs = load_eav_attr_values_if_not_loaded
-      eav = eavs.find { |i| i.attrId == at.id } #eav in the db
       name = at.eav_attr_name
       entity_foreign_key = self.class.instance_variable_get("@eav_entity_foreign_key")
       attr_foreign_key = self.class.instance_variable_get("@eav_attr_foreign_key")
       eav_value_class = self.class.instance_variable_get("@eav_value_class")
-      ivalue = send(name)
+      eav = eavs.find { |i| i.send(attr_foreign_key) == at.id } #eav in the db, can be missing
+      ivalue = send(name).to_s
       if attribute_changed?(name)
         if eav
           if ivalue.blank?
@@ -70,16 +71,16 @@ module EavPresenter
           if instance_variables.include?("@#{at.eav_attr_name}")
             instance_variable_get("@#{at.eav_attr_name}")
           else
-            eavs.find { |i| i.attrId == at.id }.try(:value)
+            eavs.find { |i| i.send(self.class.instance_variable_get("@eav_attr_foreign_key")) == at.id }.try(:value)
           end
         }
 
         define_method("#{at.eav_attr_name}=") { |value|
-          raise "EAV value for #{at.eav_attr_name} must be a string or nil" unless value.nil? || value.is_a?(String)
+          # raise "EAV value for #{at.eav_attr_name} must be a string or nil" unless value.nil? || value.is_a?(String)
           eavs = load_eav_attr_values_if_not_loaded
           instance_variable_set("@#{at.eav_attr_name}",value)
-          old = eavs.find { |i| i.attrId == at.id }.try(:value)
-          if value == old
+          old = eavs.find { |i| i.send(self.class.instance_variable_get("@eav_attr_foreign_key")) == at.id }.try(:value)
+          if value.to_s == old.to_s # to consider nil and blank as same
             changed_attributes.delete(at.eav_attr_name)
           else
             changed_attributes[at.eav_attr_name] = old
@@ -91,4 +92,3 @@ module EavPresenter
     end
   end
 end
-

data/lib/test.rb

@@ -0,0 +1,68 @@
+require 'rubygems'
+require 'active_record'
+require 'eav_presenter.rb'
+
+ActiveRecord::Base.establish_connection({
+  :adapter => 'jdbcmysql',
+  :host => 'localhost',
+  :username => 'root',
+  :database => 'eav_test'
+})
+
+ActiveRecord::Migration.class_eval do
+  create_table :posts do |t|
+        t.integer  :id
+        t.string :title
+   end
+
+   create_table :post_attributes do |t|
+      t.integer :id
+      t.string :name
+   end
+
+   create_table :post_attr_values do |t|
+      t.integer :post_id
+      t.integer :attr_id
+      t.string :value
+   end 
+end
+
+class Post < ActiveRecord::Base
+  set_table_name :posts
+end
+
+class PostAttribute < ActiveRecord::Base
+  set_table_name :post_attributes
+  def eav_attr_name
+    name
+  end
+end
+
+
+class PostAttrValue < ActiveRecord::Base
+  set_table_name :post_attr_values
+end
+
+class EavPost < Post
+  include EavPresenter
+  eav :eav_attr_class => PostAttribute, :eav_value_class => PostAttrValue, :eav_entity_foreign_key => "post_id",
+    :eav_attr_foreign_key => "attr_id"
+end
+
+Post.create({:title => 'p1'})
+Post.create({:title => 'p2'})
+PostAttribute.create({:name => 'email'})
+PostAttribute.create({:name => 'twitter'})
+PostAttribute.create({:name => 'facebook'})
+PostAttribute.create({:name => 'web'})
+
+
+EavPost.create({:title => 'p3', :twitter => 'tw3', :web => ''})
+
+=begin
+ActiveRecord::Migration.class_eval do
+   drop_table :posts
+   drop_table :post_attribute
+   drop_table :post_attr_values
+end
+=end

metadata

@@ -2,7 +2,7 @@
 name: eav_presenter
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.1
+  version: "0.2"
 platform: ruby
 authors: 
   - Nilesh Trivedi
@@ -10,8 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-10-30 00:00:00 +05:30
-default_executable: 
+date: 2012-03-19 00:00:00 Z
 dependencies: 
   - !ruby/object:Gem::Dependency 
     name: activerecord
@@ -35,7 +34,7 @@ extra_rdoc_files: []
 
 files: 
   - lib/eav_presenter.rb
-has_rdoc: true
+  - lib/test.rb
 homepage: http://github.com/nileshtrivedi/eav_presenter
 licenses: []
 
@@ -59,7 +58,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.5.1
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: ActiveRecord model like presenter for Entity-Attribute-Value pattern