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,8 +156,10 @@ $form->runIf(f('add_cotis') && !$form->hasErrors(),
                  }
              }, 'add_cotis_2');
 
-if ($step)
+if (! $form->hasErrors())
 {
+    if ($step)
+    {
         try
         {
             $liste = $facture->getCotis((int)f('membre_cotis'));
@@ -162,9 +168,9 @@ if ($step)
         {
             $form->addError($e->getMessage());
         }
-}
-elseif (count($data) > 0)
-{
+    }
+    elseif (count($data) > 0)
+    {
         if ($target)
         {
             $id = $facture->add($data, $plugin->getConfig('pattern'));
@@ -178,6 +184,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)