linky 0.0.2 → 0.0.3

data/README.markdown

@@ -11,9 +11,9 @@ A simple gem for safely linking keywords within HTML (i.e., it's more than a `gs
 Call the `Linky.link` method: 
 
     require 'linky'
-    html = "<p>link all occurances of the word "more."</p><div>some more text</div>"
+    html = "<p>link the first occurance 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>"
+    # returns "<p>link the first occurance of the word "<a href="http://more.com" class="linky">more</a>."</p><div>some more text</div>"
 
 ## Docs
 

data/lib/linky/linky.rb

@@ -28,15 +28,19 @@ class Linky
         
         # call the block to determine what we'll replace the needle with
         new_needle = block.call match[1]
+
+
        
         # break the string into three parts
         prefix, keyword, postfix = break_on_boundary match[1], haystack.to_s 
-        
+
         haystack.content = prefix
         haystack.add_next_sibling Nokogiri::HTML::DocumentFragment.parse(new_needle)
         if !postfix.nil? and !postfix.empty?
-          haystack.next_sibling.add_next_sibling Nokogiri::HTML::DocumentFragment.parse(postfix)
-          haystack.next_sibling.next_sibling.content = postfix
+          if postfix_content = Nokogiri::HTML::DocumentFragment.parse(postfix)
+            postfix_content.content = postfix
+            haystack.next_sibling.add_next_sibling postfix_content 
+          end
         end
         true
       elsif haystack.name != "a" && haystack.respond_to?(:children)

data/spec/linky_spec.rb

@@ -24,4 +24,10 @@ describe Linky, "#link" do
     new_html = Linky.link "he", "#he", html
     new_html.should == "hi there, <a href=\"#he\">he</a> is funny and he is a he.<div>more he stuff.</div>"
   end
+
+  it "should not remove links" do
+    html = "<p>California <a href=\"topics/cabernet-sauvignon\">Cabernet Sauvignon</a></p>"
+    new_html = Linky.link "California", "topics/california", html
+    new_html.should == "<p><a href=\"topics/california\">California</a> <a href=\"topics/cabernet-sauvignon\">Cabernet Sauvignon</a></p>"
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: linky
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: false
   segments: 
   - 0
   - 0
-  - 2
-  version: 0.0.2
+  - 3
+  version: 0.0.3
 platform: ruby
 authors: 
 - Matt Parker
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-12-05 00:00:00 -05:00
+date: 2010-12-07 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency