Project

General

Profile

Actions

バグ #1239

closed

絵文字のあとにメンションがあるとリンクが正しく作らず、リンクもある場合大量のscore_by_regexpが実行され続ける

Added by Akira Ouchi over 6 years ago. Updated over 6 years ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
twitter
ブランチ:
クラッシュする:
No

Description

mikutterを--debugで実行しているとたまに「{:trim=>"アカウント名", :target=>"@アカウント名"}」という出力があり
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/master/entry/core/plugin/twitter/twitter.rb#L476
で出力していることまで確認したのですが、これが無限に出力され続ける事象を確認しました。

デバッグコードにツイートの文字列を出すように追記したところ、該当のツイートを確認できました。

puts({trim: trim, target: target})

puts({text:text,trim: trim, target: target})

ツイート内容から再現テストした結果、絵文字のあとにメンションがあるとリンクが正しく作られないことがわかりました。

(1) ☺︎︎ みたいな顔文字のあとにメンションがあると、メンションのリンクが正しく作られない。Unicodeのバイト数の解釈的ななにか?
(2) 絵文字がなければメンションのリンクが正常に作られる
(3) (1)に加え、メンションの後にリンクがあると、score_by_regexpが実行され続けて大量のputs({trim: trim, target: target})が出力されるようになる

なお、(3)の現象は、画面上に該当のツイートがなければ出力が止まることを確認しています。


Files

Akkiesoft-994952426481307649.png (52.8 KB) Akkiesoft-994952426481307649.png Akira Ouchi, 2018-05-11 23:54

Related issues

Related to 提案 #1241: Scoreの初期値をModelが提供できるようにするtoshi_aの判断待ちActions
Related to バグ #1242: scoreが何度も呼ばれる終了あひる 家鴨Actions
Related to バグ #1273: puts({trim: trim, target: target}) のデバッグメッセージが残っている終了Izumi TsutsuiActions
Actions #1

Updated by cob odo over 6 years ago

  • Related to 提案 #1241: Scoreの初期値をModelが提供できるようにする added
Actions #2

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 分類待ち to 実装待ち

(1) ☺︎︎ みたいな顔文字のあとにメンションがあると、メンションのリンクが正しく作られない。Unicodeのバイト数の解釈的ななにか?
(2) 絵文字がなければメンションのリンクが正常に作られる

twemojiプラグインの問題だと考えたので、そちらにpull-req送ってます。

https://github.com/cobodo/mikutter-twemoji/pull/1

(3) (1)に加え、メンションの後にリンクがあると、score_by_regexpが実行され続けて大量のputs({trim: trim, target: target})が出力されるようになる

これは誤ってコミットしてしまったものですね。削除しておきます。

また、画面内にある時に複数回呼ばれる問題は、 #1242 で報告されているので、こちらで扱います

Actions #3

Updated by toshi_a 初音 over 6 years ago

  • Related to バグ #1242: scoreが何度も呼ばれる added
Actions #4

Updated by Izumi Tsutsui over 6 years ago

  • Related to バグ #1273: puts({trim: trim, target: target}) のデバッグメッセージが残っている added
Actions #5

Updated by Izumi Tsutsui over 6 years ago

  • Status changed from 実装待ち to 終了

(1)(2) はプラグイン側の問題でプルリクマージ済み、
(3) のうち score が複数回呼ばれる問題は #1242 ということなので
(3) の puts の trim メッセージの件のみ分離して #1273 に入れました。
こちらはクローズにします。

Actions

Also available in: Atom PDF