Project

General

Profile

Bug（バグ） #2253

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

Added by Rimpei Ogawa about 13 years ago. Updated almost 9 years ago.

Status:
Fixed（完了）
Priority:
High（高め）
Assignee:
Naoya Tozuka
Target version:
OpenPNE 3.7.0
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 about 13 years ago

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

Revision b714d9d1 (diff)
Added by Naoya Tozuka about 13 years ago

(refs #2253) Reverted commits for an appropriate fix

Revision bc724f31 (diff)
Added by Naoya Tozuka about 13 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 about 13 years ago

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

Revision ee80c5cc (diff)
Added by Naoya Tozuka about 13 years ago

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

History

#1 Updated by Naoya Tozuka about 13 years ago

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

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

#2 Updated by Naoya Tozuka about 13 years ago

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

#3 Updated by Rimpei Ogawa about 13 years ago

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

レビューOKです。

#4 Updated by Rimpei Ogawa about 13 years ago

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

#5 Updated by Rimpei Ogawa about 13 years ago

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

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

#6 Updated by Naoya Tozuka about 13 years ago

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

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

#7 Updated by Naoya Tozuka about 13 years ago

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

#8 Updated by Rimpei Ogawa about 13 years ago

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

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

#9 Updated by Minoru Takai about 13 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 about 13 years ago

  • 360対象 set to beta13

#11 Updated by Yuma Sakata almost 13 years ago

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

テストOKです。

#12 Updated by kaoru n almost 9 years ago

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

Also available in: Atom PDF