search

pois0 / Juktaway

Lightweight Twitter Client for Android
BSD 2-Clause "Simplified" License
7 stars 1 forks source link

Fix: crash when the url contains $ + number #51

Closed crossfire closed 5 years ago

crossfire commented 5 years ago

ツイート内などの短縮 URL の展開後のものに $ と数字の組み合わせが入っていると，replaceAll に展開後 URL が入った際に当該部分が後方参照として扱われるために，ArrayIndexOutOfBoundsExceptionでクラッシュするようです(Justaway 時代からの不具合)。この PR はそれを解消するものです。

URL 例: https://www.abc.net.au/news/2019-03-25/kenyan-school-teacher-wins-$1-million-teaching-prize/10935276 ツイート例: https://twitter.com/abcnews/status/1109977957479538688

pois0 commented 5 years ago

不要なエスケープ及びインポートの除去, また修正箇所にリファクタリングを施した上でマージしました。 本コミットはv3.0.4にて取り込まれる予定です。 ご協力感謝申し上げます。