yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Adds further validation to the Umbraco Member APIs to not allow a loginname as an empty string

Browse Source
This commit is contained in:
hartvig
2011-09-27 07:18:36 -02:00
parent ac8e9828e9
commit 300a31d6b0
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
umbraco/cms/businesslogic/member/Member.cs
View File

@@ -215,6 +215,10 @@ namespace umbraco.cms.businesslogic.member
public static Member MakeNew(string Name, string LoginName, string Email, MemberType mbt, User u)
{
var loginName = (!String.IsNullOrEmpty(LoginName)) ? LoginName : Name;
if (String.IsNullOrEmpty(loginName))
throw new ArgumentException("The loginname must be different from an empty string", "loginName");
// Test for e-mail
if (Email != "" && Member.GetMemberFromEmail(Email) != null)
throw new Exception(String.Format("Duplicate Email! A member with the e-mail {0} already exists", Email));
@@ -482,6 +486,8 @@ namespace umbraco.cms.businesslogic.member
}
set
{
if (String.IsNullOrEmpty(value))
throw new ArgumentException("The loginname must be different from an empty string", "LoginName");
if (value.Contains(","))
throw new ArgumentException("The parameter 'LoginName' must not contain commas.");
SqlHelper.ExecuteNonQuery(