Skip to content

Commit

Permalink
refactor: ♻️ delete isSecuringOptions from typologies files
Browse files Browse the repository at this point in the history
  • Loading branch information
nutfdt committed May 23, 2024
1 parent 329d579 commit 5095d4d
Show file tree
Hide file tree
Showing 14 changed files with 21 additions and 34 deletions.
1 change: 0 additions & 1 deletion frontend/src/utils/firearms-utils/autre-pistolet.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ export const autre_pistolet = {
displayLabel: 'Pistolet divers',
category: 'A, B ou D',
isDummyTypology: false,
isSecuringOptions: true,
options_text: '',
options: {
drulov: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 <span class="font-bold">direction sans risque</span>, inspectez l’<span class="font-bold">extrémité du tube sous le canon</span>. Sélectionnez ce que vous voyez :',
options: {
front_magazine: {
Expand Down
1 change: 0 additions & 1 deletion frontend/src/utils/firearms-utils/epaule-a-pompe.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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 <span class="font-bold">direction sans risque</span>, en manipulant avec précaution',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 <span class="font-bold">zone centrale de l’arme</span>',
options: {
cle_pouce: {
Expand Down
1 change: 0 additions & 1 deletion frontend/src/utils/firearms-utils/epaule-a-verrou.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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 <span class="font-bold">direction sans risque</span>, 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',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Original file line number Diff line number Diff line change
Expand Up @@ -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
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,4 @@ export const epaule_mecanisme_ancien = {
displayLabel: "Arme d'épaule à mécanisme ancien",
category: 'D',
isDummyTypology: false,
isSecuringOptions: false,
} as const
Original file line number Diff line number Diff line change
Expand Up @@ -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' }
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,4 @@ export const pistolet_mecanisme_ancien = {
displayLabel: 'Pistolet à mécanisme ancien',
category: 'D',
isDummyTypology: false,
isSecuringOptions: false,
} as const
Original file line number Diff line number Diff line change
Expand Up @@ -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 : <span class="font-bold">bouton à proximité du pontet du côté gauche de la poignée</span>, OU <span class="font-bold">bouton sur le talon</span> de la crosse.',
textOptions: '',
options: {
Expand Down
2 changes: 0 additions & 2 deletions frontend/src/utils/firearms-utils/revolver.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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 <span class="font-bold">direction sécurisée</span> , sélectionnez ce que vous voyez.',
options_step_1: {
black_powder: {
Expand Down Expand Up @@ -221,5 +220,4 @@ export const revolver_black_powder = {
displayLabel: 'Revolver à poudre noire',
category: 'D',
isDummyTypology: false,
isSecuringOptions: false,
} as const
Original file line number Diff line number Diff line change
Expand Up @@ -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: {
Expand Down
38 changes: 19 additions & 19 deletions frontend/src/views/GuideSecuringFirearm/SecuringAchievement.vue
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,25 @@ function goToMissingCardPageIfMissing () {
<template>
<div class="fr-container">
<div class="result fr-col-11 fr-col-lg-6 m-auto">
<div v-if="!resultTree[typology]?.isSecuringOptions">
<div v-if="resultTree[typology]?.options_step_1 || resultTree[typology]?.options || resultTree[typology]?.text_steps">
<h2 class="mt-5 text-center">
Fin de la mise en sécurité de l’arme
</h2>
<p class="ending font-600 text-center">
Vous venez de sécuriser votre arme !
</p>
<p>
La manipulation étant terminée, vous pouvez identifier votre arme ou retourner au menu.
</p>
<div class="fr-col-sm-6 fr-col-lg-8 mx-auto text-center">
<img
src="@/assets/guide-mise-en-securite/icones/fin-mise-en-securite-ok.jpg"
alt="alt"
class="img-deco"
>
</div>
</div>
<div v-else>
<h2 class="mt-5 text-center">
Pas de guide de mise en sécurité pour votre arme
</h2>
Expand All @@ -44,24 +62,6 @@ function goToMissingCardPageIfMissing () {
</div>
<ContactExpert v-if="isUserUsingCrosscall()" />
</div>
<div v-else>
<h2 class="mt-5 text-center">
Fin de la mise en sécurité de l’arme
</h2>
<p class="ending font-600 text-center">
Vous venez de sécuriser votre arme !
</p>
<p>
La manipulation étant terminée, vous pouvez identifier votre arme ou retourner au menu.
</p>
<div class="fr-col-sm-6 fr-col-lg-8 mx-auto text-center">
<img
src="@/assets/guide-mise-en-securite/icones/fin-mise-en-securite-ok.jpg"
alt="alt"
class="img-deco"
>
</div>
</div>
</div>
</div>
<div class="footer">
Expand Down

0 comments on commit 5095d4d

Please sign in to comment.