Project

General

Profile

Bug(バグ) #2253

Yahoo! ケータイから入力した絵文字が [] という文字列で保存されてしまう

Added by Rimpei Ogawa over 11 years ago. Updated about 7 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2011-06-29
Due date:
% Done:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

#2135 の変更以降で発生します。

Yahoo! ケータイから入力した絵文字が [] という文字列で保存されてしまう。

Causes (原因)

入力の1バイト目がShiftJISで 0xF7, 0xF9, 0xFB の場合、そのバイトを含む2バイトをSoftbank絵文字と見なし、バイナリ列からOpenPNE形式文字列への変換を行う。

従来、この2バイトをSoftBank旧来の「ISO-2022絵文字 (1B 24 xx yy 0F)」に変換した上でテーブルルックアップしていたが、#2135 にてテーブル側(→lib/vendor/OpenPNE2/KtaiEmoji/Softbank.php)がこのバイナリフォーマットを廃し、実体参照で保持するように変更されたので、ルックアップに失敗するようになったのが原因である。

Way to fix (修正内容)

lib/vendor/OpenPNE2/KtaiEmoji.php において、ISO-2022絵文字ではなく実体参照をキーとしてテーブルルックアップを行うように修正する。


Related issues

Related to OpenPNE 3 - Bug(バグ) #2135: Yahoo! ケータイにおける SSL 仕様変更対応(絵文字) Fixed(完了) 2011-05-25
Related to OpenPNE 3 - Backport(バックポート) #2254: Yahoo! ケータイから入力した絵文字が [] という文字列で保存されてしまう Fixed(完了) 2011-06-29
Related to OpenPNE 3 - Backport(バックポート) #2255: Yahoo! ケータイから入力した絵文字が [] という文字列で保存されてしまう Fixed(完了) 2011-06-29

Associated revisions

Revision 4d842ba9 (diff)
Added by Naoya Tozuka over 11 years ago

Yahoo! ケータイで入力した絵文字が正しく保存されない問題(#2135以降)の解決 (fixes #2253)

Revision b714d9d1 (diff)
Added by Naoya Tozuka over 11 years ago

(refs #2253) Reverted commits for an appropriate fix

Revision bc724f31 (diff)
Added by Naoya Tozuka over 11 years ago

Fixed the problem on Yahoo! mobile that saved emoji as empty string '[]' after #2135 (fixes #2253)

Revision 747cadbe (diff)
Added by Naoya Tozuka over 11 years ago

renamed the variable $web to $codepoint which explains itself better (fixes #2253)

Revision ee80c5cc (diff)
Added by Naoya Tozuka over 11 years ago

Added a unit test for #2253 (refs #2253)

History

#1 Updated by Naoya Tozuka over 11 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 4d842ba9439f0c498492382f74fbca40abd4b780 で適用されました。
(コミットメッセージが日本語になっていましたので、 b714d9d1 にて一旦revertします)

#2 Updated by Naoya Tozuka over 11 years ago

更新履歴 bc724f316912208a3292791e462513549302bc6e で適用されました。

#3 Updated by Rimpei Ogawa over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです。

#4 Updated by Rimpei Ogawa over 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
  • % Done changed from 70 to 50

#5 Updated by Rimpei Ogawa over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

再度ソースコードを確認したところ、 $web という変数名が不適切ですので変更してください。

#6 Updated by Naoya Tozuka over 11 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 commit:"671dff6d5279c872f0de2303c0569268686a7763" で適用されました。

#7 Updated by Naoya Tozuka over 11 years ago

更新履歴 747cadbe51c7e61a2393e1925c0f46e88b648274 で適用されました。

#8 Updated by Rimpei Ogawa over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

修正確認しました。OKです。

#9 Updated by Minoru Takai over 11 years ago

  • Priority changed from Normal(通常) to High(高め)
  • Target version set to OpenPNE 3.7.0
  • 3.6 で発生するか set to Yes

このチケットの内容に対して優先度が Normal なのは不自然なので High に引き上げます。

#10 Updated by Shingo Yamada over 11 years ago

  • 360対象 set to beta13

#11 Updated by Yuma Sakata about 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#12 Updated by kaoru n about 7 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF