Project

General

Profile

Actions

Bug(バグ) #4031

closed

存在しない /js/.js を読み込もうとして404エラーが発生している

Added by Youichi Kimura about 8 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2020-01-31
Due date:
% Done:

100%

Estimated time:
(Total: 0:00 h)
3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Unknown (未調査)

Description

現象

管理画面のガジェット設定 (/pc_backend.php/design/gadget) を開くと下記のような script 要素が記載されており、/js/.js にはファイルが存在しないため 404 エラーが発生する。

<script type="text/javascript" src="/js/.js"></script>

原因

下記のコードで op_jquery_url は存在しない設定であるため null を返すことが原因。

source:lib/helper/opJavascriptHelper.php@43190aa1#L28:

sfContext::getInstance()->getResponse()->addJavascript(sfConfig::get('op_jquery_url'));

op_jquery_url#2617 で prototype.js から jQuery に移行する際に使用していたものであり、この設定は 207729a0 で削除されている。

修正内容

sfConfig::get('op_jquery_url')'jquery.min.js' に置き換える


Subtasks 2 (0 open2 closed)

Backport(バックポート) #4363: 存在しない /js/.js を読み込もうとして404エラーが発生しているFixed(完了)kaoru n2020-01-31

Actions
Backport(バックポート) #4381: 存在しない /js/.js を読み込もうとして404エラーが発生しているFixed(完了)isao sano2020-01-31

Actions
Actions #1

Updated by Youichi Kimura about 8 years ago

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

下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/380

Actions #2

Updated by kaoru n over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

Actions #3

Updated by kaoru n almost 5 years ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x
Actions #4

Updated by kaoru n almost 5 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
  • Assignee changed from Youichi Kimura to kaoru n

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

https://github.com/openpne/OpenPNE3/pull/380
については、old-master 向けであるためクローズします。

Actions #5

Updated by Rimpei Ogawa over 4 years ago

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

Updated by isao sano over 4 years ago

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

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

Actions #7

Updated by kaoru n over 4 years ago

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

マージしました

Actions

Also available in: Atom PDF