Suppression code inutile

admin/_facture_common.php

@@ -51,7 +51,7 @@ if ( !$target ) {
 // Traitement
 $data=[];
 $form->runIf(f('save') && !$form->hasErrors(),
-             function () use ($client, &$data)
+             function () use ($client, &$data, $form)
              {
                  try
                  {
@@ -101,6 +101,10 @@ $form->runIf(f('save') && !$form->hasErrors(),
                          $data['receveur_membre'] = 1;
                          $data['receveur_id'] = f('membre');
                      }
+                     else
+                     {
+                         throw new UserException('Vous devez indiquer si le receveur est un client ou un membre');
+                     }
 
                  }
                  catch(UserException $e)
@@ -152,6 +156,8 @@ $form->runIf(f('add_cotis') && !$form->hasErrors(),
                  }
              }, 'add_cotis_2');
 
+if (! $form->hasErrors())
+{
     if ($step)
     {
         try
@@ -179,6 +185,7 @@ elseif (count($data) > 0)
             throw new UserException('Erreur d\'édition du reçu');
         }
     }
+}
 
 // Affichage
 

admin/pdf.php

@@ -8,7 +8,6 @@ $users = new Users\Users;
 
 f(['id' => 'required|numeric']);
 $id = (int) qg('id');
-$id_field = Users\DynamicFields::getNameFieldsSQL();
 $sign_tag = UserTemplate\Functions::signature();
 
 // Vérification que le document existe
@@ -25,7 +24,6 @@ try
 	if ($f->receveur_membre)
 	{
 		$c = $users->get($f->receveur_id);
-		$c->$id_field = $c->$identite;
 		foreach(['ville','code_postal','adresse'] as $v)
 		{
 			if($c->$v == '')
@@ -37,7 +35,6 @@ try
 	else
 	{
 		$c = $client->get($f->receveur_id);
-		$c->$id_field = $c->nom;
 	}
 }
 catch(UserException $e)