Enlève l’heuristique des organismes dans le déploiement

betagouv · Nov 6, 2024 · 778dfda · 778dfda
1 parent 3e5efad
commit 778dfda
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 26 deletions.
diff --git a/app/controllers/ApiController.scala b/app/controllers/ApiController.scala
@@ -343,9 +343,7 @@ case class ApiController @Inject() (
         "Accès non autorisé au dashboard de déploiement"
       ) { () =>
         val userGroups = userGroupService.allOrThrow.filter(group =>
-          group.organisation
-            .orElse(Organisation.deductedFromName(group.name))
-            .exists(_.id === Organisation.franceServicesId)
+          group.organisation.exists(_.id === Organisation.franceServicesId)
         )
         val franceServiceInstances = organisationService.franceServiceInfos.instances
         val doNotMatchTheseEmails =
@@ -439,11 +437,7 @@ case class ApiController @Inject() (
             for {
               group <- userGroups.filter(group =>
                 group.areaIds.contains[UUID](area.id)
-                  && organisationSet.exists(
-                    group.organisation
-                      .orElse(Organisation.deductedFromName(group.name))
-                      .contains[Organisation]
-                  )
+                  && organisationSet.exists(group.organisation.contains[Organisation])
               )
               user <- users if user.groupIds.contains[UUID](group.id)
             } yield user

diff --git a/app/models/Organisation.scala b/app/models/Organisation.scala
@@ -157,15 +157,4 @@ object Organisation {
       subjects: Seq[Subject]
   )
 
-  def deductedFromName(name: String): Option[Organisation] = {
-    val lowerCaseName = name.toLowerCase().stripSpecialChars
-    // Hack: `.reverse` the orgs so we can match first MSAP before MSA and
-    // Sous-Préf before Préf
-    all.reverse
-      .find { organisation =>
-        lowerCaseName.contains(organisation.shortName.toLowerCase().stripSpecialChars) ||
-        lowerCaseName.contains(organisation.name.toLowerCase().stripSpecialChars)
-      }
-  }
-
 }
diff --git a/app/serializers/UserAndGroupCsvSerializer.scala b/app/serializers/UserAndGroupCsvSerializer.scala
@@ -264,11 +264,22 @@ object UserAndGroupCsvSerializer {
       }
     }
 
+    def organisationFromNameHeuristic(name: String): Option[Organisation] = {
+      val lowerCaseName = name.toLowerCase().stripSpecialChars
+      // Hack: `.reverse` the orgs so we can match first MSAP before MSA and
+      // Sous-Préf before Préf
+      Organisation.all.reverse
+        .find { organisation =>
+          lowerCaseName.contains(organisation.shortName.toLowerCase().stripSpecialChars) ||
+          lowerCaseName.contains(organisation.name.toLowerCase().stripSpecialChars)
+        }
+    }
+
     def matchOrganisationId: CSVMap = {
       val optionOrganisation = csvMap
         .get(GROUP_ORGANISATION.key)
         .orElse(csvMap.get(GROUP_NAME.key))
-        .flatMap(Organisation.deductedFromName)
+        .flatMap(organisationFromNameHeuristic)
       optionOrganisation match {
         case Some(organisation) =>
           csvMap + (GROUP_ORGANISATION.key -> organisation.id.id)

diff --git a/app/views/allUsersByGroup.scala.html b/app/views/allUsersByGroup.scala.html
@@ -93,11 +93,10 @@
               </a>
             }
             @userGroup.areaIds.flatMap(Area.fromId).map(_.name).sorted.mkString(", ") /
-            <b>@userGroup.name</b> / @(userGroup.organisation, Organisation.deductedFromName(userGroup.name)) match {
-              case (Some(organisation), _) => {
+            <b>@userGroup.name</b> / @userGroup.organisation match {
+              case Some(organisation) => {
                 @organisation.shortName / }
-              case (_, Some(organisation)) => { @organisation.shortName / }
-              case _ => {}
+              case None => {}
             } INSEE : @userGroup.inseeCode  <br>
             <span class="text--font-size-medium">@userGroup.description</span>
           }{

diff --git a/app/views/deploymentDashboard.scala.html b/app/views/deploymentDashboard.scala.html
@@ -5,8 +5,8 @@
     <div class="mdl-cell mdl-cell--12-col">
         <a class="mdl-button mdl-button--raised" href="@routes.AreaController.franceServiceDeploymentDashboard">Tableau Déploiement France Services</a>
     </div>
-    <h5 class="title--addline">Nombre d'agents instructeurs par organisation</h5>
-    <i>Les agents sont comptés double si ils sont dans plusieurs départements. Les organisations manquantes ne sont pas affichées. Certains agents ne sont pas détectés comme faisant partie d'une organisation (Vous devez corriger le groupe dans ce cas)</i>
+    <h5 class="title--addline">Nombre d’agents instructeurs par organisme</h5>
+    <i>Les agents sont comptés double si ils sont dans plusieurs départements. Les organismes manquants ne sont pas affichés. Certains agents ne sont pas détectés comme faisant partie d'une organisation (Vous devez corriger le groupe dans ce cas)</i>
     <div id="aplus-admin-deployment-table" class="mdl-cell mdl-cell--12-col"></div>
     <div class="mdl-cell mdl-cell--12-col">
         <a class="mdl-button mdl-button--raised" href="@routes.AreaController.deploymentDashboard?uniquement-fs=non">Afficher tous les organismes</a>