From c2a858b90348f170163fac97c7b1f5dd67bbaab4 Mon Sep 17 00:00:00 2001 From: Shannon Date: Wed, 20 Nov 2013 08:55:09 +1100 Subject: [PATCH] Ensures approval status is checked when validating a member --- .../members/MembersMembershipProvider.cs | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/umbraco.providers/members/MembersMembershipProvider.cs b/src/umbraco.providers/members/MembersMembershipProvider.cs index f6103cc936..aa51e6276c 100644 --- a/src/umbraco.providers/members/MembersMembershipProvider.cs +++ b/src/umbraco.providers/members/MembersMembershipProvider.cs @@ -793,12 +793,10 @@ namespace umbraco.providers.members } } - // TODO: Make approving optional for all member types, forcing it on is not good - - // check for approve status. If not approved, then set the member property to null - //if (!CheckApproveStatus(m)) { - // m = null; - //} + //check for approve status. If not approved, then set the member property to null + if (!CheckApproveStatus(m)) { + m = null; + } // maybe update login date if (m != null && string.IsNullOrEmpty(_lastLoginPropertyTypeAlias) == false)