post reread fixes

config/nuxtConfig/build.js

@@ -23,12 +23,6 @@ export default {
     ]
   },
 
-  // Port and local host
-  server: {
-    port: 3002,
-    host: '0.0.0.0', // default: localhost,
-  },
-
   // Poll for hot reloading with docker
   watchers: {
     webpack: {

middleware/auth.ts

@@ -1,6 +1,6 @@
 import { Middleware } from '@nuxt/types'
 
-const auth: Middleware = async ({ store, redirect }) => {
+const auth: Middleware = ({ store, redirect }) => {
   // Si l'utilisateur n'est pas connecté on le redirige vers la page login
   if (!store.state.profile.access) {
     return redirect('/login')

notes_relecture.md

@@ -1,36 +0,0 @@
-
-## Générique
-
-* ok pour transférer le projet gitlab sur le namespace commun?
-
-## Config
-
-* config de l'host et du port: la doc mentionne que ce n'est pas conseillé 
-  de les définir ici, en tout cas pour la prod <https://nuxtjs.org/docs/2.x/features/configuration#edit-host-and-port>
-* Vincent, avais tu vu ce plugin? https://vuex-orm.github.io/plugin-axios/guide/setup.html
-
-
-# Divers
-
-* Pourrait-on renommer UrlArgs en Request?
-* manquerait pas un await dans middleware/auth.js?
-* un intérêt à utiliser les extensions .client.js et .server.js pour les plugins? (https://fr.nuxtjs.org/docs/2.x/directory-structure/plugins#convention-pour-le-nommage-des-plugins)
-* DatesUtils.sortDates: est-ce qu'un sort standard ne suffirait pas?
-* useChecker -> renommer en useValidator?
-* voir à factoriser le code des menus des use/...Menu
-* la couleur des liens est limite  limite question accessibilité (voir: https://local.admin2.opentalent.fr/organization/subscription)
-
-pour reproduire l'erreur lors des tests:
-
-    node_modules/jest/bin/jest.js --colors --verbose --runTestsByPath ./tests/unit/component/Ui/Xeditable/Text.spec.js --no-coverage
-
-
-## Questions
-
-* classe Connection.ts: pqoi AxiosRequestConfig accepte le mot clé progress et pas showPogress?
-* dans la doc composition-api, ils mettent leurs fonctions 'use' dans un répertoire 'composables', est-ce qu'on ne ferait pas la même chose?
-
-
-## Todo
-
-* renommer les data/processors en providers, et les faire hériter de dataprovider

package.json

@@ -3,7 +3,7 @@
   "version": "0.1.0",
   "private": true,
   "scripts": {
-    "dev": "nuxt",
+    "dev": "nuxt --hostname '0.0.0.0' --port 3002",
     "build": "nuxt build",
     "start": "nuxt start",
     "generate": "nuxt generate",

pages/organization/index.vue

@@ -316,7 +316,7 @@ import {OrganizationAddressPostal} from "~/models/Organization/OrganizationAddre
 import {ContactPoint} from "~/models/Core/ContactPoint";
 import {BankAccount} from "~/models/Core/BankAccount";
 import {repositoryHelper} from "~/services/store/repository";
-import UseChecker from "~/use/form/useChecker";
+import UseValidator from "~/use/form/useValidator";
 import {UseNavigationHelpers} from "~/use/form/useNavigationHelpers";
 
 export default defineComponent({
@@ -328,7 +328,7 @@ export default defineComponent({
 
     const repositories = getRepositories()
 
-    const {siretError, siretErrorMessage, checkSiret} = UseChecker.useHandleSiret(i18n, $dataProvider)
+    const {siretError, siretErrorMessage, checkSiret} = UseValidator.useHandleSiret(i18n, $dataProvider)
     const checkSiretHook = async (siret: string, field: string, updateRepository: any) => {
       await checkSiret(siret)
       if(!siretError.value)

services/connection/connection.ts

@@ -56,14 +56,14 @@ class Connection {
    * GET Item : préparation de la config pour la récupération d'un item
    * @param {string} url
    * @param {number} id
-   * @param {boolean} progress
+   * @param {boolean} showProgress
    * @return {Promise<any>}
    */
-  public static getItem (url: string, id: number, progress: boolean = true): Promise<any> {
+  public static getItem (url: string, id: number, showProgress: boolean = true): Promise<any> {
     const config: AxiosRequestConfig = {
       url: `${url}/${id}`,
       method: HTTP_METHOD.GET,
-      progress
+      progress: showProgress
     }
     return Connection.request(config)
   }

services/utils/datesUtils.ts

@@ -30,10 +30,6 @@ export default class DatesUtils {
    * @param dates
    */
   sortDate (dates: Array<string>): Array<string> {
-    return dates.sort((a, b) => {
-      if (a > b) { return 1 }
-      if (a < b) { return -1 }
-      return 0
-    })
+    return dates.sort()
   }
 }

tests/unit/use/form/useChecker.spec.ts → tests/unit/use/form/useValidator.spec.ts

@@ -1,5 +1,5 @@
 import VueI18n from 'vue-i18n'
-import UseChecker from '~/use/form/useChecker'
+import UseValidator from '~/use/form/useValidator'
 import DataProvider from '~/services/data/dataProvider'
 
 jest.mock('~/services/data/dataProvider')
@@ -13,14 +13,14 @@ describe('useHandleSiret()', () => {
   })
 
   it('should init 3 parameters correctly', async () => {
-    const siretResponse = UseChecker.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
+    const siretResponse = UseValidator.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
     expect(siretResponse.siretErrorMessage.value).toEqual('')
     expect(siretResponse.siretError.value).toBeFalsy()
     expect(siretResponse.checkSiret).toBeDefined()
   })
 
   it('should update parameters if Siret is correct', async () => {
-    const siretResponse = UseChecker.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
+    const siretResponse = UseValidator.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
     DataProviderMock.prototype.invoke.mockImplementation(async () => {
       return { isCorrect: true }
     })
@@ -30,7 +30,7 @@ describe('useHandleSiret()', () => {
   })
 
   it('should update parameters if Siret is not correct', async () => {
-    const siretResponse = UseChecker.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
+    const siretResponse = UseValidator.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
     DataProviderMock.prototype.invoke.mockImplementation(async () => {
       return { isCorrect: false }
     })
@@ -41,7 +41,7 @@ describe('useHandleSiret()', () => {
   })
 
   it('should update parameters if there is no response during the check siret', async () => {
-    const siretResponse = UseChecker.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
+    const siretResponse = UseValidator.useHandleSiret(VueI18nMock.prototype, DataProviderMock.prototype)
     DataProviderMock.prototype.invoke.mockImplementation(jest.fn())
     await siretResponse.checkSiret('123456')
     expect(siretResponse.siretErrorMessage.value).toEqual('')

use/form/useChecker.ts → use/form/useValidator.ts

@@ -5,10 +5,10 @@ import { DataManager } from '~/types/interfaces'
 
 /**
  * @category Use/form
- * @class UseChecker
+ * @class UseValidator
  * Use Classe pour des utils de verifications
  */
-class UseChecker {
+class UseValidator {
   /**
    * Use méthode fournissant une fonction pour tester la validité d'un Siret ainsi que la gestion du message d'erreur
    */
@@ -39,4 +39,4 @@ class UseChecker {
   }
 }
 
-export default UseChecker
+export default UseValidator