Changeset b71a93afaa069e73dee680b0a22e95f77a1cd660

Timestamp:

07/28/10 10:09:13 (5 weeks ago)

Author:

Alban Peignier <alban.peignier@…>

Parents:

9c57242449064854924f422f41f763d41dcae867

Children:

e67e28d59604c3c34a634ea37824b23bc828b268

git-committer:

Alban Peignier <alban.peignier@free.fr> / 2010-07-28T10:09:13Z+0200

Message:

Add :allow_blank option to acts_as_ip_port

Files:

• lib/acts_as_ip_port.rb (modified) (2 diffs)
• spec/lib/acts_as_ip_port_spec.rb (modified) (2 diffs)

lib/acts_as_ip_port.rb

@@ -9 +9 @@
     def acts_as_ip_port(*names)
       options = names.extract_options!
+      
+      options = { :allow_blank => false }.update(options)
 
       minimum_value, message = 
@@ -25 +27 @@
         end
 
-        validates_numericality_of name, :only_integer => true, :greater_than => minimum_value, :less_than => 65536, :message => message
+        validates_numericality_of name, :only_integer => true, :greater_than => minimum_value, :less_than => 65536, :message => message, :allow_blank => options[:allow_blank]
       end
     end

spec/lib/acts_as_ip_port_spec.rb

@@ -10 +10 @@
     acts_as_ip_port :port
     acts_as_ip_port :user_port, :user_port => true
+    acts_as_ip_port :optionnal_port, :allow_blank => true
   end
 
@@ -41 +42 @@
   end
 
+  it "should not accept value blank value by default" do
+    @model.should_not allow_values_for(:port, "")
+  end
+
+  context "when allow_blank is true" do
+
+    it "should accept value blank value" do
+      @model.should allow_values_for(:optionnal_port, "")
+    end
+    
+  end
+
   describe "user port" do