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

Opened 12 years ago

Last modified 12 years ago

#1660 new enhancement

エラー遷移の際、極力$_REQUESTを使わず、openpne_forward()の引数で対応するようにする

Reported by: ebihara Owned by: nobody
Priority: minor Milestone:
Component: その他 Version:
Keywords: Cc:

Description

[openpne-dev:77]において、bossatamaさんから以下の指摘があった。

webapp/modules/pc/page/c_enevt_add_confirm.phpにおいて、
        if (!db_commu_is_c_commu_member($target_c_commu_id, $u)) {
            $_REQUEST['target_c_commu_id'] = $target_c_commu_id;
            $_REQUEST['msg'] = "イベント作成をおこなうにはコミュニティ
					に参加する必要があります";
            openpne_forward('pc', 'page', "c_home");
            exit;
のような使われ方をされています。
ところが、openpne_forwardは
openpne_forward(第1,第2,第3,第4)の第4パラメータがあり?、上記の例では
$_REQUESTなどという変数を使用せずに、第4パラメータにセットすべきではない
かと思います。

など、OpenPNE1.x時代の仕様を引きずったままの箇所が散見されるので修正する。

参考:http://groups.google.co.jp/group/openpne-dev/browse_thread/thread/ef26378e801b3b86#37c12f0097922587

Change History (3)

comment:1 Changed 12 years ago by ebihara

Milestone: OpenPNE2.11.2OpenPNE2.12

2.11.2では見送り

comment:2 Changed 12 years ago by kiwa

Milestone: OpenPNE2.12.0
Version: 2.11.x

priorityとmilestoneを通常の未着手要望にあわせました。

comment:3 Changed 12 years ago by kiwa

Priority: majorminor
Note: See TracTickets for help on using tickets.