Ticket #2353 (closed 改善提案: 修正済)

Opened 7 years ago

Last modified 7 years ago

モーダルウィンドウのJavaScriptライブラリを変更

Reported by: pineray Owned by: pineray
Priority: Milestone: EC-CUBE2.13.1
Component: フロント Version: 2.12.5
Keywords: Cc: nanasess
修正済み: yes

Description

現在使用しているライブラリが、長らく更新されていないので、セキュリティフィクスや新機種への対応が困難な状態となっている。

アクティブなライブラリへ変更し、ついでにひとつにまとめたい。

Change History

comment:1 Changed 7 years ago by pineray

  • Owner changed from somebody to pineray
  • 修正済み set
  • Status changed from new to assigned

colorbox (  http://www.jacklmoore.com/colorbox/ ) に変更しました。
r23138 r23139 にて対応しました。

comment:2 follow-up: ↓ 3 Changed 7 years ago by nanasess

  • 修正済み unset

本修正の影響か、お届け先の追加・変更画面で JavaScript? エラーが出てお届け先の追加ができません。

以下のようなエラーが表示されています。

eccube.js:568
TypeError: 'undefined' is not a function (evaluating '$("a.expansion").colorbox()')

comment:3 in reply to: ↑ 2 Changed 7 years ago by pineray

nanasess への返信

本修正の影響か、お届け先の追加・変更画面で JavaScript? エラーが出てお届け先の追加ができません。

以下のようなエラーが表示されています。

eccube.js:568
TypeError: 'undefined' is not a function (evaluating '$("a.expansion").colorbox()')

r23153 にて対応しました。

comment:4 follow-up: ↓ 5 Changed 7 years ago by nanasess

管理画面のログイン画面でも、 $("a.expansion").colorbox() が undefined なエラーになりますね。

すべてのテンプレートで colorbox をロードするのではなく、以下のような修正でいかがでしょうか?

  • html/js/eccube.js

    a b  
    571571        }); 
    572572 
    573573        // モーダルウィンドウ 
    574         $("a.expansion").colorbox(); 
     574        if ($('a.expansion').length) { 
     575            $('a.expansion').colorbox(); 
     576        } 
    575577    }); 
    576578})(window); 

comment:5 in reply to: ↑ 4 Changed 7 years ago by h_yoshimoto

  • Cc nanasess added
  • 修正済み set

nanasess への返信

r23264 にてコミットしました。
よろしくお願いいたします。

comment:6 Changed 7 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.