Enhance ESLint Configuration and Improve Code Consistency

- Updated ESLint configuration in `.eslintrc.cjs` to enforce error handling for unused variables, allowing for better code quality and maintainability.
- Added `clone-deep` import in `OpenForm.vue` to facilitate deep cloning of objects, enhancing functionality.
- Refactored variable destructuring in `ColumnCondition.vue` and `useAlert.js` to improve readability by using underscore for ignored parameters.
- Modified `pinia-history.js` to use underscore for unused parameters in the `get` and `set` functions, aligning with the new ESLint rules.

These changes aim to improve code quality, maintainability, and consistency across the codebase.
This commit is contained in:
Julien Nahum 2025-03-27 11:07:39 +01:00
parent 0d9c658638
commit 3dd3147b19
5 changed files with 8 additions and 7 deletions

View File

@ -14,5 +14,7 @@ module.exports = {
"prefer-rest-params": "off",
"vue/valid-template-root": "off",
"no-undef": "off",
"no-unused-vars": ["error", { "argsIgnorePattern": "^_" }],
"@typescript-eslint/no-unused-vars": ["error", { "argsIgnorePattern": "^_" }]
},
};

View File

@ -134,6 +134,7 @@ import CachedDefaultTheme from "~/lib/forms/themes/CachedDefaultTheme.js"
import FormTimer from './FormTimer.vue'
import { storeToRefs } from 'pinia'
import { FormMode, createFormModeStrategy } from "~/lib/forms/FormModeStrategy.js"
import clonedeep from 'clone-deep'
export default {
name: 'OpenForm',

View File

@ -104,7 +104,7 @@ export default {
},
operators() {
return Object.entries(this.available_filters[this.property.type].comparators)
.filter(([filterKey, value]) => this.customValidation || (!this.customValidation && !value.custom_validation_only))
.filter(([_, value]) => this.customValidation || (!this.customValidation && !value.custom_validation_only))
.map(([filterKey]) => {
return {
value: filterKey,

View File

@ -66,11 +66,11 @@ export function useAlert () {
let description = ''
if (Object.keys(error.errors).length > 0) {
const errorLines = Object.entries(error.errors)
.map(([field, messages]) => {
.map(([_, messages]) => {
// Format each message
const formattedMessages = messages.map(message => {
return `<li>${message}</li>`
});
})
return formattedMessages.join('')
})

View File

@ -12,12 +12,10 @@ function mergeOptions(options) {
max: 30,
persistent: false,
persistentStrategy: {
// eslint-disable-next-line no-unused-vars
get: function (store, type) {
get: function (_store, _type) {
// Todo
},
// eslint-disable-next-line no-unused-vars
set: function (store, type, value) {
set: function (_store, _type, _value) {
// Todo
},
remove: function (store, type) {