diff --git a/htdocs/adherents/card.php b/htdocs/adherents/card.php
index fc019ae53f5..b33e03d5855 100644
--- a/htdocs/adherents/card.php
+++ b/htdocs/adherents/card.php
@@ -1821,6 +1821,14 @@ if (is_object($objcanvas) && $objcanvas->displayCanvasExists($action)) {
$company = new Societe($db);
$result = $company->fetch($object->socid);
print $company->getNomUrl(1);
+
+ // Show link to invoices
+ $tmparray = $company->getOutstandingBills('customer');
+ if (!empty($tmparray['refs'])) {
+ print ' - '.img_picto($langs->trans("Invoices"), 'bill', 'class="paddingright"').''.$langs->trans("Invoices").': '.count($tmparray['refs']);
+ // TODO Add alert if warning on at least one invoice late
+ print '';
+ }
} else {
print ''.$langs->trans("NoThirdPartyAssociatedToMember").'';
}
diff --git a/htdocs/adherents/subscription.php b/htdocs/adherents/subscription.php
index 2ea8c62dd03..3a967677e17 100644
--- a/htdocs/adherents/subscription.php
+++ b/htdocs/adherents/subscription.php
@@ -260,7 +260,7 @@ if ($user->rights->adherent->cotisation->creer && $action == 'subscription' && !
}
// Check if a payment is mandatory or not
- if (!$error && $adht->subscription) { // Member type need subscriptions
+ if ($adht->subscription) { // Member type need subscriptions
if (!is_numeric($amount)) {
// If field is '' or not a numeric value
$errmsg = $langs->trans("ErrorFieldRequired", $langs->transnoentities("Amount"));
@@ -273,24 +273,30 @@ if ($user->rights->adherent->cotisation->creer && $action == 'subscription' && !
if (GETPOST("subscription")) {
if (!GETPOST("label")) {
$errmsg = $langs->trans("ErrorFieldRequired", $langs->transnoentities("Label"));
+ setEventMessages($errmsg, null, 'errors');
+ $error++;
+ $action = 'addsubscription';
}
if (GETPOST("paymentsave") != 'invoiceonly' && !GETPOST("operation")) {
$errmsg = $langs->trans("ErrorFieldRequired", $langs->transnoentities("PaymentMode"));
+ setEventMessages($errmsg, null, 'errors');
+ $error++;
+ $action = 'addsubscription';
}
if (GETPOST("paymentsave") != 'invoiceonly' && !(GETPOST("accountid", 'int') > 0)) {
$errmsg = $langs->trans("ErrorFieldRequired", $langs->transnoentities("FinancialAccount"));
+ setEventMessages($errmsg, null, 'errors');
+ $error++;
+ $action = 'addsubscription';
}
} else {
if (GETPOST("accountid", 'int')) {
$errmsg = $langs->trans("ErrorDoNotProvideAccountsIfNullAmount");
+ setEventMessages($errmsg, null, 'errors');
+ $error++;
+ $action = 'addsubscription';
}
}
- if ($errmsg) {
- $error++;
- setEventMessages($errmsg, null, 'errors');
- $error++;
- $action = 'addsubscription';
- }
}
}
}
@@ -602,6 +608,14 @@ if ($rowid > 0) {
$company = new Societe($db);
$result = $company->fetch($object->fk_soc);
print $company->getNomUrl(1);
+
+ // Show link to invoices
+ $tmparray = $company->getOutstandingBills('customer');
+ if (!empty($tmparray['refs'])) {
+ print ' - '.img_picto($langs->trans("Invoices"), 'bill', 'class="paddingright"').''.$langs->trans("Invoices").': '.count($tmparray['refs']);
+ // TODO Add alert if warning on at least one invoice late
+ print '';
+ }
} else {
print ''.$langs->trans("NoThirdPartyAssociatedToMember").'';
}