diff --git a/src/Umbraco.Core/Services/UserService.cs b/src/Umbraco.Core/Services/UserService.cs index 74002646da..55d7dd9742 100644 --- a/src/Umbraco.Core/Services/UserService.cs +++ b/src/Umbraco.Core/Services/UserService.cs @@ -146,6 +146,9 @@ namespace Umbraco.Core.Services IsLockedOut = false, IsApproved = true }; + //adding default sections content and media + user.AddAllowedSection("content"); + user.AddAllowedSection("media"); if (SavingUser.IsRaisedEventCancelled(new SaveEventArgs(user), this)) return user; diff --git a/src/Umbraco.Tests/Services/UserServiceTests.cs b/src/Umbraco.Tests/Services/UserServiceTests.cs index f3100aa133..c7002ce79d 100644 --- a/src/Umbraco.Tests/Services/UserServiceTests.cs +++ b/src/Umbraco.Tests/Services/UserServiceTests.cs @@ -398,6 +398,10 @@ namespace Umbraco.Tests.Services var user1 = ServiceContext.UserService.CreateUserWithIdentity("test1", "test1@test.com", userType); + var result1 = ServiceContext.UserService.GetUserById((int)user1.Id); + //expect 2 sections by default + Assert.AreEqual(2, result1.AllowedSections.Count()); + //adds some allowed sections user1.AddAllowedSection("test1"); user1.AddAllowedSection("test2"); @@ -405,8 +409,9 @@ namespace Umbraco.Tests.Services user1.AddAllowedSection("test4"); ServiceContext.UserService.Save(user1); - var result1 = ServiceContext.UserService.GetUserById((int)user1.Id); - Assert.AreEqual(4, result1.AllowedSections.Count()); + result1 = ServiceContext.UserService.GetUserById((int)user1.Id); + //expect 6 sections including the two default sections + Assert.AreEqual(6, result1.AllowedSections.Count()); //simulate clearing the sections foreach (var s in user1.AllowedSections) @@ -419,11 +424,9 @@ namespace Umbraco.Tests.Services ServiceContext.UserService.Save(result1); //assert - //re-get result1 = ServiceContext.UserService.GetUserById((int)user1.Id); Assert.AreEqual(2, result1.AllowedSections.Count()); - } [Test] @@ -448,7 +451,6 @@ namespace Umbraco.Tests.Services var result2 = ServiceContext.UserService.GetUserById((int)user2.Id); Assert.IsFalse(result1.AllowedSections.Contains("test")); Assert.IsFalse(result2.AllowedSections.Contains("test")); - } [Test] @@ -546,7 +548,7 @@ namespace Umbraco.Tests.Services Assert.That(updatedItem.StartMediaId, Is.EqualTo(originalUser.StartMediaId)); Assert.That(updatedItem.Email, Is.EqualTo(originalUser.Email)); Assert.That(updatedItem.Username, Is.EqualTo(originalUser.Username)); - Assert.That(updatedItem.AllowedSections.Count(), Is.EqualTo(0)); + Assert.That(updatedItem.AllowedSections.Count(), Is.EqualTo(2)); } } }