From 59fc9bfe86ce531574b4d1ee12123f6579690c68 Mon Sep 17 00:00:00 2001
From: Antony Le Courtes <antony.lecourtes@unicaen.fr>
Date: Tue, 20 Jul 2021 15:52:50 +0200
Subject: [PATCH] Corrections
---
.../src/Connecteur/Siham/SihamConnecteur.php | 8 ++++----
.../ExportRh/src/Controller/ExportRhController.php | 12 ++++++------
.../view/export-rh/export-rh/exporter.phtml | 14 +++++++-------
3 files changed, 17 insertions(+), 17 deletions(-)
diff --git a/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php b/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php
index 365d7c3104..98906c4184 100644
--- a/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php
+++ b/module/ExportRh/src/Connecteur/Siham/SihamConnecteur.php
@@ -164,7 +164,7 @@ class SihamConnecteur implements ConnecteurRhInterface
$dossierIntervenant = $this->getServiceDossier()->getByIntervenant($intervenant);
//Synchronisation Tel pro
- if ($datas['generiqueFieldset']['telPro']) {
+ if ($datas['generiqueFieldset']['telPro'] && !empty($dossierIntervenant->getTelPro())) {
$params = [
'matricule' => $intervenantRh->getCodeRh(),
'numero' => $dossierIntervenant->getTelPro(),
@@ -175,7 +175,7 @@ class SihamConnecteur implements ConnecteurRhInterface
}
//Synchronisation Tel perso
- if ($datas['generiqueFieldset']['telPerso']) {
+ if ($datas['generiqueFieldset']['telPerso'] && !empty($dossierIntervenant->getTelPerso())) {
$params = [
'matricule' => $intervenantRh->getCodeRh(),
'numero' => $dossierIntervenant->getTelPerso(),
@@ -186,7 +186,7 @@ class SihamConnecteur implements ConnecteurRhInterface
}
//Synchronisation email pro
- if ($datas['generiqueFieldset']['emailPro']) {
+ if ($datas['generiqueFieldset']['emailPro'] && !empty($dossierIntervenant->getEmailPro())) {
$params = [
'matricule' => $intervenantRh->getCodeRh(),
'numero' => $dossierIntervenant->getEmailPro(),
@@ -197,7 +197,7 @@ class SihamConnecteur implements ConnecteurRhInterface
}
//Synchronisation email perso
- if ($datas['generiqueFieldset']['emailPerso']) {
+ if ($datas['generiqueFieldset']['emailPerso'] && !empty($dossierIntervenant->getEmailPerso())) {
$params = [
'matricule' => $intervenantRh->getCodeRh(),
'numero' => $dossierIntervenant->getEmailPerso(),
diff --git a/module/ExportRh/src/Controller/ExportRhController.php b/module/ExportRh/src/Controller/ExportRhController.php
index 36a5583026..46a2d0930d 100644
--- a/module/ExportRh/src/Controller/ExportRhController.php
+++ b/module/ExportRh/src/Controller/ExportRhController.php
@@ -74,12 +74,12 @@ class ExportRhController extends AbstractController
{
/* Initialisation */
- $role = $this->getServiceContext()->getSelectedIdentityRole();
- $intervenant = $role->getIntervenant() ?: $this->getEvent()->getParam('intervenant');
- $intervenantRh = '';
- $form = '';
- $nameConnecteur = '';
- $affectationFonctionnelle = '';
+ $role = $this->getServiceContext()->getSelectedIdentityRole();
+ $intervenant = $role->getIntervenant() ?: $this->getEvent()->getParam('intervenant');
+ $intervenantRh = '';
+ $form = '';
+ $nameConnecteur = '';
+ $affectationEnCours = '';
if (!$intervenant) {
throw new \LogicException('Intervenant non précisé ou inexistant');
diff --git a/module/ExportRh/view/export-rh/export-rh/exporter.phtml b/module/ExportRh/view/export-rh/export-rh/exporter.phtml
index 74ef9303d7..c89b43fecd 100755
--- a/module/ExportRh/view/export-rh/export-rh/exporter.phtml
+++ b/module/ExportRh/view/export-rh/export-rh/exporter.phtml
@@ -132,7 +132,7 @@ if ($priseEnCharge) {
</td>
<td><?= $intervenantDossier->getNomUsuel() ?></td>
- <td class="active"><?= $intervenantRh->getNomUsuel() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getNomUsuel() : '' ?></td>
</tr>
<tr>
<td>
@@ -140,7 +140,7 @@ if ($priseEnCharge) {
<?= $this->formLabel($generiqueFieldset->get('telPro')) ?> :
</td>
<td><?= $intervenantDossier->getTelPro() ?></td>
- <td class="active"><?= $intervenantRh->getTelPro() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getTelPro() : '' ?></td>
</tr>
<tr>
<td>
@@ -148,7 +148,7 @@ if ($priseEnCharge) {
<?= $this->formLabel($generiqueFieldset->get('telPerso')) ?> :
</td>
<td><?= $intervenantDossier->getTelPerso() ?></td>
- <td class="active"><?= $intervenantRh->getTelPerso() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getTelPerso() : '' ?></td>
</tr>
<tr>
<td>
@@ -156,7 +156,7 @@ if ($priseEnCharge) {
<?= $this->formLabel($generiqueFieldset->get('emailPro')) ?> :
</td>
<td><?= $intervenantDossier->getEmailPro() ?></td>
- <td class="active"><?= $intervenantRh->getEmailPro() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getEmailPro() : '' ?></td>
</tr>
<tr>
<td>
@@ -164,7 +164,7 @@ if ($priseEnCharge) {
<?= $this->formLabel($generiqueFieldset->get('emailPerso')) ?> :
</td>
<td><?= $intervenantDossier->getEmailPerso() ?></td>
- <td class="active"><?= $intervenantRh->getEmailPerso() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getEmailPerso() : '' ?></td>
</tr>
<tr>
@@ -173,7 +173,7 @@ if ($priseEnCharge) {
<?= $this->formLabel($generiqueFieldset->get('iban')) ?> :
</td>
<td><?= $intervenantDossier->getIBAN() ?></td>
- <td class="active"><?= $intervenantRh->getIBAN() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getIBAN() : '' ?></td>
</tr>
<tr>
<td>
@@ -181,7 +181,7 @@ if ($priseEnCharge) {
<?= $this->formLabel($generiqueFieldset->get('bic')) ?> :
</td>
<td><?= $intervenantDossier->getBIC() ?></td>
- <td class="active"><?= $intervenantRh->getBIC() ?></td>
+ <td class="active"><?= (!empty($intervenantRh)) ? $intervenantRh->getBIC() : '' ?></td>
</tr>
</tbody>
</table>
--
GitLab