From adcc8a2ca318cf83c50ff30dcf5e7956bb6c34f6 Mon Sep 17 00:00:00 2001 From: Colin Rotherham Date: Wed, 9 Nov 2022 14:59:28 +0000 Subject: [PATCH] Code fixes for I18n, Accordion, Character count As identified by the `tsc` TypeScript compiler --- src/govuk/components/accordion/accordion.mjs | 1 - src/govuk/components/character-count/character-count.test.js | 2 +- src/govuk/i18n.mjs | 4 +--- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/govuk/components/accordion/accordion.mjs b/src/govuk/components/accordion/accordion.mjs index aacf984ceb..d50c7d7f3c 100644 --- a/src/govuk/components/accordion/accordion.mjs +++ b/src/govuk/components/accordion/accordion.mjs @@ -37,7 +37,6 @@ var TRANSLATIONS_DEFAULT = { function Accordion ($module, config) { this.$module = $module this.$sections = $module.querySelectorAll('.govuk-accordion__section') - this.$showAllButton = '' this.browserSupportsSessionStorage = helper.checkForSessionStorage() var defaultConfig = { diff --git a/src/govuk/components/character-count/character-count.test.js b/src/govuk/components/character-count/character-count.test.js index fd1a867b19..b42d6f7a71 100644 --- a/src/govuk/components/character-count/character-count.test.js +++ b/src/govuk/components/character-count/character-count.test.js @@ -663,7 +663,7 @@ describe('Character count', () => { await renderAndInitialise(page, 'character-count', { nunjucksParams: examples.default, - config: { + javascriptConfig: { // Override maxlength to 10 maxlength: 10 }, diff --git a/src/govuk/i18n.mjs b/src/govuk/i18n.mjs index 9e74cb1242..ea22598a56 100644 --- a/src/govuk/i18n.mjs +++ b/src/govuk/i18n.mjs @@ -9,13 +9,11 @@ * @param {string} config.locale - An overriding locale for the PluralRules functionality. */ export function I18n (translations, config) { - config = config || {} - // Make list of translations available throughout function this.translations = translations || {} // The locale to use for PluralRules and NumberFormat - this.locale = config.locale || document.documentElement.lang || 'en' + this.locale = (config && config.locale) || document.documentElement.lang || 'en' } /**