Project

General

Profile

Actions

Backport(バックポート) #4386

closed

Bug(バグ) #3485: PC向けUIの未ログイン時に jQuery 等のライブラリがロードされない

PC向けUIの未ログイン時に jQuery 等のライブラリがロードされない

Added by isao sano over 4 years ago. Updated over 3 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2013-11-15
Due date:
% Done:

100%

Estimated time:

Description

Overview (現象)

jquery.min.js などのライブラリが PC 向け UI の未ログイン時には読み込まれない。これにより jQuery 等に依存するプラグインが未ログイン時の画面で動作できない状態となっている。

Causes (原因)

source:apps/pc_frontend/templates/_layout.php@27853bc5 の下記コードの通り、未ログイン時には use_javascript('jquery.min.js') を通過しないためこれらのライブラリのロードが行われない。

<?php if (opConfig::get('enable_jsonapi') && opToolkit::isSecurePage()): ?>
<?php
use_helper('Javascript');

use_javascript('jquery.min.js');
use_javascript('jquery.tmpl.min.js');
use_javascript('jquery.notify.js');
use_javascript('op_notify.js');
$jsonData = array(
  'apiKey' => $sf_user->getMemberApiKey(),
  'apiBase' => app_url_for('api', 'homepage'),
  'baseUrl' => $sf_request->getRelativeUrlRoot().'/',
);

echo javascript_tag('
var openpne = '.json_encode($jsonData).';
');
?>
<?php endif ?>

opConfig::get('enable_jsonapi') && opToolkit::isSecurePage() という条件は API キーの出力を行うか否かを判断するための条件であるが、 9f50bdf66e3cdaaa4372669902995a1558d8cbe8 にて誤って use_javascript をこの if ブロック内に追加してしまったことが原因で当チケットの問題が発生した。

Way to fix (修正内容)

jQuery 等をロードする use_javascriptopConfig::get('enable_jsonapi') && opToolkit::isSecurePage() の if ブロックの外に移動する。

Actions #1

Updated by kaoru n over 4 years ago

  • Target version changed from OpenPNE 3.10.1 to OpenPNE 3.10.x
Actions #2

Updated by kaoru n over 3 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/635
にてプルリクエストしました

Actions #3

Updated by Rimpei Ogawa over 3 years ago

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

Updated by kaoru n over 3 years ago

  • Target version changed from OpenPNE 3.10.x to OpenPNE 3.10.5
Actions #5

Updated by isao sano over 3 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

試験完了しました。
問題ありません。

Actions #6

Updated by kaoru n over 3 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Actions

Also available in: Atom PDF