admin/_facture_common.php

@@ -51,7 +51,7 @@ if ( !$target ) {
 // Traitement
 $data=[];
 $form->runIf(f('save') && !$form->hasErrors(),
-             function () use ($client, &$data, $form)
+             function () use ($client, &$data)
              {
                  try
                  {
@@ -101,10 +101,6 @@ $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)
@@ -156,36 +152,33 @@ $form->runIf(f('add_cotis') && !$form->hasErrors(),
                  }
              }, 'add_cotis_2');
 
-if (! $form->hasErrors())
+if ($step)
 {
-    if ($step)
+    try
     {
-        try
-        {
-            $liste = $facture->getCotis((int)f('membre_cotis'));
-        }
-        catch (UserException $e)
-        {
-            $form->addError($e->getMessage());
-        }
+        $liste = $facture->getCotis((int)f('membre_cotis'));
     }
-    elseif (count($data) > 0)
+    catch (UserException $e)
     {
-        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');
-        }
+        $form->addError($e->getMessage());
     }
 }
+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,6 +8,7 @@ $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
@@ -24,6 +25,7 @@ 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 == '')
@@ -35,6 +37,7 @@ try
 	else
 	{
 		$c = $client->get($f->receveur_id);
+		$c->$id_field = $c->nom;
 	}
 }
 catch(UserException $e)