Bug(バグ) #3006
apiKeyがログイン前の状態でも表示されてしまっている
開始日:
2012-04-20
期日:
進捗率:
0%
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)
説明
概要¶
スマートフォンAPI(JSON API)を使用する際に必要となる apiKey がログイン前の状態でも表示されてしまっていて(表示されているのはapiKeyというプロパティ名だけであって実体は表示されていない),
かつOpenPNE3をデフォルトでインストール時に apiKeyが表示されないという問題がある.
Causes (原因)¶
apps/pc_frontend_templates/_layout.phpp
の11行目にて,以下のような判定が行われている為,ログイン前でもapiKey表示され,OpenPNEインストール時はapiKeyが表示されなくなってしまっている.
<?php if (Doctrine::getTable('SnsConfig')->get('enable_jsonapi')): ?>
Way to fix (修正内容)¶
ログイン後に表示させ,かつOpenPNEインストール時はデフォルトでapiKeyを出力させるには以下のような修正が適切であると考える.
diff --git a/apps/pc_frontend/templates/_layout.php b/apps/pc_frontend/templates/_layout.php index e9dc908..951a918 100644 --- a/apps/pc_frontend/templates/_layout.php +++ b/apps/pc_frontend/templates/_layout.php @@ -8,7 +8,7 @@ <?php if (Doctrine::getTable('SnsConfig')->get('customizing_css')): ?> <link rel="stylesheet" type="text/css" href="<?php echo url_for('@customizing_css') ?>" /> <?php endif; ?> -<?php if (Doctrine::getTable('SnsConfig')->get('enable_jsonapi')): ?> +<?php if (opConfig::get('enable_jsonapi') && opToolkit::isSecurePage()): ?> <?php use_helper('Javascript');
関連するチケット
関係しているリビジョン
(fixes #3006) fixed not to show apiKey when user is not logged in and to show apiKey when OpenPNE3 installed.
(fixes #3006) fixed not to show apiKey when user is not logged in and to show apiKey when OpenPNE3 installed on smartphone layouts.
履歴
#1 Shouta Kashiwagi がほぼ12年前に更新
- 説明 を更新 (diff)
- ステータス を New(新規) から Accepted(着手) に変更
#2 Shouta Kashiwagi がほぼ12年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 98f097f624d6e0ccfedcdda47e641f29fbb4ce15 で適用されました。
#3 Yuya Watanabe がほぼ12年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
#4 Shouta Kashiwagi がほぼ12年前に更新
- ステータス を Pending Testing(テスト待ち) から Pending Review(レビュー待ち) に変更
- 進捗率 を 70 から 50 に変更
更新履歴 30e161b2f413a006b784299fff58fad6c4cdd382 で適用されました。
#5 Yuya Watanabe がほぼ12年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
#6 Shinichi Urabe がほぼ10年前に更新
- コピー先 Bug(バグ) #3589: pc_frontend 全画面で呼び出される util.js が必須とする jquery.min.js が呼び出されていない条件がある を追加