ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

#2087 closed defect (fixed)

フレンド名称の変更が反映されない場所がある

Reported by: imamura623 Owned by: ogawa
Priority: major Milestone: OpenPNE2.10.6
Component: 指定しない Version: 2.8.x & 2.10.x & 2.11.x
Keywords: OpenPNE2.8.11 OpenPNE2.11.6 Cc:

Description (last modified by kiwa)

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=15846 より転記

携帯版ばっかりいじっているので、今日も携帯ネタです。

バージョン:2.10.0


/webapp/util/ktai.php
で指定している、エラーメッセージの配列ですが、

5 => "このフレンドは、現在リンク承認待ちです",
6 => "このフレンドは、すでにリンク済みです",

のように、テーブル c_admin_config で定義した値を無視して固定値が書かれています。

この場合、

5 => "この".WORD_FRIEND_HALF."は、現在リンク承認待ちです",
6 => "この".WORD_FRIEND_HALF."は、すでにリンク済みです",

などのように定義されるべきではないでしょうか?


同様に、
/do/f_link_request_insert_c_friend_confirm.php
でも、

ktai_display_error('このメンバーは既に'.WORD_MY_FRIEND_HALF.'に登録されています。');

といったエラー処理をしていますが、これも、

ktai_display_error('この'.WORD_FRIEND_HALF'.'は既に'.WORD_MY_FRIEND_HALF.'に登録されています。');
E2.8.11 OpenPNE2.11.6
というように処理すべきだと思います。


2点、例を挙げましたが、要約すると、

『c_admin_config で呼称を定義している箇所に関して、ソースへの固定値ベタ書きは直して欲しい』

ということです。

Change History (8)

comment:1 Changed 12 years ago by kiwa

Priority: minormajor

comment:2 Changed 12 years ago by kiwa

Keywords: 再現待ち removed
Priority: majorminor

comment:3 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.11 OpenPNE2.11.6 added
Milestone: OpenPNE2.10.6

comment:4 Changed 12 years ago by ogawa

Owner: changed from nobody to ogawa
Status: newassigned
Summary: 携帯のエラーメッセージでフレンド名称の変更が反映されない場所があるフレンド名称の変更が反映されない場所がある

pc/adminも併せて修正します。

comment:5 Changed 12 years ago by ogawa

Component: pne-ktai指定しない
Priority: minormajor

comment:6 Changed 12 years ago by ogawa

Keywords: 確認中 added

修正しました。

comment:7 Changed 12 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

確認しました。テストお願いします。

comment:8 Changed 12 years ago by kiwa

Description: modified (diff)
Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

確認しました。問題なさそうです。

Note: See TracTickets for help on using tickets.