Explorar el Código

fix after code review

Vincent GUFFON hace 3 años
padre
commit
33a6f869e8

+ 1 - 1
components/Form/Organization/Address.vue

@@ -165,7 +165,7 @@ export default defineComponent({
       return actions
     })
 
-    /** Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       model: OrganizationAddressPostal,
       query: () => query,

+ 1 - 0
components/Form/Organization/BankAccount.vue

@@ -84,6 +84,7 @@ export default defineComponent({
       return actions
     })
 
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       model: BankAccount,
       query: () => query,

+ 1 - 0
components/Form/Organization/ContactPoint.vue

@@ -98,6 +98,7 @@ export default defineComponent({
       return actions
     })
 
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       model: ContactPoint,
       query: () => query,

+ 1 - 1
components/Form/Parameters/Cycle.vue

@@ -64,7 +64,7 @@ export default defineComponent({
       return actions
     })
 
-    /** Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       model: Cycle,
       query: () => query,

+ 1 - 1
components/Form/Parameters/EducationTiming.vue

@@ -72,7 +72,7 @@ export default defineComponent({
       return actions
     })
 
-    /** Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       model: EducationTiming,
       query: () => query,

+ 1 - 1
components/Form/Parameters/ResidenceArea.vue

@@ -64,7 +64,7 @@ export default defineComponent({
       return actions
     })
 
-    /** Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       model: ResidenceArea,
       query: () => query,

+ 2 - 2
components/Ui/Form.vue

@@ -147,12 +147,12 @@ export default defineComponent({
               }
 
               new Form(store).addViolations(fields)
-              page.addAlerts(TYPE_ALERT.ALERT, ['invalide_form'])
+              page.addAlerts(TYPE_ALERT.ALERT, ['invalid_form'])
             }
           }
         }
       }else{
-        page.addAlerts(TYPE_ALERT.ALERT, ['invalide_form'])
+        page.addAlerts(TYPE_ALERT.ALERT, ['invalid_form'])
       }
     }
 

+ 3 - 2
lang/field/fr-FR.js

@@ -1,5 +1,6 @@
 export default (context, locale) => {
   return ({
+    parameters: 'Paramètres',
     cycle: 'Cycle',
     timing: 'Durée d\'un enseignement (en minutes)',
     educationTiming: 'Durée d\'un enseignement (en minutes)',
@@ -10,7 +11,7 @@ export default (context, locale) => {
     passwordSMS: 'Mot de passe SMS',
     usernameSMS: 'Nom d\'utilisateur SMS',
     smsSenderName: 'Personnaliser le nom de l\'expéditeur SMS',
-    attendance: 'Abscences',
+    attendance: 'Absences',
     sendAttendanceEmail: 'Prévenir automatiquement la famille par mail en cas d\'absence non justifiée',
     sendAttendanceSms: 'Prévenir automatiquement la famille par sms en cas d\'absence non justifiée',
     bulletinReceiver: 'Adresser le bulletin à',
@@ -39,7 +40,7 @@ export default (context, locale) => {
     otherWebsite: 'Votre site Opentalent est',
     timezone: 'Fuseau horaire',
     qrCode: 'QrCode pour la licence',
-    studentsAreAdherents: 'Les élèves sont adhérents également de l\'association',
+    studentsAreAdherents: 'Les élèves sont également adhérents de l\'association',
     showAdherentList: 'Afficher la liste des adhérents et leurs coordonnées',
     endCourseDate: 'Date de fin des cours ',
     startCourseDate: 'Date de début des cours ',

+ 3 - 3
lang/help/fr-FR.js

@@ -7,8 +7,8 @@
  */
 export default (context, locale) => {
   return ({
-    bulletinEditWithoutEvaluationHelp:'Dans le cas ou la case n\'est pas cochée, si aucune évaluation dans le bulletin alors le bulletin n\'est pas exporté',
-    bulletinShowEducationWithoutEvaluationHelp: 'Dans le cas ou la case est cochée, alors pour chaque enseignement, on affiche le texte \"Aucune évaluation\" si aucune évaluation',
+    bulletinEditWithoutEvaluationHelp:'Dans le cas où la case n\'est pas cochée, s\'il n\'y a aucune évaluation dans le bulletin, alors le bulletin n\'est pas exporté',
+    bulletinShowEducationWithoutEvaluationHelp: 'Dans le cas où la case est cochée, alors on affiche le texte "Aucune évaluation" pour chaque enseignement sans évaluation',
     type_of_practices_autocomplete: 'Sélectionnez parmi la liste des types de pratiques, une ou plusieurs pratiques correspondant aux activités de votre structure',
     logo_upload: '<div>Le logo est utilisée: </div>' +
       `\n<ul>` +
@@ -17,7 +17,7 @@ export default (context, locale) => {
       '    <li>dans la recherche d\'une structure sur le site Opentalent ou sur le site d\'une fédération si vous êtes membre de la CMF ' +
       '        (voir <a target="_blank" href="https://fmfaucigny.opentalent.fr/presentation/societes-adherentes">exemple ici</a>)</li>' +
       '</ul>',
-    communication_image_upload: 'L\'image est utilisée\n' +
+    communication_image_upload: 'L\'image est utilisé\n' +
       'dans la description détaillée d\'une structure sur le site Opentalent ou ' +
       'sur le site d\'une fédération si vous êtes membre de la CMF ' +
       '(voir <a target="_blank" href="https://fmfaucigny.opentalent.fr/presentation/societes-adherentes">exemple ici</a>)'

+ 5 - 4
lang/rulesAndErrors/fr-FR.js

@@ -1,13 +1,14 @@
 export default (context, locale) => {
   return ({
     value_need_to_be_bigger_than_0: 'La valeur doit être plus grande que 0',
-    forbidden: 'Vous ne posséder pas les droits nécessaires pour effectuer cette opération',
+    forbidden: 'Vous ne possédez pas les droits nécessaires pour effectuer cette opération',
     wrong_mobyt_credentials: 'Identifiants SMS incorrects',
     smsSenderName_error: 'Seuls les caractères alphanumériques sont permis, sans espaces, sans accent et sans caractères spéciaux',
-    value_need_to_be_beetween_0_100: 'La valeur doit être comprise entre 0 et 100',
-    invalide_form: 'Formulaire invalide',
+    between_1_and_10: 'La valeur doit être comprise entre 0 et 10',
+    between_0_and_100: 'La valeur doit être comprise entre 0 et 10',
+    invalid_form: 'Formulaire invalide',
     required: 'Ce champs est obligatoire',
-    name_length_rule: 'La taille du nom doit être de moins de 128 caractères',
+    name_length_rule: 'La longueur du nom doit être de moins de 128 caractères',
     siret_error: 'N° de siret non valide',
     email_error: 'Adresse email invalide',
     phone_error: 'Numéro de téléphone invalide',

+ 1 - 1
pages/parameters.vue

@@ -5,7 +5,7 @@
 
       <v-row justify="center" align="center" class="bannerTopForm mt-5">
         <v-col cols="12" class="ot_dark_grey ot_white--text">
-          <h4>Paramètres</h4>
+          <h4>{{ $t('parameters') }}</h4>
         </v-col>
       </v-row>
 

+ 1 - 0
pages/parameters/billing/index.vue

@@ -55,6 +55,7 @@ export default defineComponent({
     const repository = repositoryHelper.getRepository(ResidenceArea)
     const query: ComputedRef<Query> = computed(() => repository.query())
 
+    /** todo Computed properties needs to be returned as functions until nuxt3 : https://github.com/nuxt-community/composition-api/issues/207 **/
     return {
       query: () => query,
       model: ResidenceArea

+ 1 - 1
pages/parameters/student.vue

@@ -156,7 +156,7 @@ export default defineComponent({
 function getRules (i18n: any) {
   return {
     averageRules: [
-      (averageValue: number) => (averageValue >= 1 && averageValue <= 100) || i18n.t('value_need_to_be_beetween_0_100')
+      (averageValue: number) => (averageValue >= 1 && averageValue <= 100) || i18n.t('between_0_and_100')
     ]
   }
 }

+ 1 - 5
services/data/baseDataManager.ts

@@ -42,11 +42,7 @@ abstract class BaseDataManager extends Hookable implements DataManager {
 
     await this.triggerHooks(queryArguments)
 
-    try {
-      return await this._invoke(queryArguments)
-    } catch (error) {
-      throw error
-    }
+    return await this._invoke(queryArguments)
   }
 
   /**