Merge pull request #304 from mocoso/fix-autolink-for-emoji

Ruby: Fix auto_link so that it doesn't remove emoji
twitter · Mar 25, 2020 · 03dcea5 · 03dcea5
2 parents 4f3c985 + 541aede
commit 03dcea5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/rb/lib/twitter-text/autolink.rb b/rb/lib/twitter-text/autolink.rb
@@ -81,6 +81,8 @@ def auto_link_entities(text, entities, options = {}, &block)
  link_to_screen_name(entity, chars, options, &block)
  elsif entity[:cashtag]
  link_to_cashtag(entity, chars, options, &block)
+ elsif entity[:emoji]
+ entity[:emoji]
  end
  end
  end

diff --git a/rb/spec/autolinking_spec.rb b/rb/spec/autolinking_spec.rb
@@ -18,6 +18,16 @@ def url; end
  @autolinked_text = TestAutolink.new.auto_link(original_text) if original_text
  end
 
+ describe "plain text" do
+ context "plain text with emoji" do
+ def original_text; "gotcha 👍"; end
+
+ it "should be unchanged" do
+ expect(@autolinked_text).to eq("gotcha 👍")
+ end
+ end
+ end
+
  describe "username autolinking" do
  context "username preceded by a space" do
  def original_text; "hello @jacob"; end