loofah 0.2.1 → 0.2.2

data.tar.gz.sig

@@ -1,2 +1,4 @@
-P=w	���9e�2��3[#�ft������$����1�.����<t�&dă\,����+v��/���SFZ��},��h�#�:��5�����I��Ȟ`���h�^/�������m����4�A+�-��;�l��a����Ϊ��+�O+pU@?=&=W�#ܳIW���;�w^ÍU�
-H�(��^5�Us,@��e��\t>�s�}�*$�p^-6���c�b�S|U0���cT�mJU��	��H��N^�E��
+ z�L&����3V���(y|5�+�k�7��<�Aae*<dIw���M�x�hLQ�F��1�מ�.
naa���
+K����	�@'��8�_j��ޗ	��w�!�٫���|�T�[[nd�
+2�H��<YQ�Nl|U�Ap:�Z2���ZH����;V�L�H�و��q%R����n����c�feFO
+s譸8p��AƇ���yƺS�ef����$���m��J�ǸvLೢN�j�gV:�����*G,�þ�

data/CHANGELOG.rdoc

@@ -1,5 +1,12 @@
 = Changelog
 
+== 0.2.2 (2009-09-30)
+
+Enhancements:
+
+* ActiveRecord extension scrubs fields in a before_validation callback
+  (was previously in a before_save)
+
 == 0.2.1 (2009-09-19)
 
 Enhancements:

data/README.rdoc

@@ -64,7 +64,7 @@ OR
 
   # app/model/post.rb
   class Post < ActiveRecord::Base
-    html_fragment :body, :scrub => :prune  # scrubs 'body' in a before_save
+    html_fragment :body, :scrub => :prune  # scrubs 'body' in a before_validation
   end
 
 == REQUIREMENTS

data/lib/loofah.rb

@@ -161,7 +161,7 @@ require 'loofah/deprecated'
 #
 module Loofah
   # The version of Loofah you are using
-  VERSION = '0.2.1'
+  VERSION = '0.2.2'
 
   # The minimum required version of Nokogiri
   REQUIRED_NOKOGIRI_VERSION = '1.3.3'

data/lib/loofah/active_record.rb

@@ -1,6 +1,6 @@
 module Loofah
   #
-  # Loofah can scrub ActiveRecord attributes in a before_save callback:
+  # Loofah can scrub ActiveRecord attributes in a before_validation callback:
   #
   #   # in environment.rb
   #   Rails::Initializer.run do |config|
@@ -15,7 +15,7 @@ module Loofah
   #
   #   # app/model/post.rb
   #   class Post < ActiveRecord::Base
-  #     html_fragment :body, :scrub => :prune  # scrubs 'body' in a before_save
+  #     html_fragment :body, :scrub => :prune  # scrubs 'body' in a before_validation
   #   end
   #
   module ActiveRecordExtension
@@ -25,7 +25,7 @@ module Loofah
     #
     def html_fragment(attr, options={})
       raise ArgumentError, "html_fragment requires :scrub option" unless method = options[:scrub]
-      before_save do |record|
+      before_validation do |record|
         record[attr] = Loofah.scrub_fragment(record[attr], method).to_s
       end
     end
@@ -36,7 +36,7 @@ module Loofah
     #
     def html_document(attr, options={})
       raise ArgumentError, "html_document requires :scrub option" unless method = options[:scrub]
-      before_save do |record|
+      before_validation do |record|
         record[attr] = Loofah.scrub_document(record[attr], method).to_s
       end
     end

data/lib/loofah/deprecated.rb

@@ -27,7 +27,7 @@ module Loofah
 
     private
 
-    def warn_once(message)
+    def warn_once(message) # :nodoc:
       @aooga ||= {}
       unless @aooga.key?(message)
         warn message unless @aooga[message]

data/test/test_active_record.rb

@@ -25,11 +25,11 @@ class TestActiveRecord < Test::Unit::TestCase
       should "scrub the specified field" do
         Loofah.expects(:scrub_fragment).with(HTML_STRING, :prune).once
         Loofah.expects(:scrub_fragment).with(PLAIN_TEXT, :prune).never
-        @post.save
+        @post.valid?
       end
 
       should "generate strings" do
-        @post.save
+        @post.valid?
         assert_equal String, @post.html_string.class
         assert_equal HTML_STRING, @post.html_string
       end
@@ -44,11 +44,11 @@ class TestActiveRecord < Test::Unit::TestCase
       should "scrub the specified field, but not other fields" do
         Loofah.expects(:scrub_document).with(HTML_STRING, :strip).once
         Loofah.expects(:scrub_document).with(PLAIN_TEXT, :strip).never
-        @post.save
+        @post.valid?
       end
 
       should "generate strings" do
-        @post.save
+        @post.valid?
         assert_equal String, @post.html_string.class
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: loofah
 version: !ruby/object:Gem::Version 
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors: 
 - Mike Dalessio
@@ -31,7 +31,7 @@ cert_chain:
   FlqnTjy13J3nD30uxy9a1g==
   -----END CERTIFICATE-----
 
-date: 2009-08-19 00:00:00 -04:00
+date: 2009-08-30 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency