Merge pull request #122 from bradly/emoji-pipeline-asset-path

Adding support to supply the path to the emoji sprite in the emoji pipeline

lib/html/pipeline/emoji_filter.rb

@@ -12,6 +12,7 @@ class Pipeline
     #
     # Context:
     #   :asset_root (required) - base url to link to emoji sprite
+    #   :asset_path (optional) - url path to link to emoji sprite. :file_name can be used as a placeholder for the sprite file name. If no asset_path is set "emoji/:file_name" is used.
     class EmojiFilter < Filter
       # Build a regexp that matches all valid :emoji: names.
       EmojiPattern = /:(#{Emoji.names.map { |name| Regexp.escape(name) }.join('|')}):/
@@ -56,10 +57,22 @@ def asset_root
         context[:asset_root]
       end
 
+      # The url path to link emoji sprites
+      #
+      # :file_name can be used in the asset_path as a placeholder for the sprite file name. If no asset_path is set in the context "emoji/:file_name" is used.
+      # Returns the context's asset_path or the default path if no context asset_path is given.
+      def asset_path(name)
+        if context[:asset_path]
+          context[:asset_path].gsub(":file_name", "#{::CGI.escape(name)}.png")
+        else
+          File.join("emoji", "#{::CGI.escape(name)}.png")
+        end
+      end
+
       private
 
       def emoji_url(name)
-        File.join(asset_root, "emoji", "#{::CGI.escape(name)}.png")
+        File.join(asset_root, asset_path(name))
       end
     end
   end

test/html/pipeline/emoji_filter_test.rb

@@ -21,4 +21,10 @@ def test_required_context_validation
     }
     assert_match /:asset_root/, exception.message
   end
+
+  def test_custom_asset_path
+    filter = EmojiFilter.new("<p>:+1:</p>", {:asset_path => ':file_name', :asset_root => 'https://foo.com'})
+    doc = filter.call
+    assert_match "https://foo.com/%2B1.png", doc.search('img').attr('src').value
+  end
 end