Bump eslint-plugin-vue from 7.20.0 to 9.6.0 (#7732)

* Bump eslint-plugin-vue from 7.20.0 to 9.6.0 Bumps [eslint-plugin-vue](https://github.com/vuejs/eslint-plugin-vue) from 7.20.0 to 9.6.0. - [Release notes](https://github.com/vuejs/eslint-plugin-vue/releases) - [Commits](vuejs/eslint-plugin-vue@v7.20.0...v9.6.0) --- updated-dependencies: - dependency-name: eslint-plugin-vue dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * eslint: Lower severity level for newly introduced rules * Update more eslint and prettier dependencies… …and use eslint-plugin-prettier-vue instead of eslint-plugin-prettier. * Fix script type -> lang Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dominik Schmidt <[email protected]>
owncloud · Oct 13, 2022 · 83d136c · 83d136c
1 parent 4b49928
commit 83d136c
Show file tree

Hide file tree

Showing 27 changed files with 410 additions and 415 deletions.
diff --git a/package.json b/package.json
@@ -79,7 +79,7 @@
     "cucumber-html-reporter": "^5.5.0",
     "depcheck": "^1.4.3",
     "ejs": "^3.1.5",
-    "eslint": "^7.32.0",
+    "eslint": "^8.25.0",
     "flush-promises": "^1.0.2",
     "focus-trap": "^6.9.0",
     "focus-trap-vue": "^1.1.1",

diff --git a/packages/eslint-config/index.js b/packages/eslint-config/index.js
@@ -4,12 +4,7 @@ module.exports = {
     es6: true,
     amd: true
   },
-  extends: [
-    'standard',
-    'prettier/standard',
-    'plugin:prettier/recommended',
-    'plugin:jest/recommended'
-  ],
+  extends: ['plugin:vue/recommended', 'plugin:prettier-vue/recommended', 'plugin:jest/recommended'],
   parser: 'vue-eslint-parser',
   parserOptions: {
     parser: {
@@ -41,15 +36,20 @@ module.exports = {
   overrides: [
     {
       files: ['**/*.vue'],
-      extends: ['plugin:vue/recommended', 'prettier/vue']
+      extends: ['plugin:vue/recommended', 'plugin:prettier-vue/recommended'],
+      rules: {
+        'vue/multi-word-component-names': 'warn',
+        'vue/no-v-text-v-html-on-component': 'warn'
+      }
     },
     {
       files: ['**/*.ts'],
       parser: '@typescript-eslint/parser',
       extends: ['plugin:@typescript-eslint/recommended'],
       rules: {
         '@typescript-eslint/no-extra-semi': 'off',
-        '@typescript-eslint/no-explicit-any': 'off'
+        '@typescript-eslint/no-explicit-any': 'off',
+        'prefer-const': 'warn'
       }
     }
   ]

diff --git a/packages/eslint-config/package.json b/packages/eslint-config/package.json
@@ -8,19 +8,19 @@
     "@babel/eslint-parser": "^7.19.1",
     "@typescript-eslint/eslint-plugin": "^5.39.0",
     "@typescript-eslint/parser": "^5.39.0",
-    "eslint-config-prettier": "^6.10.0",
-    "eslint-config-standard": "^16.0.3",
+    "eslint-config-prettier": "^8.5.0",
+    "eslint-config-standard": "^17.0.0",
     "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-jest": "^27.1.1",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^4.0.0",
-    "eslint-plugin-promise": "^5.2.0",
+    "eslint-plugin-prettier-vue": "^4.2.0",
+    "eslint-plugin-promise": "^6.0.1",
     "eslint-plugin-unused-imports": "^2.0.0",
-    "eslint-plugin-vue": "^7.13.0",
-    "eslint-plugin-vuejs-accessibility": "^0.7.1",
+    "eslint-plugin-vue": "^9.6.0",
+    "eslint-plugin-vuejs-accessibility": "^1.2.0",
     "typescript": "^4.8.4"
   },
   "peerDependencies": {
-    "eslint": "^7.32.0"
+    "eslint": "^8.25.0"
   }
 }
diff --git a/packages/prettier-config/package.json b/packages/prettier-config/package.json
@@ -5,6 +5,6 @@
     ".": "./index.js"
   },
   "dependencies": {
-    "prettier": "^2.3.2"
+    "prettier": "^2.7.1"
   }
 }
diff --git a/packages/web-app-files/src/components/AppBar/AppBar.vue b/packages/web-app-files/src/components/AppBar/AppBar.vue
@@ -37,7 +37,7 @@
           <slot
             v-if="showActionsOnSelection || selectedFiles.length === 0"
             name="actions"
-            :limitedScreenSpace="limitedScreenSpace"
+            :limited-screen-space="limitedScreenSpace"
           />
           <batch-actions
             v-if="showBatchActions"

diff --git a/packages/web-app-files/src/components/SideBar/Shares/Collaborators/RoleDropdown.vue b/packages/web-app-files/src/components/SideBar/Shares/Collaborators/RoleDropdown.vue
@@ -83,10 +83,7 @@
         </li>
       </oc-list>
       <div
-        class="
-          files-recipient-custom-permissions-drop-cancel-confirm-btns
-          oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-        "
+        class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom"
       >
         <oc-button
           size="small"

diff --git a/...ests/unit/components/SideBar/Shares/Collaborators/__snapshots__/RoleDropdown.spec.js.snap b/...ests/unit/components/SideBar/Shares/Collaborators/__snapshots__/RoleDropdown.spec.js.snap
@@ -36,10 +36,7 @@ exports[`RoleDropdown for file shares custom permissions inherits the parents sh
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -83,10 +80,7 @@ exports[`RoleDropdown for file shares custom permissions inherits the parents sh
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -130,10 +124,7 @@ exports[`RoleDropdown for file shares custom permissions inherits the parents sh
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -177,10 +168,7 @@ exports[`RoleDropdown for file shares when an existing role is present does not
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -224,10 +212,7 @@ exports[`RoleDropdown for file shares when an existing role is present renders a
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -271,10 +256,7 @@ exports[`RoleDropdown for file shares when an existing role is present renders a
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -312,10 +294,7 @@ exports[`RoleDropdown for file shares when no existing role is present renders a
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>
@@ -359,10 +338,7 @@ exports[`RoleDropdown for file shares when no existing role is present renders a
       <oc-checkbox-stub id="files-collaborators-permission-share" value="[object Object],[object Object]" option="[object Object]" label="Share" size="large" data-testid="files-collaborators-permission-share" class="oc-mr-xs files-collaborators-permission-checkbox"></oc-checkbox-stub>
     </li>
   </oc-list-stub>
-  <div class="
-        files-recipient-custom-permissions-drop-cancel-confirm-btns
-        oc-px-m oc-py-s oc-mt-m oc-rounded-bottom
-      ">
+  <div class="files-recipient-custom-permissions-drop-cancel-confirm-btns oc-px-m oc-py-s oc-mt-m oc-rounded-bottom">
     <oc-button-stub type="button" size="small" submit="button" variation="passive" appearance="outline" justifycontent="center" gapsize="medium">Cancel</oc-button-stub>
     <oc-button-stub type="button" size="small" submit="button" variation="primary" appearance="filled" justifycontent="center" gapsize="medium" class="oc-ml-s">Apply</oc-button-stub>
   </div>

diff --git a/packages/web-app-files/tests/unit/composables/composables.setup.ts b/packages/web-app-files/tests/unit/composables/composables.setup.ts
@@ -10,8 +10,8 @@ localVue.use(Vuex)
 export const createComposableWrapper = (setup: SetupFunction<Data, Data>): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div></div>`,
-      setup
+      setup,
+      template: `<div></div>`
     }),
     {
       localVue

diff --git a/packages/web-app-files/tests/unit/composables/fileListHeaderPosition/spec/index.ts b/packages/web-app-files/tests/unit/composables/fileListHeaderPosition/spec/index.ts
@@ -12,12 +12,12 @@ localVue.use(Vuex)
 export const createWrapper = (): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div>{{ y }}</div>`,
       setup() {
         const { y, refresh } = useFileListHeaderPosition()
 
         return { y, refresh, store: useStore() }
-      }
+      },
+      template: `<div>{{ y }}</div>`
     }),
     {
       localVue,

diff --git a/packages/web-app-files/tests/unit/composables/sort/spec/index.ts b/packages/web-app-files/tests/unit/composables/sort/spec/index.ts
@@ -10,8 +10,8 @@ localVue.use(Vuex)
 export const createWrapper = (setup: SetupFunction<Data, Data>): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div></div>`,
-      setup
+      setup,
+      template: `<div></div>`
     }),
     {
       localVue

diff --git a/packages/web-app-files/tests/unit/composables/upload/spec/index.ts b/packages/web-app-files/tests/unit/composables/upload/spec/index.ts
@@ -46,8 +46,8 @@ export const createWrapper = (
 ): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div></div>`,
-      setup
+      setup,
+      template: `<div></div>`
     }),
     {
       localVue,

diff --git a/packages/web-app-preview/src/App.vue b/packages/web-app-preview/src/App.vue
@@ -36,10 +36,7 @@
     <template v-else>
       <div
         v-show="activeMediaFileCached"
-        class="
-          oc-width-1-1 oc-flex oc-flex-center oc-flex-middle oc-p-s oc-box-shadow-medium
-          preview-player
-        "
+        class="oc-width-1-1 oc-flex oc-flex-center oc-flex-middle oc-p-s oc-box-shadow-medium preview-player"
       >
         <img
           v-if="activeMediaFileCached.isImage"
@@ -68,16 +65,7 @@
     </template>
     <div class="oc-position-medium oc-position-bottom-center preview-details">
       <div
-        class="
-          oc-background-brand
-          oc-p-s
-          oc-width-large
-          oc-flex
-          oc-flex-middle
-          oc-flex-center
-          oc-flex-around
-          preview-controls-action-bar
-        "
+        class="oc-background-brand oc-p-s oc-width-large oc-flex oc-flex-middle oc-flex-center oc-flex-around preview-controls-action-bar"
       >
         <oc-button
           class="preview-controls-previous"

diff --git a/packages/web-app-user-management/src/views/Users.vue b/packages/web-app-user-management/src/views/Users.vue
@@ -99,6 +99,7 @@ import { useGraphClient } from 'web-client/src/composables'
 import AppTemplate from '../components/AppTemplate.vue'
 
 export default defineComponent({
+  name: 'UsersView',
   components: {
     AppTemplate,
     UsersList,

diff --git a/packages/web-client/src/webdav/getFileUrl.ts b/packages/web-client/src/webdav/getFileUrl.ts
@@ -22,7 +22,8 @@ export const GetFileUrlFactory = (
       }
     ): Promise<string> {
       const inlineDisposition = disposition === 'inline'
-      let { path, downloadURL } = resource
+      const { path } = resource
+      let { downloadURL } = resource
 
       let signed = true
       if (!downloadURL && !inlineDisposition) {

diff --git a/packages/web-pkg/tests/unit/composables/capability/spec/index.ts b/packages/web-pkg/tests/unit/composables/capability/spec/index.ts
@@ -10,8 +10,8 @@ localVue.use(Vuex)
 export const createWrapper = (setup: SetupFunction<Data, Data>): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div></div>`,
-      setup
+      setup,
+      template: `<div></div>`
     }),
     {
       localVue

diff --git a/packages/web-pkg/tests/unit/composables/router/spec/index.ts b/packages/web-pkg/tests/unit/composables/router/spec/index.ts
@@ -13,8 +13,8 @@ export const createWrapper = (
 ): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div>${template}</div>`,
-      setup
+      setup,
+      template: `<div>${template}</div>`
     }),
     {
       localVue,

diff --git a/packages/web-pkg/tests/unit/composables/store/spec/index.ts b/packages/web-pkg/tests/unit/composables/store/spec/index.ts
@@ -10,8 +10,8 @@ localVue.use(Vuex)
 export const createWrapper = (setup: SetupFunction<Data, Data>): Wrapper<Vue> =>
   mount(
     defineComponent({
-      template: `<div></div>`,
-      setup
+      setup,
+      template: `<div></div>`
     }),
     {
       localVue,

diff --git a/packages/web-runtime/src/components/UploadInfo.vue b/packages/web-runtime/src/components/UploadInfo.vue
@@ -1,10 +1,7 @@
 <template>
   <div v-if="showInfo" id="upload-info" class="oc-rounded oc-box-shadow-medium">
     <div
-      class="
-        upload-info-title
-        oc-flex oc-flex-between oc-flex-middle oc-px-m oc-py-s oc-rounded-top
-      "
+      class="upload-info-title oc-flex oc-flex-between oc-flex-middle oc-px-m oc-py-s oc-rounded-top"
     >
       <p class="oc-my-xs" v-text="uploadInfoTitle" />
       <oc-button

diff --git a/packages/web-runtime/src/defaults/vue.js b/packages/web-runtime/src/defaults/vue.js
@@ -29,7 +29,7 @@ Vue.use(VueMeta, {
 Vue.use(PortalVue)
 Vue.use(AsyncComputed)
 
-Vue.component('avatar-image', Avatar)
+Vue.component('AvatarImage', Avatar)
 
 Vue.mixin(focusMixin)
 Vue.mixin(lifecycleMixin)

diff --git a/packages/web-runtime/src/layouts/Application.vue b/packages/web-runtime/src/layouts/Application.vue
@@ -44,6 +44,7 @@ import {
 import { watch, defineComponent } from '@vue/composition-api'
 
 export default defineComponent({
+  name: 'ApplicationLayout',
   components: {
     AppLoadingSpinner,
     MessageBar,

diff --git a/packages/web-runtime/src/layouts/Loading.vue b/packages/web-runtime/src/layouts/Loading.vue
@@ -12,6 +12,7 @@
 import { mapGetters } from 'vuex'
 
 export default {
+  name: 'LoadingLayout',
   computed: {
     ...mapGetters(['configuration'])
   }

diff --git a/packages/web-runtime/src/layouts/Plain.vue b/packages/web-runtime/src/layouts/Plain.vue
@@ -3,3 +3,11 @@
     <router-view />
   </div>
 </template>
+
+<script lang="ts">
+import { defineComponent } from '@vue/composition-api'
+
+export default defineComponent({
+  name: 'PlainLayout'
+})
+</script>