Project

General

Profile

Bug(バグ) #3006

apiKeyがログイン前の状態でも表示されてしまっている

Added by Shouta Kashiwagi over 7 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2012-04-20
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

概要

スマートフォン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');

Related issues

Related to OpenPNE 3 - Backport(バックポート) #3007: apiKeyがログイン前の状態でも表示されてしまっている Fixed(完了) 2012-04-20
Copied to OpenPNE 3 - Bug(バグ) #3589: pc_frontend 全画面で呼び出される util.js が必須とする jquery.min.js が呼び出されていない条件がある Invalid(無効) 2014-04-09

Associated revisions

Revision 98f097f6 (diff)
Added by Shouta Kashiwagi over 7 years ago

(fixes #3006) fixed not to show apiKey when user is not logged in and to show apiKey when OpenPNE3 installed.

Revision 30e161b2 (diff)
Added by Shouta Kashiwagi over 7 years ago

(fixes #3006) fixed not to show apiKey when user is not logged in and to show apiKey when OpenPNE3 installed on smartphone layouts.

History

#1 Updated by Shouta Kashiwagi over 7 years ago

  • Description updated (diff)
  • Status changed from New(新規) to Accepted(着手)

#2 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 98f097f624d6e0ccfedcdda47e641f29fbb4ce15 で適用されました。

#3 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#4 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
  • % Done changed from 70 to 50

更新履歴 30e161b2f413a006b784299fff58fad6c4cdd382 で適用されました。

#5 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#6 Updated by Shinichi Urabe over 5 years ago

  • Copied to Bug(バグ) #3589: pc_frontend 全画面で呼び出される util.js が必須とする jquery.min.js が呼び出されていない条件がある added

#7 Updated by kaoru n over 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

#8 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8RC1 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF