プロジェクト

全般

プロフィール

Backport(バックポート) #3007

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

Shouta Kashiwagi約12年前に追加. 約12年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
対象バージョン:
開始日:
2012-04-20
期日:
進捗率:

100%


説明

概要

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

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #3006: apiKeyがログイン前の状態でも表示されてしまっている Won't fix(対応せず) 2012-04-20

関係しているリビジョン

リビジョン 79cc2c74 (差分)
Shouta Kashiwagi約12年前に追加

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

リビジョン 20b2c367 (差分)
Shouta Kashiwagi約12年前に追加

(fixes #3007, BP from #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年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#2 Shouta Kashiwagi約12年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 79cc2c7400fbc05bf9f7f00c70c1e71d48b6707e で適用されました。

#3 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#4 Shouta Kashiwagi約12年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率70 から 50 に変更

更新履歴 20b2c3679f40c1165211e51d738db8a954ac0fc2 で適用されました。

#5 Shouta Kashiwagi約12年前に更新

スマートフォン版でも不備があったので #note-4 のコミットで適用しました.

#6 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#7 Yuma Sakata約12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

他の形式にエクスポート: Atom PDF