Project

General

Profile

Backport(バックポート) #3761

pc_frontend において util.js が二重に出力される場合がある

Added by Chiharu Nakajima almost 5 years ago. Updated almost 5 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2014-04-15
Due date:
% Done:

100%


Description

現象

member/home および member/profile において util.js が二重に出力される現象が発生する
また、plugin においても発生し得る

head 要素内の script 要素を抜粋

... snip ...
<script type="text/javascript" src="/honsen/OpenPNE3/web/js/util.js"></script>
<script type="text/javascript" src="/honsen/OpenPNE3/web/js/jquery.min.js"></script>
<script type="text/javascript" src="/honsen/OpenPNE3/web/js/jquery-ui.min.js"></script>
<script type="text/javascript" src="/honsen/OpenPNE3/web/js/util.js"></script>

原因

commit:406bb371 の修正により [util.js] が追加されたことにより

apps/pc_frontend/modules/member/config/view.yml にある以下の記述の [util] とは別ものと扱われ、二重に util.js が表示される現象が発生した

homeSuccess:
  javascripts: [jquery.min.js, jquery-ui.min.js, util]
  layout: layoutA

profileSuccess:
  javascripts: [util]
  layout: layoutA

修正内容

homeSuccess, profileSuccess における javascripts: [util] の記述を削除する


Related issues

Copied from OpenPNE 3 - Bug(バグ) #3595: pc_frontend において util.js が二重に出力される場合がある Won't fix(対応せず) 2014-04-15

Associated revisions

Revision b3d972ff (diff)
Added by Chiharu Nakajima almost 5 years ago

(refs #3761, BP from #3595) output util.js once

Revision 8190bae3
Added by Rimpei Ogawa almost 5 years ago

Merge pull request #241 from C-nakajima/t-3761

(refs #3761, BP from #3595) output util.js once

History

#1 Updated by Chiharu Nakajima almost 5 years ago

  • Copied from Bug(バグ) #3595: pc_frontend において util.js が二重に出力される場合がある added

#2 Updated by Chiharu Nakajima almost 5 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.15

#3 Updated by Chiharu Nakajima almost 5 years ago

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

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

#4 Updated by Rimpei Ogawa almost 5 years ago

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

#5 Updated by isao sano almost 5 years ago

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

#6 Updated by isao sano almost 5 years ago

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

#7 Updated by Rimpei Ogawa almost 5 years ago

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

Also available in: Atom PDF