From 38635485cd8e99943fb1f202bc28c9cd2f856750 Mon Sep 17 00:00:00 2001 From: Antony Le Courtes <antony.lecourtes@unicaen.fr> Date: Mon, 18 Oct 2021 10:38:25 +0200 Subject: [PATCH] =?UTF-8?q?Message=20pour=20obliger=20=C3=A0=20renseigner?= =?UTF-8?q?=20l'affectation=20et=20le=20type=20d'emploi=20lors=20d'un=20re?= =?UTF-8?q?nouvellement=20ou=20d'une=20prise=20en=20charge?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../src/Connecteur/Siham/SihamConnecteur.php | 2 +- .../src/Controller/ExportRhController.php | 37 +++++++++++-------- .../view/export-rh/export-rh/exporter.phtml | 28 ++++++++++++-- 3 files changed, 47 insertions(+), 20 deletions(-) diff --git a/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php b/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php index c263c2e0fb..c4494db561 100644 --- a/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php +++ b/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php @@ -583,7 +583,7 @@ class SihamConnecteur implements ConnecteurRhInterface 'listeCarriere' => $carriere, 'listeModalitesServices' => $service, 'listeStatuts' => $statut, - 'listeContrats' => $contrat, + //'listeContrats' => $contrat, 'listePositions' => $position, 'motifEntree' => 'REN', 'matricule' => $matricule, diff --git a/module/ExportRh/src/Controller/ExportRhController.php b/module/ExportRh/src/Controller/ExportRhController.php index 42cdd9b529..b2fc659af6 100644 --- a/module/ExportRh/src/Controller/ExportRhController.php +++ b/module/ExportRh/src/Controller/ExportRhController.php @@ -136,7 +136,7 @@ class ExportRhController extends AbstractController //On regarde si il a une affectation en cours pour l'année courante si oui alors on propose uniquement une synchronisation des données personnelles $affectationEnCours = current($this->exportRhService->getAffectationEnCoursIntervenantRh($intervenant)); $contratEnCours = current($this->exportRhService->getContratEnCoursIntervenantRh($intervenant)); - + $renouvellement = true; if (!empty($affectationEnCours)) { $renouvellement = false; @@ -185,9 +185,7 @@ class ExportRhController extends AbstractController throw new \LogicException('Intervenant non précisé ou inexistant'); } - $posts = $this->getRequest()->getPost(); - - + $posts = $this->getRequest()->getPost(); $result = $this->exportRhService->priseEnChargeIntrervenantRh($intervenant, $posts); if ($result !== false) { @@ -216,18 +214,25 @@ class ExportRhController extends AbstractController if (!$intervenant) { throw new \LogicException('Intervenant non précisé ou inexistant'); } - - $dateExport = new \DateTime(); - $intervenant->setExportDate($dateExport); - $this->getServiceIntervenant()->save($intervenant); - $posts = $this->getRequest()->getPost(); - $result = $this->exportRhService->renouvellementIntervenantRh($intervenant, $posts); - if ($result !== false) { - $this->exportRhService->cloreDossier($intervenant); - $this->flashMessenger()->addSuccessMessage('Le renouvellement s\'est déroulé avec succés et le dossier a été cloturé'); - $this->getServiceIntervenant()->updateExportDate($intervenant); - } else { - $this->flashMessenger()->addErrorMessage('Un problème est survenu lors de la tentative de renouvellement de l\'intervenant'); + $posts = $this->getRequest()->getPost(); + $missingArgument = 0; + if (empty($posts['connecteurForm']['affectation'])) { + $this->flashMessenger()->addErrorMessage('Vous n\'avez pas choisi d\'affectation pour l\'agent'); + $missingArgument++; + } + if (empty($posts['connecteurForm']['emploi'])) { + $this->flashMessenger()->addErrorMessage('Vous n\'avez pas choisi de type d\'emploi pour l\'agent'); + $missingArgument++; + } + if ($missingArgument == 0) { + $result = $this->exportRhService->renouvellementIntervenantRh($intervenant, $posts); + if ($result !== false) { + $this->exportRhService->cloreDossier($intervenant); + $this->flashMessenger()->addSuccessMessage('Le renouvellement s\'est déroulé avec succés et le dossier a été cloturé'); + $this->getServiceIntervenant()->updateExportDate($intervenant); + } else { + $this->flashMessenger()->addErrorMessage('Un problème est survenu lors de la tentative de renouvellement de l\'intervenant'); + } } } } catch (\Exception $e) { diff --git a/module/ExportRh/view/export-rh/export-rh/exporter.phtml b/module/ExportRh/view/export-rh/export-rh/exporter.phtml index 31f6b303d1..516696e0f1 100755 --- a/module/ExportRh/view/export-rh/export-rh/exporter.phtml +++ b/module/ExportRh/view/export-rh/export-rh/exporter.phtml @@ -183,7 +183,7 @@ if ($priseEnCharge) { </form> <p> - <button onclick="$('.form-export-rh').submit();" class="btn btn-primary">Prendre en charge l'intervenant dans <?= $nameConnecteur; ?></button> + <button id="sihamButton" class="btn btn-primary">Prendre en charge l'intervenant dans <?= $nameConnecteur; ?></button> </p> <?php elseif ($renouvellement): ?> <!--Partie générique du formulaire--> @@ -196,7 +196,7 @@ if ($priseEnCharge) { ?> </form> <p> - <button onclick="$('.form-export-rh').submit();" class="btn btn-primary">Renouveller l'intervenant dans <?= $nameConnecteur; ?></button> + <button id="sihamButton" class="btn btn-primary">Renouveller l'intervenant dans <?= $nameConnecteur; ?></button> </p> <?php else: ?> <!--Partie générique du formulaire--> @@ -205,7 +205,7 @@ if ($priseEnCharge) { ?> </form> <p> - <button onclick="$('.form-export-rh').submit();" class="btn btn-primary">Synchroniser les données personnelles dans <?= $nameConnecteur; ?></button> + <button id="sihamButton" class="btn btn-primary">Synchroniser les données personnelles dans <?= $nameConnecteur; ?></button> </p> <?php endif; ?> <?php endif; ?> @@ -215,5 +215,27 @@ if ($priseEnCharge) { <?php else: ?> <div class="alert alert-warning">Le module export RH n'est disponible que pour l'année universitaire en cours.</div> <?php endif; ?> +<script type="text/javascript"> + $("document").ready(function () + { + + $("#sihamButton").click(function () { + + var emploi = $("select[name='connecteurForm[emploi]']"); + var affectation = $("select[name='connecteurForm[affectation]']"); + + if (emploi.val() == '' || affectation.val() == '') { + alert("Vous devez obligatoirement renseigner une affectation et un type d'emploi pour un renouvellement ou une prise en charge dans SIHAM") + } else { + $('.form-export-rh').submit(); + } + + }); + }); + + +</script> + + -- GitLab