فهرست منبع

validate and fix alert bars

Olivier Massot 2 سال پیش
والد
کامیت
b3585ea911

+ 4 - 2
components/Layout/AlertBar.vue

@@ -11,7 +11,7 @@ Contient les différentes barres d'alertes qui s'affichent dans certains cas
     <LayoutAlertBarSwitchUser />
 
     <client-only>
-      <LayoutAlertBarCotisation v-if="organizationProfile.isCmf && can('manage', 'cotisation')" />
+      <LayoutAlertBarCotisation v-if="organizationProfile.isCmf && canManageCotisation" />
     </client-only>
 
     <LayoutAlertBarSwitchYear />
@@ -24,5 +24,7 @@ Contient les différentes barres d'alertes qui s'affichent dans certains cas
   import {useAbility} from "@casl/vue";
 
   const organizationProfile = useOrganizationProfileStore()
-  const { can } = useAbility()
+  const ability = useAbility()
+
+  const canManageCotisation = ability.can('manage', 'cotisation')
 </script>

+ 6 - 2
components/Layout/AlertBar/Cotisation.vue

@@ -6,7 +6,6 @@ Barre d'alerte qui s'affiche pour donner l'état de la cotisation
 
 <template>
   <main>
-    <!-- TODO : vérifier le bon fonctionnement -->
     <UiSystemBar
         v-if="alert !== null"
         :text="$t(alert.text)"
@@ -32,7 +31,6 @@ const baseLegacyUrl: string = runtimeConfig.baseUrlAdminLegacy
 
 const cotisationYear: Ref<number | null> = ref(null)
 
-
 /**
  * Redirige l'utilisateur vers la page des cotisations
  */
@@ -96,3 +94,9 @@ if (cotisation.value !== null) {
 }
 
 </script>
+
+<style scoped lang="scss">
+  :deep(.clickable:hover) {
+    text-decoration: none !important;
+  }
+</style>

+ 1 - 1
components/Layout/AlertBar/SuperAdmin.vue

@@ -36,7 +36,7 @@ Barre d'alerte qui s'affiche lorsque l'utilisateur est un super admin en mode sw
     const originalAccessId = accessProfile.originalAccess ? accessProfile.originalAccess.id : null
 
     if (show && orgId && originalAccessId) {
-      return UrlUtils.join(baseLegacyUrl, 'switch_user', orgId, originalAccessId, 'exit')
+      return UrlUtils.join(baseLegacyUrl, '#', 'switch_user', orgId, originalAccessId, 'exit')
     }
     return ''
   })

+ 7 - 7
components/Layout/AlertBar/SwitchUser.vue

@@ -5,12 +5,11 @@ Barre qui s'affiche lorsque l'utilisateur possède un compte multi user
 -->
 
 <template>
-  <!-- TODO : fonctionnement à valider -->
-  <UiSystemBar v-if="show" text-color="info">
+  <UiSystemBar v-if="show" class="theme-info">
     <v-icon small icon="fas fa-info-circle" />
-    <span v-html="$t('multi_account_alert', { fullname })" />
-    <v-icon class="ml-1" small icon="fa fa-users" />
-    {{$t('multi_account_alert_next')}}
+    <span v-html="$t('multi_account_alert', { fullName })" />&nbsp;
+
+    <v-icon class="pl-1" small icon="fa fa-users"/> &nbsp;{{$t('multi_account_alert_next')}}
   </UiSystemBar>
 </template>
 
@@ -21,8 +20,9 @@ Barre qui s'affiche lorsque l'utilisateur possède un compte multi user
   const accessProfile = useAccessProfileStore()
   const { hasMenu } = useMenu()
 
-  const show = hasMenu('Family')
-  const fullname = `${accessProfile.givenName} ${accessProfile.name}`
+  const show = computed(() => hasMenu('MyFamily'))
+
+  const fullName = `${accessProfile.givenName} ${accessProfile.name}`
 </script>
 
 <style scoped lang="scss">

+ 4 - 5
components/Layout/AlertBar/SwitchYear.vue

@@ -6,13 +6,11 @@ Barre d'alerte qui s'affiche lorsque l'utilisateur n'est pas sur l'année couran
 
 <template>
   <!-- TODO : fonctionnement à valider -->
-  <UiSystemBar v-if="show" text-color="warning">
+  <UiSystemBar v-if="show" class="theme-warning">
     {{$t('not_current_year')}}
 
-    <a @click="resetYear">
-      <strong class="text-neutral-strong">
-        {{$t('not_current_year_reset')}}
-      </strong>
+    <a @click="resetYear" class="text-decoration-none on-warning" style="cursor: pointer;">
+      &nbsp;<strong>{{$t('not_current_year_reset')}}</strong>
     </a>
   </UiSystemBar>
 </template>
@@ -43,6 +41,7 @@ Barre d'alerte qui s'affiche lorsque l'utilisateur n'est pas sur l'année couran
       accessProfile.activityYear = organizationProfile.currentActivityYear
     }
 
+    // Il faut ajouter un patch sur le profile ici
     setDirty(false)
 
     window.location.reload()

+ 3 - 0
composables/layout/useMenu.ts

@@ -26,6 +26,9 @@ export const useMenu = () => {
 
   /**
    * Construct all Menus
+   * TODO: ce serait mieux de conserver les ids des menus même non possédés, de façon à pouvoir différencier un menu
+   * non possédé et un id incorrect dans getMenu par exemple. J'ai eu du mal capter pourquoi hasMenu('Family') renvoyait
+   * false, jusqu'à ce que je tilte que le menu s'appellait MyFamily, et pas Family
    */
   const buildAllMenu = () => {
     MenuComposer.build(runtimeConfig, ability, organizationProfile, accessProfile as AccessProfile, layoutState)

+ 2 - 2
lang/fr.json

@@ -386,9 +386,9 @@
   "cotisation_access": "Accéder au renouvellement de cotisation à ma fédération",
   "information_new_online_registration": "Nouvelle préinscription",
   "not_production_environment": "ATTENTION ! Vous êtes sur un environnement {env}.",
-  "multi_account_alert": "Vous êtes connecté, en tant que <strong>{fullname}</strong>, avec un accès famille. Utilisez l'icône",
+  "multi_account_alert": "Vous êtes connecté, en tant que <strong>{fullName}</strong>, avec un accès famille. Utilisez l'icône",
   "multi_account_alert_next": "en haut à droite pour changer les informations des autres membres de votre famille.",
-  "not_current_year": "Votre logiciel est actuellement placé dans une autre année que celle actuelle, et/ou affiche des données passées/futures.",
+  "not_current_year": "Les données affichées correspondent à une autre année que l'année actuelle, et/ou sont des données passées ou futures.",
   "not_current_year_reset": "Cliquez ici pour afficher les données de l'année actuelle.",
   "welcome": "Accueil",
   "address_book": "Répertoire",