プロジェクト

全般

プロフィール

Bug(バグ) #4031

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

Youichi Kimura7年以上前に追加. ほぼ4年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2020-01-31
期日:
進捗率:

100%

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

説明

現象

管理画面のガジェット設定 (/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' に置き換える


子チケット

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

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

関係しているリビジョン

リビジョン 83752808 (差分)
Youichi Kimura約4年前に追加

fix reference to non-existence .js file (fixes #4031)

リビジョン 8dd80832
kaoru nほぼ4年前に追加

Merge pull request #574 from nishizoe/t-4031

fix reference to non-existence .js file (fixes #4031)

履歴

#1 Youichi Kimura7年以上前に更新

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

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

#2 kaoru nほぼ7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

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

#3 kaoru n約4年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE 3.10.x に変更

#4 kaoru n約4年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更
  • 担当者Youichi Kimura から kaoru n に変更

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

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

#5 Rimpei Ogawaほぼ4年前に更新

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

#6 isao sanoほぼ4年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

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

#7 kaoru nほぼ4年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

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