Merge branches 'll-refonte-connecteurs' and 'master' of https://git.unicaen.fr/open-source/OSE

module/Application/src/Application/Controller/AgrementController.php

@@ -38,7 +38,6 @@ class AgrementController extends AbstractController
     use WorkflowServiceAwareTrait;
 
 
-
     /**
      * Initialisation des filtres Doctrine pour les historique.
      * Objectif : laisser passer les enregistrements passés en historique pour mettre en évidence ensuite les erreurs
@@ -306,9 +305,11 @@ class AgrementController extends AbstractController
         //plusieurs années pour plusieurs intervenants avec un même code
 
         $listeIntervenants = $this->getServiceIntervenant()->getIntervenants($intervenant);
+        foreach ($listeIntervenants as $objectIntervenant) {
             $this->getServiceWorkflow()->calculerTableauxBord([
                 'agrement',
                 'contrat',
-        ], $listeIntervenants);
+            ], $objectIntervenant);
+        }
     }
 }
\ No newline at end of file

module/Application/src/Application/Controller/PieceJointeController.php

@@ -38,7 +38,6 @@ class PieceJointeController extends AbstractController
     use WorkflowServiceAwareTrait;
 
 
-
     /**
      * Initialisation des filtres Doctrine pour les historique.
      * Objectif : laisser passer les enregistrements passés en historique pour mettre en évidence ensuite les erreurs
@@ -457,9 +456,11 @@ class PieceJointeController extends AbstractController
         //Récupérer tous les intervenants avec le même code intervenant
         $intervenants = $this->getServiceIntervenant()->getIntervenants($intervenant);
         //On recalcule le tbl piece_jointe pour tous les intervenants ayant le même code intervenant que l'intervenant de l'année en cours
+        foreach ($intervenants as $objectIntervenant) {
             $this->getServiceWorkflow()->calculerTableauxBord([
                 'piece_jointe',
-        ], $intervenants);
+            ], $objectIntervenant);
+        }
     }
 
 }

module/Application/src/Application/Service/SeuilChargeService.php

@@ -36,7 +36,6 @@ class SeuilChargeService extends AbstractEntityService
     use TableauBordServiceAwareTrait;
 
 
-
     /**
      * retourne la classe des entités
      *
@@ -142,28 +141,15 @@ class SeuilChargeService extends AbstractEntityService
      *
      * @param SeuilCharge $entity
      *
-     * @throws \RuntimeException
      * @return self
+     * @throws \RuntimeException
      */
     public function save($entity)
     {
         parent::save($entity);
 
-        $params = [
-            'TYPE_INTERVENTION_ID' => $entity->getTypeIntervention()->getId(),
-            'ANNEE_ID' => $entity->getAnnee()->getId(),
-        ];
-
-        if ($entity->getStructure()){
-            $params['STRUCTURE_ID'] = $entity->getStructure()->getId();
-        }
-
-        if ($entity->getGroupeTypeFormation()){
-            $params['GROUPE_TYPE_FORMATION_ID'] = $entity->getGroupeTypeFormation()->getId();
-        }
-
-        $this->getServiceTableauBord()->calculer('chargens_seuils_def', $params);
-        $this->getServiceTableauBord()->calculer('chargens', $params);
+        $this->getServiceTableauBord()->calculer('chargens_seuils_def', 'ANNEE_ID', $entity->getAnnee()->getId());
+        $this->getServiceTableauBord()->calculer('chargens', 'ANNEE_ID', $entity->getAnnee()->getId());
 
         return $entity;
     }
@@ -349,7 +335,7 @@ class SeuilChargeService extends AbstractEntityService
      */
     public function finderByContext(QueryBuilder $qb = null, $alias = null)
     {
-        list($qb, $alias) = $this->initQuery($qb, $alias);
+        [$qb, $alias] = $this->initQuery($qb, $alias);
 
         $this->finderByAnnee($this->getServiceContext()->getAnnee(), $qb);
         if ($cStructure = $this->getServiceContext()->getStructure()) {

module/Application/src/Application/Service/WorkflowService.php

@@ -330,7 +330,7 @@ class WorkflowService extends AbstractService
 
     /**
      * @param array           $tableauxBords
-     * @param Intervenant|Intervenant[]|string $intervenant
+     * @param Intervenant|int $intervenant
      */
     public function calculerTableauxBord($tableauxBords = [], $intervenant): array
     {
@@ -366,26 +366,14 @@ class WorkflowService extends AbstractService
 
         foreach ($deps as $dep => $null) {
             if (isset($tbls[$dep])) {
-                if (is_array($intervenant)) {
-                    $params = 'INTERVENANT_ID IN (';
-                    $c      = 0;
-                    foreach ($intervenant as $i) {
-                        $c++;
-                        if ($c > 1) $params .= ',';
-                        $params .= $i->getId();
-                    }
-                    $params .= ')';
-                } elseif ($intervenant instanceof \Application\Entity\Db\Intervenant) {
-                    $params = ['intervenant_id' => $intervenant->getId()];
+                if ($intervenant instanceof \Application\Entity\Db\Intervenant) {
+                    $value = $intervenant->getId();
                 } else {
-                    $params = 'INTERVENANT_ID IN (' . $intervenant . ')';
+                    $value = $intervenant;
                 }
 
                 try {
-                    $this->getServiceTableauBord()->calculer(
-                        $dep,
-                        $params
-                    );
+                    $this->getServiceTableauBord()->calculer($dep, 'INTERVENANT_ID', $value);
                 } catch (\Exception $e) {
                     $errors[$dep] = $e;
                 }