acts-as-optionable 0.4.1 → 0.4.2

data/README.rdoc

@@ -44,6 +44,11 @@ You can also store an optional "kind" attribute as a hint to what you are storin
   style.set_option("bgcolor", "white", :kind => "color")
   style.get_option("bgcolor").kind => "color"
   
+Add a "category" if you want to group options:
+
+  style.set_option("bgcolor", "white", :category => "colors")
+  style.get_option("bgcolor").category => "colors"  
+  
 You can also store an optional display name for human consumption:
 
   style.set_option("bgcolor", "white", :display_name => "Background Color")
@@ -55,13 +60,14 @@ You can also store an optional display name for human consumption:
   class StyleWithDefaults < ActiveRecord::Base
     acts_as_optionable
   
-    specify_option :background_color, :default => "white", :kind => "color"
+    specify_option :background_color, :default => "white", :kind => "color", :category => "background"
     specify_option :color, :default => "black"
   end
   
   style = StyleWithDefaults.create
   style.get_option("background_color").value => "white"
   style.get_option("background_color").kind => "color"
+  style.get_option("background_color").category => "background"
   
 === Specifying instance options at runtime
 

data/VERSION

@@ -1 +1 @@
-0.4.1
+0.4.2

data/generators/option/templates/create_options.rb

@@ -5,6 +5,7 @@ class CreateOptions < ActiveRecord::Migration
       t.string :display_name
       t.string :value
       t.string :kind
+      t.string :category
       t.references :optionable, :polymorphic => true
       t.timestamps
     end

data/generators/option/templates/option.rb

@@ -46,6 +46,7 @@ class Option < ActiveRecord::Base
     option_hash["display_name"] = display_name
     option_hash["default"] = default if default
     option_hash["kind"] = kind if kind
+    option_hash["category"] = category if category
     option_hash.dup
   end
     

data/lib/acts_as_optionable/acts_as_optionable.rb

@@ -17,7 +17,7 @@ module ActiveRecord
       module IntanceMethods
         # Store an option persistently and override default option
         def set_option(name, value, opts = {})
-          option = get_stored_option(name) || options.build(:name => name.to_s, :value => value, :kind => opts[:kind], :display_name => opts[:display_name])
+          option = get_stored_option(name) || options.build(:name => name.to_s, :value => value, :kind => opts[:kind], :display_name => opts[:display_name], :category => opts[:category])
           return if !opts[:force_set] && new_option_matches_current?(option, value)
           option.value = value
           ret = option.save!

data/lib/acts_as_optionable/specify_option.rb

@@ -6,7 +6,7 @@ module ActiveRecord
           # Setup a default value at the class level.
           def specify_option(option_name, opts = {})
             name = option_name.to_s
-            optionable_specified_options[name] = Option.new_readonly(:name => name, :default => opts[:default], :kind => opts[:kind], :display_name => opts[:display_name])
+            optionable_specified_options[name] = Option.new_readonly(:name => name, :default => opts[:default], :kind => opts[:kind], :display_name => opts[:display_name], :category => opts[:category])
           end
           
           # Returns a hash of options specified at the class level
@@ -40,7 +40,7 @@ module ActiveRecord
             @instance_specified_options = {}
             opts.each do |option_name, attributes|
               attributes.symbolize_keys!
-              @instance_specified_options[option_name.to_s] = Option.new_readonly(:name => option_name.to_s, :default => attributes[:default], :kind => attributes[:kind], :display_name => attributes[:display_name])
+              @instance_specified_options[option_name.to_s] = Option.new_readonly(:name => option_name.to_s, :default => attributes[:default], :kind => attributes[:kind], :display_name => attributes[:display_name], :category => attributes[:category])
             end
           end
         end

data/spec/acts_as_optionable_spec.rb

@@ -9,7 +9,7 @@ class SpecifiedMixin < Mixin
   acts_as_optionable
   
   specify_option :foo, :default => "FOOFOO"
-  specify_option :bar, :default => "BARBAR", :kind => "example", :display_name => "Bar Bar"
+  specify_option :bar, :default => "BARBAR", :kind => "example", :display_name => "Bar Bar", :category => "tabs"
 end
 
 class NoSpecifiedMixin < Mixin
@@ -31,6 +31,7 @@ def setup_db
       t.string :display_name
       t.string :value
       t.string :kind
+      t.string :category
       t.references :optionable, :polymorphic => true
       t.timestamps
     end
@@ -84,6 +85,10 @@ describe "ActsAsOptionable" do
       @optionable.get_option(:bar).display_name.should == "Bar Bar"
     end
     
+    it "should be able to specify a category" do
+      @optionable.get_option(:bar).category.should == "tabs"
+    end
+    
     it "should not mix specifications across unrelated classes" do
       class Foobar < Mixin
         acts_as_optionable
@@ -123,6 +128,11 @@ describe "ActsAsOptionable" do
       @optionable.get_option(:kind_is_set).kind.should == "example_kind"
     end
     
+    it "should have the category if set" do
+      @optionable.instance_specified_options = @options_template.merge(:category_is_set => {:default => "category_is_set", :category => "tabs" })
+      @optionable.get_option(:category_is_set).category.should == "tabs"
+    end
+    
     it "should have the display name if set" do
       @optionable.instance_specified_options = @options_template.merge(:display_name_is_set => {:default => "kind_is_set", :kind => "example_kind", :display_name => "Example Name" })
       @optionable.get_option(:display_name_is_set).display_name.should == "Example Name"
@@ -204,11 +214,16 @@ describe "ActsAsOptionable" do
         @optionable.get_option(@key).value.should be_false
       end
       
-      it "should allowing storing the option kind" do
+      it "should allow storing the option kind" do
         @optionable.set_option(@key, "red", :kind => "color")
         @optionable.get_option(@key).kind.should == "color"
       end
       
+      it "should allow storing the option category" do
+        @optionable.set_option(@key, "foo", :category => "tabs")
+        @optionable.get_option(@key).category.should == "tabs"
+      end
+      
       it "should not update the option if it already matches current" do
         @optionable.set_option(@key, "red", "color")
         timestamp = @optionable.get_option(@key).updated_at.dup
@@ -314,7 +329,7 @@ describe "ActsAsOptionable" do
   
   describe "getting options and defaults as a hash" do
     before(:each) do
-      @optionable.set_option("example_option", "example_value", :kind => "example", :display_name => "Example Name")
+      @optionable.set_option("example_option", "example_value", :kind => "example", :display_name => "Example Name", :category => "tabs")
       @options_and_defaults_hash = @optionable.options_and_defaults_hash
     end
     
@@ -336,6 +351,10 @@ describe "ActsAsOptionable" do
       @options_and_defaults_hash["bar"]["kind"].should == "example"
     end
     
+    it "should include the category" do
+      @options_and_defaults_hash["example_option"]["category"].should == "tabs"
+    end
+    
     it "should include the display name" do
       @options_and_defaults_hash["example_option"]["display_name"].should == "Example Name"
     end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 4
-  - 1
-  version: 0.4.1
+  - 2
+  version: 0.4.2
 platform: ruby
 authors: 
 - Brendon Murphy
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-05-05 00:00:00 -07:00
+date: 2010-05-06 00:00:00 -07:00
 default_executable: 
 dependencies: []