From 16e2890da80098bbb67378d8308e7637b0d52317 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Mon, 20 Apr 2009 15:30:38 +0000
Subject: [PATCH] Add protection if login not provided

---
 htdocs/adherents/adherent.class.php | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/htdocs/adherents/adherent.class.php b/htdocs/adherents/adherent.class.php
index 193356b527d..7708318e6b8 100644
--- a/htdocs/adherents/adherent.class.php
+++ b/htdocs/adherents/adherent.class.php
@@ -262,10 +262,16 @@ class Adherent extends CommonObject
 		// Check parameters
 		if ($conf->global->ADHERENT_MAIL_REQUIRED && ! isValidEMail($this->email))
 		{
+			$langs->load("companies");
 			$this->error = $langs->trans("ErrorBadEMail",$this->email);
 			return -1;
 		}
 		if (! $this->datec) $this->datec=gmmktime();
+		if (empty($this->login))
+		{
+			$this->error = $langs->trans("ErrorWrongValueForParameterX","Login");
+			return -1;
+		}
 
 		$this->db->begin();
 
@@ -275,7 +281,7 @@ class Adherent extends CommonObject
 		$sql.= " VALUES (";
 		$sql.= " '".$this->db->idate($this->datec)."',";
 		$sql.= " '".$this->login."',";
-		$sql.= " ".($user->id>0?$user->id:"null").",";	// Can be null because member can be create by a guest
+		$sql.= " ".($user->id>0?$user->id:"null").",";	// Can be null because member can be create by a guest or a script
 		$sql.= " null,null,'".$this->morphy."',";
 		$sql.= " '".$this->typeid."'";
 		$sql.= ")";