From 57b516c368e95fd5014c18495e62a867ab318170 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Halil=20=C4=B0brahim=20Kalkan?= Date: Mon, 23 Jan 2017 13:48:54 +0300 Subject: [PATCH] Clear comments. --- src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs | 8 +++----- src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs | 2 -- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs index dc8cc0b067..72fe231174 100644 --- a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs +++ b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs @@ -1,6 +1,3 @@ -// Copyright (c) .NET Foundation. All rights reserved. -// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information. - using System; using System.Collections.Generic; using System.Collections.ObjectModel; @@ -11,7 +8,6 @@ using Volo.ExtensionMethods.Collections.Generic; namespace Volo.Abp.Identity { - //TODO: Should set Id to a GUID (where? on repository?) //TODO: Properties should not be public! /// @@ -47,11 +43,13 @@ namespace Volo.Abp.Identity /// /// Initializes a new instance of . /// + /// Role id. /// The role name. - public IdentityRole([NotNull] string name) + public IdentityRole(Guid id, [NotNull] string name) { Check.NotNull(name, nameof(name)); + Id = id; Name = name; } diff --git a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs index fe13fa2211..79858fdb7c 100644 --- a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs +++ b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs @@ -10,10 +10,8 @@ using Volo.ExtensionMethods.Collections.Generic; namespace Volo.Abp.Identity { - //TODO: Should set Id to a GUID (where? on repository?) //TODO: Properties should not be public! //TODO: Add Name/Surname/FullName? - //TODO: Set max lenght for Id (set a default max lenght by default, like 64) public class IdentityUser : AggregateRoot, IHasConcurrencyStamp {