Bug（バグ） #4031
存在しない /js/.js を読み込もうとして404エラーが発生している
Start date:
2020-01-31
Due date:
% Done:
100%
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
History
#1
Updated by Youichi Kimura about 7 years ago
Updated by - Status changed from New（新規） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/380
#2
Updated by kaoru n over 6 years ago
Updated by - Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
- Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
対象バージョン変更により修正内容の確認が必要であるため差し戻します。
#3
Updated by kaoru n almost 4 years ago
- Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x
#4
Updated by kaoru n almost 4 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 向けであるためクローズします。
#5
Updated by Rimpei Ogawa over 3 years ago
Updated by - Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
#6
Updated by isao sano over 3 years ago
Updated by - Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
- % Done changed from 70 to 80
試験完了しました。
問題ありません。
#7
Updated by kaoru n over 3 years ago
- Status changed from Pending Merge（マージ待ち） to Fixed（完了）
- % Done changed from 80 to 100
マージしました