shuber-attr_encrypted 1.0.7 → 1.0.8

data/CHANGELOG

@@ -1,3 +1,6 @@
+2009-01-18 - Sean Huber (shuber@huberry.com)
+  * Don't attempt to encrypt/decrypt empty strings
+
 2009-01-14 - Sean Huber (shuber@huberry.com)
   * Move Class logic into Object
 

data/lib/attr_encrypted.rb

@@ -152,8 +152,8 @@ module Huberry
           
           define_class_method "encrypt_#{attribute}" do |value|
             if options[:if] && !options[:unless]
-              if value.nil?
-                encrypted_value = nil
+              if value.nil? || (value.is_a?(String) && value.empty?)
+                encrypted_value = value
               else
                 value = Marshal.dump(value) if options[:marshal]
                 encrypted_value = options[:encryptor].send options[:encrypt_method], options.merge(:value => value)
@@ -167,8 +167,8 @@ module Huberry
           
           define_class_method "decrypt_#{attribute}" do |encrypted_value|
             if options[:if] && !options[:unless]
-              if encrypted_value.nil?
-                decrypted_value = nil
+              if encrypted_value.nil? || (encrypted_value.is_a?(String) && encrypted_value.empty?)
+                decrypted_value = encrypted_value
               else
                 encrypted_value = encrypted_value.unpack(options[:encode]).to_s if options[:encode]
                 decrypted_value = options[:encryptor].send(options[:decrypt_method], options.merge(:value => encrypted_value))

data/test/attr_encrypted_test.rb

@@ -75,6 +75,10 @@ class AttrEncryptedTest < Test::Unit::TestCase
     assert_nil User.encrypt_email(nil)
   end
   
+  def test_should_not_encrypt_empty_string
+    assert_equal '', User.encrypt_email('')
+  end
+  
   def test_should_encrypt_email
     assert_not_nil User.encrypt_email('test@example.com')
     assert_not_equal 'test@example.com', User.encrypt_email('test@example.com')
@@ -92,6 +96,10 @@ class AttrEncryptedTest < Test::Unit::TestCase
     assert_nil User.decrypt_email(nil)
   end
   
+  def test_should_not_decrypt_empty_string
+    assert_equal '', User.decrypt_email('')
+  end
+  
   def test_should_decrypt_email
     encrypted_email = User.encrypt_email('test@example.com')
     assert_not_equal 'test@test.com', encrypted_email

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: shuber-attr_encrypted
 version: !ruby/object:Gem::Version 
-  version: 1.0.7
+  version: 1.0.8
 platform: ruby
 authors: 
 - Sean Huber