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

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#2762 closed defect (fixed)

携帯版ポータル使用設定を「使用しない」にした場合に、携帯版ポータルページに直接アクセスしても携帯版ログイン画面にリダイレクトされない

Reported by: yuki Owned by: masabon
Priority: minor Milestone: OpenPNE2.12.2
Component: 指定しない Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.13.2 Cc:

Description (last modified by ebihara)

■現象

携帯版ポータル使用設定が「使用しない」である場合に、携帯版ポータルページに直接アクセスしても携帯版ログイン画面にリダイレクトされない。

■原因

portal_page_portal_user_top_ktai にて、携帯版ポータル使用設定が「使用しない」であるかどうかのチェックをおこなっていなかった。

■修正内容

portal_page_portal_user_top_ktai にて、携帯版ポータル使用設定が「使用しない」であるかどうかのチェックをおこない、「使用しない」である場合は携帯版ログイン画面にリダイレクトするようにした。

■関連情報

報告内容

OPENPNE_URL/?m=ktai&a=page_portal_user_top_ktai []

というような、ポータルのURLをそのままブックマークしてしまうと、「使用しない」設定になっていても、このURLに直接飛ぶとポータルが表示されてしまいます。 

Change History (8)

comment:1 Changed 12 years ago by yuki

以下修正で改善されます。

Index: portal_user_top_ktai.php
===================================================================
--- portal_user_top_ktai.php    (リビジョン 1612)
+++ portal_user_top_ktai.php    (作業コピー)
@@ -9,6 +9,11 @@

     function execute($requests)
     {
+        $use_portal = db_portal_config('USE_PORTAL_KTAI');
+        if (!$use_portal['value']) {
+            openpne_redirect('ktai');
+        }
+
         $this->set('SNS_NAME', SNS_NAME);
         $this->set('IS_CLOSED_SNS', IS_CLOSED_SNS);

comment:2 Changed 12 years ago by ogawa

Keywords: OpenPNE2.13.2 added
Milestone: OpenPNE2.12.2
Priority: minormajor
Type: enhancementdefect
Version: 2.12.x & 2.13.x

defect に変更します。

comment:3 Changed 12 years ago by ebihara

Priority: majorminor

comment:4 Changed 12 years ago by masabon

Owner: changed from nobody to masabon
Status: newassigned

comment:5 Changed 12 years ago by masabon

Keywords: 確認中 added

以下のリビジョンで修正しました。

comment:6 Changed 12 years ago by ebihara

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

セットアップ直後では portal_config に USE_PORTAL_KTAI 存在しないため、

$use_portal = db_portal_config('USE_PORTAL_KTAI');

は null となりますが、

if (!$use_portal['value']) {

のようにアクセスすることは PHP 的には OK のようで、 Notice が出ることもありませんでした(空配列の場合はNG)。

is_null() などを使って値をチェックするのがよりベターだとは思いますが、問題はないと思います。

comment:7 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

確認しました。問題ありません。

comment:8 Changed 12 years ago by ebihara

Description: modified (diff)

フォーマットに従って Description 書き換え

Note: See TracTickets for help on using tickets.