From 32f916e9085bf47560d2b765be2be499015f5983 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@destailleur.fr>
Date: Tue, 9 Apr 2013 16:06:00 +0200
Subject: [PATCH] Qual: Add a test on not valid parameters
---
.../members/sync_members_ldap2dolibarr.php | 21 ++++++++++++-------
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/scripts/members/sync_members_ldap2dolibarr.php b/scripts/members/sync_members_ldap2dolibarr.php
index d29992a21db..6c72556f89f 100755
--- a/scripts/members/sync_members_ldap2dolibarr.php
+++ b/scripts/members/sync_members_ldap2dolibarr.php
@@ -116,19 +116,26 @@ print "----- Options:\n";
print "commitiferror=".$forcecommit."\n";
print "Mapped LDAP fields=".join(',',$required_fields)."\n";
print "\n";
+
+// Check parameters
+if (empty($conf->global->LDAP_MEMBER_DN))
+{
+ print $langs->trans("Error").': '.$langs->trans("LDAP setup for members not defined inside Dolibarr")."\n";
+ exit(1);
+}
+if ($typeid <= 0)
+{
+ print $langs->trans("Error").': Parameter id_member_type is not a valid ref of an existing member type'."\n";
+ exit(2);
+}
+
+
print "Press a key to confirm...";
$input = trim(fgets(STDIN));
print "Hit Enter to continue or CTRL+C to stop...\n";
$input = trim(fgets(STDIN));
-if (empty($conf->global->LDAP_MEMBER_DN))
-{
- print $langs->trans("Error").': '.$langs->trans("LDAP setup for members not defined inside Dolibarr");
- exit(1);
-}
-
-
// Charge tableau de correspondance des pays
$hashlib2rowid=array();
$countries=array();
--
GitLab