[openads-Commits] r4649 - in trunk: . obj tests

[nme...@adullact.net]

Author: nmeucci
Date: 2016-06-28 16:29:01 +0200 (Tue, 28 Jun 2016)
New Revision: 4649

Modified:
trunk/
trunk/HISTORY.txt
trunk/obj/commission.class.php
trunk/tests/130_commissions.robot
Log:
* Correction : Lors des diffusions des ODJ et CR de commission, les documents ?\195?\169taient
syst?\195?\169matiquement cr?\195?\169?\195?\169s. Cela dupliquait les fichiers qui devenaientt orphelins.
Dor?\195?\169navant lorsque l'on rediffuse un ODJ ou un CR le document est mis ?\195?\160 jour.
Ticket #8259


Index: trunk
===================================================================
--- trunk 2016-06-28 14:24:05 UTC (rev 4648)
+++ trunk 2016-06-28 14:29:01 UTC (rev 4649)

Property changes on: trunk
___________________________________________________________________
Modified: svn:mergeinfo
## -2,6 +2,7 ##
/branches/amelioration_optimisation_tests:4083-4144
/branches/bug-merge-fields-correspondant:4125-4421
/branches/cerfa_sitadel_juin_2016:4486-4635
+/branches/commission_update_odj_cr:4574-4648
/branches/commissions-via-nouvelles-actions:4477-4553
/branches/evo-notification-pieces-numerisees-widget:4064-4137
/branches/evo_message:4346-4362
Modified: trunk/HISTORY.txt
===================================================================
--- trunk/HISTORY.txt 2016-06-28 14:24:05 UTC (rev 4648)
+++ trunk/HISTORY.txt 2016-06-28 14:29:01 UTC (rev 4649)
@@ -51,6 +51,10 @@
erreur 500 si l'option n'était pas configurée avec le message "Erreur interne".
Désormais, dans ce cas le retour est 200 OK et le message "L'option 'annuaire'
n'est pas configurée.".
+* Correction : Lors des diffusions des ODJ et CR de commission, les documents étaient
+ systématiquement créés. Cela dupliquait les fichiers qui devenaientt orphelins.
+ Dorénavant lorsque l'on rediffuse un ODJ ou un CR le document est mis à jour.
+ Ticket #8259


3.30.0 (08/06/2016)

Modified: trunk/obj/commission.class.php
===================================================================
--- trunk/obj/commission.class.php 2016-06-28 14:24:05 UTC (rev 4648)
+++ trunk/obj/commission.class.php 2016-06-28 14:29:01 UTC (rev 4649)
@@ -1030,8 +1030,17 @@
),
$this->getMetadata("om_fichier_commission_".$champ)
);
- // Stockage.
- $uid = $this->f->storage->create($pdf_result["pdf_output"], $metadata);
+ //Si le document a déjà été finalisé
+ //on met à jour le document mais pas son uid
+ if ( $this->getVal("om_final_commission_".$champ) != 'f' ){
+ $uid = $this->f->storage->update(
+ $this->getVal("om_fichier_commission_".$champ), $pdf_result["pdf_output"], $metadata);
+ }
+ //Sinon, on joute le document et on récupère son uid
+ else {
+ //Stockage du PDF
+ $uid = $this->f->storage->create($pdf_result["pdf_output"], $metadata);
+ }

// Si le document n'a pas pu être stocké
if ($uid == "" || $uid == 'OP_FAILURE') {

Modified: trunk/tests/130_commissions.robot
===================================================================
--- trunk/tests/130_commissions.robot 2016-06-28 14:24:05 UTC (rev 4648)
+++ trunk/tests/130_commissions.robot 2016-06-28 14:29:01 UTC (rev 4649)
@@ -335,7 +335,7 @@
##
Click On Form Portlet Action commission diffuser_ordre_jour
Wait Until Keyword Succeeds ${TIMEOUT} ${RETRY_INTERVAL} Valid Message Should Be La diffusion de l'ordre du jour s'est effectuée avec succès.
-
+ ${uid_odj} Get Value css=#om_fichier_commission_ordre_jour
##
Click On Form Portlet Action commission edition_ordre_jour
# On ouvre le PDF
@@ -348,7 +348,13 @@
# On ferme le PDF
Close PDF

+ # On vérifie que le l'ordre du jour est mis à jour apres redifusion
+ Click On Form Portlet Action commission diffuser_ordre_jour
+ Wait Until Keyword Succeeds ${TIMEOUT} ${RETRY_INTERVAL} Valid Message Should Be La diffusion de l'ordre du jour s'est effectuée avec succès.
+ ${uid_odj_2} Get Value css=#om_fichier_commission_ordre_jour
+ Should Be Equal ${uid_odj} ${uid_odj_2}

+
test_05_saisie_des_avis_par_la_cellule_suivi

# On se connecte en tant que "suivi"
@@ -389,7 +395,7 @@
##
Click On Form Portlet Action commission diffuser_compte_rendu
Wait Until Keyword Succeeds ${TIMEOUT} ${RETRY_INTERVAL} Valid Message Should Be La diffusion du compte-rendu s'est effectuée avec succès.
-
+ ${uid_cr} Get Value css=#om_fichier_commission_compte_rendu
##
Click On Form Portlet Action commission edition_compte_rendu
# On ouvre le PDF
@@ -402,7 +408,13 @@
# On ferme le PDF
Close PDF

+ # On vérifie que le compte rendu est mis à jour apres redifusion
+ Click On Form Portlet Action commission diffuser_compte_rendu
+ Wait Until Keyword Succeeds ${TIMEOUT} ${RETRY_INTERVAL} Valid Message Should Be La diffusion du compte-rendu s'est effectuée avec succès.
+ ${uid_cr_2} Get Value css=#om_fichier_commission_compte_rendu
+ Should Be Equal ${uid_cr} ${uid_cr_2}

+
test_06_gestion_du_retour_de_commission_par_l_instructeur

# On se connecte en tant que "instr"