crazy_harry 0.1.0 → 0.1.2

data/README.md

@@ -100,12 +100,20 @@ will return:
  
     <h3>Hotel</h3> lodging
 
+## Fostering orphaned tags.
+
+The `.foster!` command will wrap orphaned li tags in a ul tag.  **This
+only works for li tag for now.**
+
+    CrazyHarry.fragment('<li>Flying list tag</li>').foster!.to_s
+
 ## Known Issues/TODO
 
   * De-duping does not take account of whitespace.  So, `<p>Some Content</p>` and `<p>Some Content </p>` will not be treated as duplicates.
   * Be able to turn off default actions.
   * It should be able to work on documents as well as fragments.
   * Merge `.translate!` with `.change!` 
+  * Foster other orphaned tags besides just li.
 
 ## Contributing
 

data/lib/crazy_harry/base.rb

@@ -4,6 +4,7 @@ module CrazyHarry
     include CrazyHarry::Change
     include CrazyHarry::Redact
     include CrazyHarry::Translate
+    include CrazyHarry::Foster
 
     attr_accessor :fragment, :scope, :steps, :text
 

data/lib/crazy_harry/change.rb

@@ -16,6 +16,7 @@ module CrazyHarry
 
       self.steps << generic_from_to
       self.steps << unwrap_unnecessary_paragraphs
+      self.steps << foster_orphaned_nodes
 
       run!
 
@@ -55,5 +56,27 @@ module CrazyHarry
       Loofah::Elements::BLOCK_LEVEL.include?(node.children.first.name)
     end
 
+    def foster_orphaned_nodes
+      Loofah::Scrubber.new do |node|
+        node.extend(FosterModule)
+        node.foster!
+      end
+    end
+  end
+  
+  module FosterModule
+    def foster!
+      if orphaned_li?
+        self.replace wrap_li
+      end
+    end
+
+    def orphaned_li?
+      name == 'li' && parent.name !~ /ol|ul/
+    end
+    
+    def wrap_li
+      Nokogiri.make("<ul>#{self}</ul>") 
+    end
   end
 end

data/lib/crazy_harry/foster.rb

@@ -0,0 +1,25 @@
+module CrazyHarry
+  module Foster
+
+    def foster!
+      self.steps << foster_orphaned_li
+
+      run!
+
+      self
+    end
+
+    private
+
+    def foster_orphaned_li
+      Loofah::Scrubber.new do |node|
+        node.replace("<ul>#{node}</ul>") if orphaned_li?(node)
+      end
+    end
+
+    def orphaned_li?(node)
+      node.name == 'li' && node.parent.name !~ /ol|ul/
+    end
+
+  end
+end

data/lib/crazy_harry/version.rb

@@ -1,3 +1,3 @@
 module CrazyHarry
-  VERSION = "0.1.0"
+  VERSION = "0.1.2"
 end

data/lib/crazy_harry.rb

@@ -1,6 +1,6 @@
 require 'loofah'
 
-%w(default change redact translate base version).each do |load_lib|
+%w(default foster change redact translate base version).each do |load_lib|
   require_relative "crazy_harry/#{load_lib}"
 end
 

data/spec/lib/change_spec.rb

@@ -29,6 +29,11 @@ describe CrazyHarry::Change do
         '<p><h3>Header</h3>Content</p>'
     end
 
+    it "should wrap orphaned list items" do
+      harry.fragment('<p><li>orphan</li><ul><li>item</li></p>').change!(from: 'b', to: 'h3').to_s.should ==
+        '<ul><li>orphan</li></ul><ul><li>item</li></ul>'
+    end
+    
     context "chaining" do
 
        it "should be able to chain calls" do

data/spec/lib/foster_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe CrazyHarry::Foster do
+
+    let(:harry){ CrazyHarry }
+
+    it "should wrap orphaned list items" do
+      harry.fragment('<p><li>orphan</li><ul><li>item</li></p>').foster!.to_s.should ==
+        '<ul><li>orphan</li></ul><ul><li>item</li></ul>'
+    end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: crazy_harry
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-14 00:00:00.000000000 Z
+date: 2012-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2155892060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,15 +21,10 @@ dependencies:
         version: 2.10.0
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 2.10.0
+  version_requirements: *2155892060
 - !ruby/object:Gem::Dependency
   name: guard
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2155891280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -37,15 +32,10 @@ dependencies:
         version: 1.1.1
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 1.1.1
+  version_requirements: *2155891280
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2155890600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -53,15 +43,10 @@ dependencies:
         version: 1.0.1
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 1.0.1
+  version_requirements: *2155890600
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2155889820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -69,15 +54,10 @@ dependencies:
         version: 0.6.4
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 0.6.4
+  version_requirements: *2155889820
 - !ruby/object:Gem::Dependency
   name: loofah
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2155889320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -85,15 +65,10 @@ dependencies:
         version: 1.2.1
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 1.2.1
+  version_requirements: *2155889320
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2155888760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -101,12 +76,7 @@ dependencies:
         version: 1.5.5
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 1.5.5
+  version_requirements: *2155888760
 description: CrazyHarry is a high-level html fragment sanitiser/cleaner based on Loofah.
 email:
 - todd.tyree@lonelyplanet.co.uk
@@ -125,6 +95,7 @@ files:
 - lib/crazy_harry/base.rb
 - lib/crazy_harry/change.rb
 - lib/crazy_harry/default.rb
+- lib/crazy_harry/foster.rb
 - lib/crazy_harry/redact.rb
 - lib/crazy_harry/translate.rb
 - lib/crazy_harry/version.rb
@@ -134,6 +105,7 @@ files:
 - spec/integration/crazy_harry_spec.rb
 - spec/lib/change_spec.rb
 - spec/lib/crazy_harry_spec.rb
+- spec/lib/foster_spec.rb
 - spec/lib/redact_spec.rb
 - spec/lib/translate_spec.rb
 - spec/spec_helper.rb
@@ -157,7 +129,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: crazy_harry
-rubygems_version: 1.8.22
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: A High level HTML fragment sanitizer.
@@ -167,6 +139,7 @@ test_files:
 - spec/integration/crazy_harry_spec.rb
 - spec/lib/change_spec.rb
 - spec/lib/crazy_harry_spec.rb
+- spec/lib/foster_spec.rb
 - spec/lib/redact_spec.rb
 - spec/lib/translate_spec.rb
 - spec/spec_helper.rb