From 54eea7c09bd52db59292d9c747bff32cb2e674cf Mon Sep 17 00:00:00 2001 From: David Mendy Date: Thu, 23 May 2024 11:12:54 +0200 Subject: [PATCH] refactor: :recycle: delete isSecuringOptions from typologies files --- frontend/src/utils/firearms-utils/autre-pistolet.ts | 1 - .../src/utils/firearms-utils/epaule-a-levier-sous-garde.ts | 1 - frontend/src/utils/firearms-utils/epaule-a-pompe.ts | 1 - .../src/utils/firearms-utils/epaule-a-un-coup-par-canon.ts | 1 - frontend/src/utils/firearms-utils/epaule-a-verrou.ts | 1 - .../firearms-utils/epaule-semi-auto-militaire-milieu-20e.ts | 1 - .../src/utils/firearms-utils/epaule-semi-auto-style-chasse.ts | 1 - frontend/src/utils/firearms-utils/epaule_mecanisme_ancien.ts | 1 - .../src/utils/firearms-utils/get-next-route-after-result.ts | 4 ++-- .../src/utils/firearms-utils/pistolet-mecanisme-ancien.ts | 1 - .../src/utils/firearms-utils/pistolet-semi-auto-moderne.ts | 1 - frontend/src/utils/firearms-utils/revolver.ts | 2 -- .../utils/firearms-utils/semi-auto-style-militaire-autre.ts | 1 - .../src/views/GuideSecuringFirearm/SecuringAchievement.vue | 2 +- 14 files changed, 3 insertions(+), 16 deletions(-) diff --git a/frontend/src/utils/firearms-utils/autre-pistolet.ts b/frontend/src/utils/firearms-utils/autre-pistolet.ts index 73e804372..f9331b22b 100644 --- a/frontend/src/utils/firearms-utils/autre-pistolet.ts +++ b/frontend/src/utils/firearms-utils/autre-pistolet.ts @@ -18,7 +18,6 @@ export const autre_pistolet = { displayLabel: 'Pistolet divers', category: 'A, B ou D', isDummyTypology: false, - isSecuringOptions: true, options_text: '', options: { drulov: { diff --git a/frontend/src/utils/firearms-utils/epaule-a-levier-sous-garde.ts b/frontend/src/utils/firearms-utils/epaule-a-levier-sous-garde.ts index a801882d3..d1976edd8 100644 --- a/frontend/src/utils/firearms-utils/epaule-a-levier-sous-garde.ts +++ b/frontend/src/utils/firearms-utils/epaule-a-levier-sous-garde.ts @@ -11,7 +11,6 @@ export const epaule_a_levier_sous_garde = { displayLabel: "Arme d'épaule à levier de sous-garde", category: 'B ou C', isDummyTypology: false, - isSecuringOptions: true, options_text: 'En manipulant l’arme avec précaution dans une direction sans risque, inspectez l’extrémité du tube sous le canon. Sélectionnez ce que vous voyez :', options: { front_magazine: { diff --git a/frontend/src/utils/firearms-utils/epaule-a-pompe.ts b/frontend/src/utils/firearms-utils/epaule-a-pompe.ts index 28be28b39..3dab252a3 100644 --- a/frontend/src/utils/firearms-utils/epaule-a-pompe.ts +++ b/frontend/src/utils/firearms-utils/epaule-a-pompe.ts @@ -7,7 +7,6 @@ export const epaule_a_pompe = { displayLabel: "Arme d'épaule à pompe", category: 'B ou C', isDummyTypology: false, - isSecuringOptions: true, options_text: '', text_steps: { 1: 'Observer l’arme en l’orientant dans une direction sans risque, en manipulant avec précaution', diff --git a/frontend/src/utils/firearms-utils/epaule-a-un-coup-par-canon.ts b/frontend/src/utils/firearms-utils/epaule-a-un-coup-par-canon.ts index 7c79d53da..5f09bdbad 100644 --- a/frontend/src/utils/firearms-utils/epaule-a-un-coup-par-canon.ts +++ b/frontend/src/utils/firearms-utils/epaule-a-un-coup-par-canon.ts @@ -14,7 +14,6 @@ export const epaule_a_un_coup_par_canon = { displayLabel: "Arme d'épaule à un coup par canon", category: 'B ou C', isDummyTypology: false, - isSecuringOptions: true, options_text: 'Sélectionnez ce que vous voyez sur la zone centrale de l’arme', options: { cle_pouce: { diff --git a/frontend/src/utils/firearms-utils/epaule-a-verrou.ts b/frontend/src/utils/firearms-utils/epaule-a-verrou.ts index 3b6e9aff5..4dcd5ef9e 100644 --- a/frontend/src/utils/firearms-utils/epaule-a-verrou.ts +++ b/frontend/src/utils/firearms-utils/epaule-a-verrou.ts @@ -9,7 +9,6 @@ export const epaule_a_verrou = { displayLabel: 'Arme d’épaule à verrou', category: 'B ou C', isDummyTypology: true, - isSecuringOptions: true, text_steps: { 1: 'Observer l’arme en l’orientant dans une direction sans risque, en manipulant avec précaution', 2: 'Ouvrez la culasse en tirant le levier vers le haut puis en la tirant la culasse vers l’arrière', diff --git a/frontend/src/utils/firearms-utils/epaule-semi-auto-militaire-milieu-20e.ts b/frontend/src/utils/firearms-utils/epaule-semi-auto-militaire-milieu-20e.ts index b709450fc..ea82308f2 100644 --- a/frontend/src/utils/firearms-utils/epaule-semi-auto-militaire-milieu-20e.ts +++ b/frontend/src/utils/firearms-utils/epaule-semi-auto-militaire-milieu-20e.ts @@ -5,5 +5,4 @@ export const epaule_semi_auto_style_militaire_milieu_20e = { displayLabel: 'Arme d’épaule semi-automatique ou automatique', category: 'A ou B', isDummyTypology: false, - isSecuringOptions: false, } as const diff --git a/frontend/src/utils/firearms-utils/epaule-semi-auto-style-chasse.ts b/frontend/src/utils/firearms-utils/epaule-semi-auto-style-chasse.ts index 581b95438..713bf1217 100644 --- a/frontend/src/utils/firearms-utils/epaule-semi-auto-style-chasse.ts +++ b/frontend/src/utils/firearms-utils/epaule-semi-auto-style-chasse.ts @@ -5,5 +5,4 @@ export const epaule_semi_auto_style_chasse = { displayLabel: 'Arme d’épaule semi-automatique', category: 'B ou C', isDummyTypology: false, - isSecuringOptions: false, } as const diff --git a/frontend/src/utils/firearms-utils/epaule_mecanisme_ancien.ts b/frontend/src/utils/firearms-utils/epaule_mecanisme_ancien.ts index 8148101da..2dc588b63 100644 --- a/frontend/src/utils/firearms-utils/epaule_mecanisme_ancien.ts +++ b/frontend/src/utils/firearms-utils/epaule_mecanisme_ancien.ts @@ -5,5 +5,4 @@ export const epaule_mecanisme_ancien = { displayLabel: "Arme d'épaule à mécanisme ancien", category: 'D', isDummyTypology: false, - isSecuringOptions: false, } as const diff --git a/frontend/src/utils/firearms-utils/get-next-route-after-result.ts b/frontend/src/utils/firearms-utils/get-next-route-after-result.ts index b6249b4fa..6b1cb6838 100644 --- a/frontend/src/utils/firearms-utils/get-next-route-after-result.ts +++ b/frontend/src/utils/firearms-utils/get-next-route-after-result.ts @@ -9,8 +9,8 @@ export const getNextRouteAfterResult = ({ securingTutorial, confidenceLevel, typ if (isCardDetected === false && isMeasuredGun === true) { return { name: 'MissingCard' } } else { return { name: 'IdentificationTypologyResult' } } } - const hasNoSecuringOptions = !resultTree[typology].isSecuringOptions - if (hasNoSecuringOptions) { + const hasSecuringOptions = resultTree[typology]?.options_step_1 || resultTree[typology]?.options || resultTree[typology]?.text_steps + if (!hasSecuringOptions) { return { name: 'SecuringAchievement' } } diff --git a/frontend/src/utils/firearms-utils/pistolet-mecanisme-ancien.ts b/frontend/src/utils/firearms-utils/pistolet-mecanisme-ancien.ts index c9d734115..5367d09a0 100644 --- a/frontend/src/utils/firearms-utils/pistolet-mecanisme-ancien.ts +++ b/frontend/src/utils/firearms-utils/pistolet-mecanisme-ancien.ts @@ -5,5 +5,4 @@ export const pistolet_mecanisme_ancien = { displayLabel: 'Pistolet à mécanisme ancien', category: 'D', isDummyTypology: false, - isSecuringOptions: false, } as const diff --git a/frontend/src/utils/firearms-utils/pistolet-semi-auto-moderne.ts b/frontend/src/utils/firearms-utils/pistolet-semi-auto-moderne.ts index 76d64e594..5fbcffe31 100644 --- a/frontend/src/utils/firearms-utils/pistolet-semi-auto-moderne.ts +++ b/frontend/src/utils/firearms-utils/pistolet-semi-auto-moderne.ts @@ -13,7 +13,6 @@ export const pistolet_semi_auto_moderne = { displayLabel: 'Pistolet semi-automatique', category: 'B', isDummyTypology: true, - isSecuringOptions: true, pistolet_semi_auto_moderne_text_option: 'Sélectionner ce que vous voyez sur votre arme : bouton à proximité du pontet du côté gauche de la poignée, OU bouton sur le talon de la crosse.', textOptions: '', options: { diff --git a/frontend/src/utils/firearms-utils/revolver.ts b/frontend/src/utils/firearms-utils/revolver.ts index e4b614c1a..d358fc907 100644 --- a/frontend/src/utils/firearms-utils/revolver.ts +++ b/frontend/src/utils/firearms-utils/revolver.ts @@ -39,7 +39,6 @@ export const revolver = { category: 'B', categoryWithoutSecuring: 'B ou D', isDummyTypology: true, - isSecuringOptions: true, options_step_1_text: 'En maintenant l’arme dans une direction sécurisée , sélectionnez ce que vous voyez.', options_step_1: { black_powder: { @@ -221,5 +220,4 @@ export const revolver_black_powder = { displayLabel: 'Revolver à poudre noire', category: 'D', isDummyTypology: false, - isSecuringOptions: false, } as const diff --git a/frontend/src/utils/firearms-utils/semi-auto-style-militaire-autre.ts b/frontend/src/utils/firearms-utils/semi-auto-style-militaire-autre.ts index 8c8c2d294..3a24a0d6b 100644 --- a/frontend/src/utils/firearms-utils/semi-auto-style-militaire-autre.ts +++ b/frontend/src/utils/firearms-utils/semi-auto-style-militaire-autre.ts @@ -22,7 +22,6 @@ export const semi_auto_style_militaire_autre = { displayLabel: 'Arme semi-automatique ou automatique', category: 'A ou B', isDummyTypology: true, - isSecuringOptions: true, options_text: 'Observez la position du chargeur et sélectionnez ce que vous voyez :', options: { ar15: { diff --git a/frontend/src/views/GuideSecuringFirearm/SecuringAchievement.vue b/frontend/src/views/GuideSecuringFirearm/SecuringAchievement.vue index d9395b4a6..768024b0b 100644 --- a/frontend/src/views/GuideSecuringFirearm/SecuringAchievement.vue +++ b/frontend/src/views/GuideSecuringFirearm/SecuringAchievement.vue @@ -22,7 +22,7 @@ function goToMissingCardPageIfMissing () {