Merge pull request #10 from Yoomee/bug/document-regexrc

Switch to using Nokogiri for parsing document slugs

lib/trough/pig/hooks.rb

@@ -76,7 +76,19 @@ def determine_text_change(key, content_chunk)
       def find_documents(value)
         # Find all links to /documents/:slug and return the slugs
         return [] if value.nil?
-        value.scan(/href=\S*\/documents\/(\S+[^\\])\\?['"]>/).flatten
+        html = Nokogiri::HTML(value)
+        links = find_links_for_documents(html)
+        grab_slugs_from_links(links)
+      end
+
+      def find_links_for_documents(html)
+        html.xpath("//a[contains(@href, '/documents/')]")
+      end
+
+      def grab_slugs_from_links(links)
+        links.map do |link|
+          link[:href].split("/documents/").last
+        end
       end
 
       def unlink_document_usages

trough.gemspec

@@ -23,6 +23,7 @@ Gem::Specification.new do |s|
   s.add_dependency "refile-s3"
   s.add_dependency "aws-sdk"
   s.add_dependency 'formtastic-bootstrap', '~> 3.0'
+  s.add_dependency "nokogiri"
 
   s.add_development_dependency "sqlite3"
 end