diff --git a/src/Umbraco.Tests/PluginManagerTests.cs b/src/Umbraco.Tests/PluginManagerTests.cs
index 3463af93b2..175271e64e 100644
--- a/src/Umbraco.Tests/PluginManagerTests.cs
+++ b/src/Umbraco.Tests/PluginManagerTests.cs
@@ -359,30 +359,7 @@ namespace Umbraco.Tests
Assert.IsNull(shouldNotFind);
}
-
- ///
- /// This demonstrates this issue: http://issues.umbraco.org/issue/U4-3505 - the TypeList was returning a list of assignable types
- /// not explicit types which is sort of ideal but is confusing so we'll do it the less confusing way.
- ///
- [Test]
- public void TypeList_Resolves_Explicit_Types()
- {
- var types = new HashSet();
-
- var propEditors = new PluginManager.TypeList(PluginManager.TypeResolutionKind.FindAllTypes);
- propEditors.AddType(typeof (LabelPropertyEditor));
- types.Add(propEditors);
-
- var found = types.SingleOrDefault(x => x.IsTypeList(PluginManager.TypeResolutionKind.FindAllTypes));
-
- Assert.IsNotNull(found);
-
- //This should not find a type list of this type
- var shouldNotFind = types.SingleOrDefault(x => x.IsTypeList(PluginManager.TypeResolutionKind.FindAllTypes));
-
- Assert.IsNull(shouldNotFind);
- }
-
+
[XsltExtension("Blah.Blah")]
public class MyXsltExtension
{
diff --git a/src/umbraco.providers/members/MembersMembershipProvider.cs b/src/umbraco.providers/members/MembersMembershipProvider.cs
index b8d207b9a8..45090a4acc 100644
--- a/src/umbraco.providers/members/MembersMembershipProvider.cs
+++ b/src/umbraco.providers/members/MembersMembershipProvider.cs
@@ -793,7 +793,7 @@ namespace umbraco.providers.members
}
//check for approve status. If not approved, then set the member property to null
- if (!CheckApproveStatus(m)) {
+ if (m != null && !CheckApproveStatus(m)) {
m = null;
}
@@ -859,6 +859,11 @@ namespace umbraco.providers.members
}
}
}
+ else
+ {
+ //There is no property so we shouldn't use the approve status
+ isApproved = true;
+ }
}
}
else {