Added todo and renamed MemoryDatabase._idGenerators

src/Volo.Abp.Identity.Web/Navigation/AbpIdentityWebMainMenuContributor.cs

@@ -16,6 +16,7 @@ namespace Volo.Abp.Identity.Web.Navigation
 
             var permissionChecker = context.ServiceProvider.GetRequiredService<IPermissionChecker>();
 
+            //TODO: We should not add this if none of the children available. Alternatively, we can handle it on menu rendering (would be simpler).
             var identityMenuItem = new ApplicationMenuItem("Identity", "Identity");
 
             context.Menu.AddItem(identityMenuItem);

src/Volo.Abp.MemoryDb/Volo/Abp/Domain/Repositories/MemoryDb/MemoryDatabase.cs

@@ -8,12 +8,12 @@ namespace Volo.Abp.Domain.Repositories.MemoryDb
     {
         private readonly ConcurrentDictionary<Type, object> _sets;
 
-        private readonly ConcurrentDictionary<Type, InMemoryIdGenerator> _idGenerators;
+        private readonly ConcurrentDictionary<Type, InMemoryIdGenerator> _entityIdGenerators;
 
         public MemoryDatabase()
         {
             _sets = new ConcurrentDictionary<Type, object>();
-            _idGenerators = new ConcurrentDictionary<Type, InMemoryIdGenerator>();
+            _entityIdGenerators = new ConcurrentDictionary<Type, InMemoryIdGenerator>();
         }
 
         public List<TEntity> Collection<TEntity>()
@@ -23,7 +23,7 @@ namespace Volo.Abp.Domain.Repositories.MemoryDb
 
         public TKey GenerateNextId<TEntity, TKey>()
         {
-            return _idGenerators
+            return _entityIdGenerators
                 .GetOrAdd(typeof(TEntity), () => new InMemoryIdGenerator())
                 .GenerateNext<TKey>();
         }