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

Opened 11 years ago

Last modified 11 years ago

#3698 new defect

ktai_page_fh_diaryの処理が冗長

Reported by: imamura623 Owned by: nobody
Priority: minor Milestone:
Component: 指定しない Version: 2.12.x & 2.14.x
Keywords: Cc:

Description

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

バグと言いますか、要望と言いますか・・・

ソースいじくってて気がついたのですが
class ktai_page_fh_diary extends OpenPNE_Action

の中が、えらく冗長な気がします。

// --- リクエスト変数
$target_c_diary_id = $requests['target_c_diary_id'];

$c_diary = db_diary_get_c_diary4id($target_c_diary_id);
まずこれは良いとしよう

そのあと
$target_c_member = k_p_fh_diary_c_member4c_diary_id($target_c_diary_id);

こうきて

$target_c_member_id = $target_c_member['c_member_id'];

こうきて

if (!pne_check_diary_public_flag($target_c_diary_id, $u)) {
ktai_display_error('この' . WORD_DIARY_HALF . 'にはアクセスできません');
}

こう

pne_check_diary_public_flagの中で
$c_diary = db_diary_get_c_diary4id($c_diary_id);
とやっている訳ですよ

また
k_p_fh_diary_c_member4c_diary_id の中で nicknameとかと一緒に公開範囲を取ってきているわけですが

公開範囲は$c_diaryで取れてるのだから、多分

nicknameだけとってくるならc_member_LIGHTとかで良いわけですね
その方がキャッシュされているし


h_homeの次にfh_diaryは良く使われてる筈なので
暇があったら直したいですね(直してください!) 

Change History (1)

comment:1 Changed 11 years ago by imamura623

Keywords: 再現待ち removed
Note: See TracTickets for help on using tickets.