From 2520fff77141ac7479161e6a5959dda3f3b52094 Mon Sep 17 00:00:00 2001 From: Jakob Scheid Date: Sat, 8 Aug 2026 13:53:59 +0200 Subject: [PATCH] style: remove unnecessary function whitespace Remove the unnecessary whitespace between the function name or the 'function' keyword and the opening parenthesis. --- src/App.vue | 2 +- src/config/env.js | 2 +- .../colorScheme/components/ColorSchemeButton.vue | 4 ++-- .../components/__tests__/ColorSchemeButton.test.js | 2 +- .../i18n/components/LanguageSwitchButton.vue | 8 ++++---- .../__tests__/LanguageSwitchButton.test.js | 2 +- src/features/i18n/utils/navigatorLanguage.js | 2 +- src/features/legal/services/fetchLegalContent.js | 4 ++-- src/features/search/components/Searchbar.vue | 2 +- src/features/settings/components/Selection.vue | 2 +- src/features/settings/components/SettingsInput.vue | 2 +- .../settings/components/SettingsPageLink.vue | 2 +- src/features/settings/components/Switch.vue | 2 +- .../settings/components/__tests__/Selection.test.js | 12 ++++++------ .../settings/components/__tests__/Switch.test.js | 2 +- src/features/settings/composables/useSettings.js | 6 +++--- src/features/settings/composables/useSettingsPage.js | 4 ++-- src/features/settings/stores/settingsStore.js | 8 ++++---- src/features/settings/utils/getSetting.js | 6 +++--- src/features/settings/utils/settingsPage.js | 2 +- src/features/settings/utils/settingsValidator.js | 10 +++++----- src/features/settings/views/SettingsView.vue | 4 ++-- .../settings/views/__tests__/SettingsView.test.js | 2 +- src/features/sidebar/components/Sidebar.vue | 2 +- .../sidebar/components/SidebarExpandButton.vue | 2 +- src/i18n.js | 4 ++-- src/layouts/LeftSidebarLayout.vue | 2 +- src/router/index.js | 2 +- src/test-utils/mountComponent.js | 2 +- src/utils/cssDimensions.js | 2 +- src/views/SearchView.vue | 2 +- 31 files changed, 55 insertions(+), 55 deletions(-) diff --git a/src/App.vue b/src/App.vue index 577225d..f6cf92a 100644 --- a/src/App.vue +++ b/src/App.vue @@ -30,7 +30,7 @@ const route = useRoute(); const { getSetting } = useSettings(); const colorScheme = computed(() => getSetting(['appearance', 'colorScheme'])); -const updateColorScheme = function updateColorScheme (newColorScheme) { +const updateColorScheme = function updateColorScheme(newColorScheme) { document.body.style.setProperty(colorScheme, { auto: 'normal', dark: 'dark', diff --git a/src/config/env.js b/src/config/env.js index c8418f0..0958176 100644 --- a/src/config/env.js +++ b/src/config/env.js @@ -14,7 +14,7 @@ See the License for the specific language governing permissions and limitations under the License. */ -export const getAssetsUrl = function getAssetsUrl () { +export const getAssetsUrl = function getAssetsUrl() { return new URL( `${import.meta.env.VITE_ASSETS_BUILD_ID}/`, import.meta.env.VITE_ASSETS_BASE_URL diff --git a/src/features/colorScheme/components/ColorSchemeButton.vue b/src/features/colorScheme/components/ColorSchemeButton.vue index 7cc25d2..93c8fd2 100644 --- a/src/features/colorScheme/components/ColorSchemeButton.vue +++ b/src/features/colorScheme/components/ColorSchemeButton.vue @@ -38,11 +38,11 @@ const colorSchemeIconMapper = { 'auto': 'circle-black-white' }; -const getTooltipTranslation = function (colorScheme) { +const getTooltipTranslation = function(colorScheme) { return t(`preferences.colorScheme.switch.${colorSchemeNextMapper[colorScheme]}`); }; -const setColorScheme = function setColorScheme (newColorScheme) { +const setColorScheme = function setColorScheme(newColorScheme) { setSetting(['appearance', 'colorScheme'], newColorScheme); }; diff --git a/src/features/colorScheme/components/__tests__/ColorSchemeButton.test.js b/src/features/colorScheme/components/__tests__/ColorSchemeButton.test.js index 3273133..4565f1e 100644 --- a/src/features/colorScheme/components/__tests__/ColorSchemeButton.test.js +++ b/src/features/colorScheme/components/__tests__/ColorSchemeButton.test.js @@ -23,7 +23,7 @@ const { getSetting } = useSettings(); const colorSchemeSettingPath = ['appearance', 'colorScheme']; -const getWrapper = function getWrapper () { +const getWrapper = function getWrapper() { return mountComponent(ColorSchemeButton); }; diff --git a/src/features/i18n/components/LanguageSwitchButton.vue b/src/features/i18n/components/LanguageSwitchButton.vue index e856c22..eee00a1 100644 --- a/src/features/i18n/components/LanguageSwitchButton.vue +++ b/src/features/i18n/components/LanguageSwitchButton.vue @@ -29,19 +29,19 @@ const { setSetting, getSetting } = useSettings(); const isOpen = ref(false); const languageDropdown = ref(null); -const selectLanguage = function selectLanguage (code) { +const selectLanguage = function selectLanguage(code) { setSetting(LANGUAGE_PATH, code); close(); }; watch(() => getSetting(LANGUAGE_PATH), setLocale); -const close = function () { +const close = function() { document.removeEventListener('click', closeWrapperOnClickOutside); isOpen.value = false; }; -const closeWrapperOnClickOutside = function (e) { +const closeWrapperOnClickOutside = function(e) { if (languageDropdown.value) { if (!languageDropdown.value.contains(e.target)) { close(); @@ -49,7 +49,7 @@ const closeWrapperOnClickOutside = function (e) { }; }; -const open = function () { +const open = function() { if (!isOpen.value) { isOpen.value = true; setTimeout(() => { diff --git a/src/features/i18n/components/__tests__/LanguageSwitchButton.test.js b/src/features/i18n/components/__tests__/LanguageSwitchButton.test.js index 17f0753..06fdee9 100644 --- a/src/features/i18n/components/__tests__/LanguageSwitchButton.test.js +++ b/src/features/i18n/components/__tests__/LanguageSwitchButton.test.js @@ -20,7 +20,7 @@ import { mountComponent } from '@/test-utils/mountComponent.js'; import { expect, test, describe } from 'vitest'; import LanguageSwitchButton from '../LanguageSwitchButton.vue'; -const getWrapper = function getWrapper () { +const getWrapper = function getWrapper() { return mountComponent(LanguageSwitchButton); }; diff --git a/src/features/i18n/utils/navigatorLanguage.js b/src/features/i18n/utils/navigatorLanguage.js index 7b6f6d6..f42ae24 100644 --- a/src/features/i18n/utils/navigatorLanguage.js +++ b/src/features/i18n/utils/navigatorLanguage.js @@ -14,7 +14,7 @@ See the License for the specific language governing permissions and limitations under the License. */ -export const getNavigatorLanguage = function getNavigatorLanguage () { +export const getNavigatorLanguage = function getNavigatorLanguage() { const locale = new Intl.Locale(navigator.language); return locale.language; }; diff --git a/src/features/legal/services/fetchLegalContent.js b/src/features/legal/services/fetchLegalContent.js index 663ce9d..0b65727 100644 --- a/src/features/legal/services/fetchLegalContent.js +++ b/src/features/legal/services/fetchLegalContent.js @@ -17,7 +17,7 @@ limitations under the License. import { FetchError } from '../errors'; import { getAssetsUrl } from '@/config/env'; -const fetchHtml = async function fetchHtml (url, options) { +const fetchHtml = async function fetchHtml(url, options) { let response; try { response = await fetch(url, options); @@ -29,7 +29,7 @@ const fetchHtml = async function fetchHtml (url, options) { else throw new FetchError(); }; -export const getLegalNotice = async function getLegalNotice () { +export const getLegalNotice = async function getLegalNotice() { const legalNoticeUrl = new URL('legal/legal_notice.html', getAssetsUrl()).href; const html = await fetchHtml(legalNoticeUrl); diff --git a/src/features/search/components/Searchbar.vue b/src/features/search/components/Searchbar.vue index 78e348d..18875f0 100644 --- a/src/features/search/components/Searchbar.vue +++ b/src/features/search/components/Searchbar.vue @@ -27,7 +27,7 @@ const { t } = useI18n(); const router = useRouter(); const props = defineProps(['autoSubmit']); -const submitSearch = function () { +const submitSearch = function() { if (props.autoSubmit !== undefined) { router.push({ name: 'searchResults', diff --git a/src/features/settings/components/Selection.vue b/src/features/settings/components/Selection.vue index 1512493..ef94343 100644 --- a/src/features/settings/components/Selection.vue +++ b/src/features/settings/components/Selection.vue @@ -22,7 +22,7 @@ import { useI18n } from 'vue-i18n'; const { t } = useI18n(); -const normalizeSelectedValue = function normalizeSelectedValue (value) { +const normalizeSelectedValue = function normalizeSelectedValue(value) { if (props.setting.allowMultiple) { if (Array.isArray(value)) { return value; diff --git a/src/features/settings/components/SettingsInput.vue b/src/features/settings/components/SettingsInput.vue index 958e648..70f28cb 100644 --- a/src/features/settings/components/SettingsInput.vue +++ b/src/features/settings/components/SettingsInput.vue @@ -65,7 +65,7 @@ watch(inputModel, () => { doneButtonInitial.value = false; }); -const apply = function apply () { +const apply = function apply() { store.set(props.path, inputModel.value); }; diff --git a/src/features/settings/components/SettingsPageLink.vue b/src/features/settings/components/SettingsPageLink.vue index ea3878c..8beca7d 100644 --- a/src/features/settings/components/SettingsPageLink.vue +++ b/src/features/settings/components/SettingsPageLink.vue @@ -33,7 +33,7 @@ const props = defineProps({ } }); -const followLink = function followLink () { +const followLink = function followLink() { goToSettingsPage(props.target); }; diff --git a/src/features/settings/components/Switch.vue b/src/features/settings/components/Switch.vue index 88ecce6..92b7499 100644 --- a/src/features/settings/components/Switch.vue +++ b/src/features/settings/components/Switch.vue @@ -42,7 +42,7 @@ watch(store, (newStore) => { // set value after registering watcher to avoid immediate value change enabled.value = store.get(props.path) ?? props.setting.default; -const toggle = function toggle () { +const toggle = function toggle() { enabled.value = !enabled.value; store.set(props.path, enabled.value); }; diff --git a/src/features/settings/components/__tests__/Selection.test.js b/src/features/settings/components/__tests__/Selection.test.js index 9696965..924c223 100644 --- a/src/features/settings/components/__tests__/Selection.test.js +++ b/src/features/settings/components/__tests__/Selection.test.js @@ -20,7 +20,7 @@ import { useSettingsStore } from '../../stores/settingsStore.js'; import { expect, describe, test } from 'vitest'; import { nextTick } from 'vue'; -const getWrapper = function getWrapper ({ +const getWrapper = function getWrapper({ i18n = 'selection', options = [], allowMultiple = false, @@ -43,7 +43,7 @@ const getWrapper = function getWrapper ({ }, translations, [], piniaOptions); }; -const getChecked = function getChecked (inputs) { +const getChecked = function getChecked(inputs) { return inputs.map((input) => input.element.matches(':checked')); }; @@ -199,20 +199,20 @@ describe('Selection', () => { }); test('stores multiple values', async () => { - const setValue = async function setValue (input, value) { + const setValue = async function setValue(input, value) { input.setValue(value); await nextTick(); }; - const check = function check (input) { + const check = function check(input) { return setValue(input, true); }; - const uncheck = function uncheck (input) { + const uncheck = function uncheck(input) { return setValue(input, false); }; - const expectSelectionValue = function expectSelectionValue (value, options = exampleOptions) { + const expectSelectionValue = function expectSelectionValue(value, options = exampleOptions) { value.sort(); expect(store.get('selection').sort()).toStrictEqual(value); const checkedInputs = getChecked(inputs); diff --git a/src/features/settings/components/__tests__/Switch.test.js b/src/features/settings/components/__tests__/Switch.test.js index ba6e607..80b2f53 100644 --- a/src/features/settings/components/__tests__/Switch.test.js +++ b/src/features/settings/components/__tests__/Switch.test.js @@ -20,7 +20,7 @@ import { useSettingsStore } from '../../stores/settingsStore.js'; import { expect, describe, test } from 'vitest'; import { nextTick } from 'vue'; -const getWrapper = function getWrapper ({ +const getWrapper = function getWrapper({ i18n = 'switch1', defaultValue = undefined, translations = {}, diff --git a/src/features/settings/composables/useSettings.js b/src/features/settings/composables/useSettings.js index c42ef99..33b7406 100644 --- a/src/features/settings/composables/useSettings.js +++ b/src/features/settings/composables/useSettings.js @@ -21,13 +21,13 @@ import { getSettingRecursively } from '../utils/getSetting'; /** * Provides access to the stored settings. */ -export const useSettings = function useSettings () { +export const useSettings = function useSettings() { /** * Returns the value of a specific setting. * @param {string[]} key - The settings key. * @return The value of the setting. */ - const getSetting = function getSetting (key) { + const getSetting = function getSetting(key) { const settingsStore = useSettingsStore(); return ( @@ -43,7 +43,7 @@ export const useSettings = function useSettings () { * @param {string[]} key - The setting key. * @param value - The new value for the setting. */ - const setSetting = function setSetting (key, value) { + const setSetting = function setSetting(key, value) { const settingsStore = useSettingsStore(); settingsStore.set(key.join('.'), value); }; diff --git a/src/features/settings/composables/useSettingsPage.js b/src/features/settings/composables/useSettingsPage.js index bd45729..1622dca 100644 --- a/src/features/settings/composables/useSettingsPage.js +++ b/src/features/settings/composables/useSettingsPage.js @@ -17,10 +17,10 @@ limitations under the License. import { getSettingsPagePathSegments } from '../utils/settingsPage'; import { useRouter } from 'vue-router'; -export const useSettingsPage = function useSettingsPage () { +export const useSettingsPage = function useSettingsPage() { const router = useRouter(); - const goToSettingsPage = function goToSettingsPage (pagePath) { + const goToSettingsPage = function goToSettingsPage(pagePath) { return router.push({ name: 'settings', params: { diff --git a/src/features/settings/stores/settingsStore.js b/src/features/settings/stores/settingsStore.js index 5f5133f..4852c1c 100644 --- a/src/features/settings/stores/settingsStore.js +++ b/src/features/settings/stores/settingsStore.js @@ -20,19 +20,19 @@ import { reactive } from 'vue'; export const useSettingsStore = defineStore('settings', () => { const settings = reactive({}); - const set = function set (key, value) { + const set = function set(key, value) { settings[key] = value; }; - const get = function get (key) { + const get = function get(key) { return settings[key]; }; - const remove = function remove (key) { + const remove = function remove(key) { delete settings[key]; }; - const clear = function clear () { + const clear = function clear() { Object.keys(settings).forEach(key => delete settings[key]); }; diff --git a/src/features/settings/utils/getSetting.js b/src/features/settings/utils/getSetting.js index cc1961b..e7ffad6 100644 --- a/src/features/settings/utils/getSetting.js +++ b/src/features/settings/utils/getSetting.js @@ -14,12 +14,12 @@ See the License for the specific language governing permissions and limitations under the License. */ -export const getSetting = function getSetting (name, settings) { +export const getSetting = function getSetting(name, settings) { return settings.filter((setting) => setting.name === name)[0]; }; -export const getSettingRecursively = function getSettingRecursively (sectionPath, settings) { - const oneSettingsLevel = function oneSettingsLevel (cursor, acc) { +export const getSettingRecursively = function getSettingRecursively(sectionPath, settings) { + const oneSettingsLevel = function oneSettingsLevel(cursor, acc) { const setting = getSetting(cursor[0], acc.content); if (!setting) return; diff --git a/src/features/settings/utils/settingsPage.js b/src/features/settings/utils/settingsPage.js index a6245ea..64697b1 100644 --- a/src/features/settings/utils/settingsPage.js +++ b/src/features/settings/utils/settingsPage.js @@ -19,7 +19,7 @@ limitations under the License. * @param {string} path - The settings page path. * @returns {Array} The settings page path segments. */ -export const getSettingsPagePathSegments = function getSettingsPagePathSegments (path) { +export const getSettingsPagePathSegments = function getSettingsPagePathSegments(path) { if (typeof path !== 'string') return []; return path .replace(/^\.+|\.+$/g, '') diff --git a/src/features/settings/utils/settingsValidator.js b/src/features/settings/utils/settingsValidator.js index 159de5a..e284238 100644 --- a/src/features/settings/utils/settingsValidator.js +++ b/src/features/settings/utils/settingsValidator.js @@ -16,13 +16,13 @@ limitations under the License. const VALID_TYPES = ['bool', 'number', 'string', 'selection', 'section']; -export const assertString = function assertString (value, path) { +export const assertString = function assertString(value, path) { if (typeof value !== 'string' || value.trim() === '') { throw new Error(`[settings] "${path}" must be a non-empty string`); } }; -export const assertType = function assertType (value, path) { +export const assertType = function assertType(value, path) { if (!VALID_TYPES.includes(value)) { throw new Error( `[settings] "${path}" has invalid type "${value}". Must be one of: ${VALID_TYPES.join(', ')}` @@ -30,7 +30,7 @@ export const assertType = function assertType (value, path) { } }; -export const validateSelectionOptions = function validateSelectionOptions (options, path) { +export const validateSelectionOptions = function validateSelectionOptions(options, path) { if (!Array.isArray(options) || options.length === 0) { throw new Error(`[settings] "${path}.options" must be a non-empty array`); } @@ -40,7 +40,7 @@ export const validateSelectionOptions = function validateSelectionOptions (optio }); }; -export const validateEntry = function validateEntry (entry, path) { +export const validateEntry = function validateEntry(entry, path) { assertType(entry.type, `${path}.type`); assertString(entry.name, `${path}.name`); @@ -98,7 +98,7 @@ export const validateEntry = function validateEntry (entry, path) { } }; -export const validateFirstLevelSection = function validateFirstLevelSection (section, path) { +export const validateFirstLevelSection = function validateFirstLevelSection(section, path) { assertString(section.name); assertString(section.i18n); diff --git a/src/features/settings/views/SettingsView.vue b/src/features/settings/views/SettingsView.vue index c4e208a..f9ba22f 100644 --- a/src/features/settings/views/SettingsView.vue +++ b/src/features/settings/views/SettingsView.vue @@ -39,7 +39,7 @@ const activePath = computed(() => { }); const activeSection = ref(null); -const updateSettings = async function updateSettings () { +const updateSettings = async function updateSettings() { const setting = getSettingRecursively(activePath.value, settings); if (setting) { activeSection.value = setting; @@ -50,7 +50,7 @@ const updateSettings = async function updateSettings () { watch(route, (newRoute) => updateSettings(newRoute)); onMounted(updateSettings); -const toggleSidebarIfSmallScreen = function toggleSidebarIfSmallScreen (toggleSidebarFunction) { +const toggleSidebarIfSmallScreen = function toggleSidebarIfSmallScreen(toggleSidebarFunction) { if (matchMedia('(max-width: 48rem)').matches) toggleSidebarFunction(); }; diff --git a/src/features/settings/views/__tests__/SettingsView.test.js b/src/features/settings/views/__tests__/SettingsView.test.js index 8b5dc20..efaf43a 100644 --- a/src/features/settings/views/__tests__/SettingsView.test.js +++ b/src/features/settings/views/__tests__/SettingsView.test.js @@ -211,7 +211,7 @@ vi.mock('vue-router', async (importOriginal) => { }; }); -const getWrapper = function getWrapper ({ translations = {} } = {}) { +const getWrapper = function getWrapper({ translations = {} } = {}) { return mountComponent(SettingsView, {}, translations, [], {}); }; diff --git a/src/features/sidebar/components/Sidebar.vue b/src/features/sidebar/components/Sidebar.vue index ab94113..c277dfd 100644 --- a/src/features/sidebar/components/Sidebar.vue +++ b/src/features/sidebar/components/Sidebar.vue @@ -26,7 +26,7 @@ const props = defineProps({ }); const emit = defineEmits(['toggleExpanded']); -const toggleSidebar = function toggleSidebar () { +const toggleSidebar = function toggleSidebar() { emit('toggleExpanded'); }; diff --git a/src/features/sidebar/components/SidebarExpandButton.vue b/src/features/sidebar/components/SidebarExpandButton.vue index 14a0314..dad4977 100644 --- a/src/features/sidebar/components/SidebarExpandButton.vue +++ b/src/features/sidebar/components/SidebarExpandButton.vue @@ -19,7 +19,7 @@ import Icon from '@/features/icons/components/Icon.vue'; const emit = defineEmits(['click']); -const click = function click (event) { +const click = function click(event) { emit('click', event); }; diff --git a/src/i18n.js b/src/i18n.js index 5a14219..efaa0e9 100644 --- a/src/i18n.js +++ b/src/i18n.js @@ -40,7 +40,7 @@ export const i18n = createI18n({ const loadedLanguages = new Set(); -export async function loadLanguage (locale) { +export async function loadLanguage(locale) { if (!SUPPORTED_LANGUAGES.includes(locale)) { locale = fallbackLocale; } @@ -58,7 +58,7 @@ export async function loadLanguage (locale) { loadedLanguages.add(locale); }; -export const setLocale = async function setLocale (locale) { +export const setLocale = async function setLocale(locale) { await loadLanguage(locale); document.documentElement.lang = locale; document.documentElement.dir = LANGUAGES_RTL.includes(locale) ? 'rtl' : 'ltr'; diff --git a/src/layouts/LeftSidebarLayout.vue b/src/layouts/LeftSidebarLayout.vue index 0471174..2cb189f 100644 --- a/src/layouts/LeftSidebarLayout.vue +++ b/src/layouts/LeftSidebarLayout.vue @@ -29,7 +29,7 @@ const props = defineProps({ const expanded = ref(props.expanded); -const toggleSidebar = function toggleSidebar () { +const toggleSidebar = function toggleSidebar() { expanded.value = !expanded.value; }; diff --git a/src/router/index.js b/src/router/index.js index 99e96c4..fe46dbb 100644 --- a/src/router/index.js +++ b/src/router/index.js @@ -68,7 +68,7 @@ const router = createRouter({ routes }); -export const updatePageTitle = function updatePageTitle (route) { +export const updatePageTitle = function updatePageTitle(route) { const title = typeof route.meta.title === 'function' ? route.meta.title(route) diff --git a/src/test-utils/mountComponent.js b/src/test-utils/mountComponent.js index 8c997d8..358eefa 100644 --- a/src/test-utils/mountComponent.js +++ b/src/test-utils/mountComponent.js @@ -20,7 +20,7 @@ import { mount } from '@vue/test-utils'; import { createI18n } from 'vue-i18n'; import { createMemoryHistory, createRouter } from 'vue-router'; -export const mountComponent = function mountComponent (component, options = {}, i18nMessages = {}, routes = [], piniaOptions = {}) { +export const mountComponent = function mountComponent(component, options = {}, i18nMessages = {}, routes = [], piniaOptions = {}) { const { setupStores, ...testingPiniaOptions diff --git a/src/utils/cssDimensions.js b/src/utils/cssDimensions.js index 4dc5922..fce7184 100644 --- a/src/utils/cssDimensions.js +++ b/src/utils/cssDimensions.js @@ -14,7 +14,7 @@ See the License for the specific language governing permissions and limitations under the License. */ -export const ensureUnit = function (d) { +export const ensureUnit = function(d) { if (!d) d = 0; // ensure that it is a string diff --git a/src/views/SearchView.vue b/src/views/SearchView.vue index 7a35aa5..7812125 100644 --- a/src/views/SearchView.vue +++ b/src/views/SearchView.vue @@ -26,7 +26,7 @@ const router = useRouter(); const searchQuery = ref(''); -const submitSearch = function () { +const submitSearch = function() { };