linky 0.0.8 → 0.0.8.1

data/lib/linky/linky.rb

@@ -3,7 +3,7 @@ class Linky
     # Link a keyword to a target within some HTML. 
     #   * keyword: what you want linked
     #   * target: where you want it linked to
-    #   * html: the html you want to look for the keyword in
+    #   * html: the html you want to look for the keyword in, or Nokogiri::HTML::DocumentFragment object.
     #   * options: any extra attributes (besides href) that you want on the link.
     #
     # Here's an example:
@@ -12,12 +12,17 @@ class Linky
     #     html = "<p>link all occurances of the word "more."</p><div>some more text</div>"
     #     Linky.link "more", "http://more.com", html, :class => "linky"
     #     # returns "<p>link the word "<a href="http://more.com" class="linky">more</a>."</p><div>some <a href="http://more.com" class="linky">more</a> text</div>"
+    #
+    # if you provide parameter html as DocumentFragment object, after calling link(), that object is updated as well.
+    # Means the output of link(keyword, target, html) == html.to_xhtml
     def link(keyword, target, html, options={})
       options = options.map {|k,v| %{#{k}="#{v}"}}.join " " 
       block = proc {|keyword| %{<a href="#{target}" #{options}>#{keyword}</a>}}
-      html_fragment = Nokogiri::HTML::DocumentFragment.parse html
+      html_fragment = html.kind_of?(Nokogiri::HTML::DocumentFragment) ? html : Nokogiri::HTML::DocumentFragment.parse(html)
+      html_fragment_orig = html_fragment.dup
+
       real_link keyword.to_s, html_fragment, &block
-      raise "We lost some content in attempting to link it. Please report a bug" unless html_fragment.text == Nokogiri::HTML::DocumentFragment.parse(html).text 
+      raise "We lost some content in attempting to link it. Please report a bug" unless html_fragment.text == html_fragment_orig.text 
       html_fragment.to_xhtml
     end
 

data/spec/linky_spec.rb

@@ -48,4 +48,11 @@ describe Linky, "#link" do
     new_html = Linky.link "apple", "topics/apple", html
     new_html.should == %(<a href="topics/apple">apple</a> &amp; orange)
   end
+
+  it "should process pre parsed html fragment object" do
+    html = "apple &amp; orange"
+    frag = Nokogiri::HTML::DocumentFragment.parse(html)
+    new_html = Linky.link "apple", "topics/apple", frag
+    new_html.should == %(<a href="topics/apple">apple</a> &amp; orange)
+  end
 end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: linky
 version: !ruby/object:Gem::Version 
-  hash: 15
+  hash: 109
   prerelease: 
   segments: 
   - 0
   - 0
   - 8
-  version: 0.0.8
+  - 1
+  version: 0.0.8.1
 platform: ruby
 authors: 
 - Matt Parker
@@ -15,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-02-16 00:00:00 -05:00
+date: 2011-02-27 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency