JS Error deleting currency – and fix

  • Resolved Ben Huson

    (@husobj)


    6 years, 12 months ago

    Am using version 4.2.9

    Trying to delete a currency triggers a JavaScript error.
    I have tracked it down to a missing bracket.

    The fix is in res/js/multi-currency.js line 326:

    elem = $( '#currency-lang-table a.otgs-ico-yes[data-language="'+lang+'"]:not([data-currency="'+currency+'"]' );

    The :not() requires a closing bracket after the square bracket.

    Kind regards

    • This topic was modified 6 years, 12 months ago by Ben Huson.
    • This topic was modified 6 years, 12 months ago by Ben Huson.
Viewing 2 replies - 1 through 2 (of 2 total)

  • Thank you for reporting this.
    I will check it now and get back to you asap.

    I will forward the issue to our developers.
    Yes there appears to be a missing bracked where you said.
    The affected file is not the minified one, so the impact should be only when using scriptdebug definition in WordPress.
    Even with that, I am not entirely sure how I can trigger the error because of that bracket as I tried it and I was able to remove the additional currency without issues.

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘JS Error deleting currency – and fix’ is closed to new replies.

Tags