property_sets 0.0.8 → 0.0.9

data/VERSION

@@ -1 +1 @@
-0.0.8
+0.0.9

data/lib/property_sets/active_record_extension.rb

@@ -1,5 +1,30 @@
+require 'delegate'
+
 module PropertySets
   module ActiveRecordExtension
+
+    class PropertySetProxy < Delegator
+      attr_accessor :record
+
+      def initialize(record)
+        self.record = record
+      end
+
+      def __getobj__
+        record
+      end
+
+      def id
+        record.id
+      end
+
+      def create(args = {})
+        record.attributes = args
+        record.save
+        record
+      end
+    end
+
     module ClassMethods
       def property_set(association, &block)
         raise "Invalid association name, letters only" unless association.to_s =~ /[a-z]+/
@@ -40,7 +65,8 @@ module PropertySets
             # The finder method which returns the property if present, otherwise a new instance with defaults
             define_method "lookup" do |arg|
               instance = detect { |property| property.name.to_sym == arg }
-              instance ||= property_class.new(@owner.class.name.underscore.to_sym => @owner, :name => arg.to_s, :value => property_class.default(arg))
+              instance ||= property_class.new(@owner.class.name.underscore.to_sym => @owner, :name => arg.to_s)
+              PropertySetProxy.new(instance)
             end
           end
         end

data/lib/property_sets/property_set_model.rb

@@ -2,18 +2,6 @@ module PropertySets
   module PropertySetModel
     module InstanceMethods
 
-      def create
-        self.value = self.class.default(name.to_sym) if self.value.nil?
-        if new_record?
-          super
-        end
-        self
-      end
-
-      def protected?
-        self.class.protected?(self.name.to_sym)
-      end
-
       def false?
         [ "false", "0", "", "off", "n" ].member?(value.to_s.downcase)
       end
@@ -22,6 +10,10 @@ module PropertySets
         !false?
       end
 
+      def value
+        read_attribute(:value) || self.class.default(name.to_sym)
+      end
+
       def to_s
         value.to_s
       end
@@ -65,10 +57,6 @@ module PropertySets
         @properties.keys
       end
 
-      def protected?(key)
-        !!@properties[key][:protected]
-      end
-
       def default(key)
         @properties[key] && @properties[key].key?(:default) ? @properties[key][:default] : nil
       end

data/property_sets.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{property_sets}
-  s.version = "0.0.8"
+  s.version = "0.0.9"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Morten Primdahl"]
-  s.date = %q{2011-01-12}
+  s.date = %q{2011-01-13}
   s.description = %q{This gem is an ActiveRecord extension which provides a convenient interface for managing per row properties}
   s.email = %q{morten@zendesk.com}
   s.extra_rdoc_files = [

data/test/test_property_sets.rb

@@ -6,7 +6,7 @@ class Account < ActiveRecord::Base
     property :bar
     property :baz
     property :hep, :default   => 'skep'
-    property :bob, :protected => true
+    property :bob
   end
 
   property_set :texts do
@@ -81,7 +81,7 @@ class TestPropertySets < ActiveSupport::TestCase
       @account.settings.foo.destroy
       @account.reload
       assert @account.settings.foo.new_record?
-      assert @account.settings.foo.update_attributes(:value => 8)
+      assert @account.settings.foo.create(:value => 8)
       assert @account.settings.foo.id
       assert @account.settings.foo.value == "8"
       assert @account.settings.hep.create

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: property_sets
 version: !ruby/object:Gem::Version 
-  hash: 15
+  hash: 13
   prerelease: false
   segments: 
   - 0
   - 0
-  - 8
-  version: 0.0.8
+  - 9
+  version: 0.0.9
 platform: ruby
 authors: 
 - Morten Primdahl
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-12 00:00:00 -08:00
+date: 2011-01-13 00:00:00 -08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency