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,33 +156,36 @@ $form->runIf(f('add_cotis') && !$form->hasErrors(),
                  }
              }, 'add_cotis_2');
 
-if ($step)
+if (! $form->hasErrors())
 {
-    try
+    if ($step)
     {
-        $liste = $facture->getCotis((int)f('membre_cotis'));
+        try
+        {
+            $liste = $facture->getCotis((int)f('membre_cotis'));
+        }
+        catch (UserException $e)
+        {
+            $form->addError($e->getMessage());
+        }
     }
-    catch (UserException $e)
+    elseif (count($data) > 0)
     {
-        $form->addError($e->getMessage());
+        if ($target)
+        {
+            $id = $facture->add($data, $plugin->getConfig('pattern'));
+            Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
+        }
+        else
+        {
+            if ($facture->edit($id, $data))
+            {
+                Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
+            }
+            throw new UserException('Erreur d\'édition du reçu');
+        }
     }
 }
-elseif (count($data) > 0)
-{
-	if ($target)
-	{
-		$id = $facture->add($data, $plugin->getConfig('pattern'));
-		Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
-	}
-	else
-	{
-		if ($facture->edit($id, $data))
-		{
-			Utils::redirect(PLUGIN_ADMIN_URL  . 'facture.php?id='.(int)$id);
-		}
-		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)