Hi again!
I did a MVP and settled on a multi-tenancy option but I didn't go into every detail and need some help with this comment
Clients can be normal IdentityUser (AbpUser). They can access products without any issues.
What I need to know is how do I separate the Client (ordinary user) from Tenant/Host? How can I show clients their menu items that are totally different from the Tentan ones? I have been having this discussion on github but remembered that I had asked this question and gotten this comment and now beleve the question belongs here.
This is (I hope) the last puzzle to get my Beta out the door so I hope you can point me in the right direction with this.
p.s I have a separate Identity Servar and have added the Account source module so I can modify it.
Ok thanks to extra pair of eyes from Liangshiwei we spotted nuget package that needed to be replaced by source.
I reccomend that this documentation
"In this case, you typically add the source code of the module to your solution and replace package references by local project references. ABP CLI automates this process for you."
should be updated to
"In this case, you typically add the source code of the module to your solution and replace every package reference in the solution with its corresponding local project references. ABP CLI automates this process for you."
Also this part "ABP CLI automates this process for you." is not correct... the CLI only adds nuget package to (in my case) Volo.Abp.Account.* and adds some properties. It does not add the references to the source code. This part of the CLI could be much much more powerful and would probably save lot of wasted time.
Btw I created a PR to the docs
I whent over this 2x to day and re-read everything I could get my hands (and tried various other things just for sport) with the same result.
I sent you email and would like assistance with this, thank you.
Sorry stil morning here.. wasn´t going to close this so reopening it!
Thank you for the answer but I should maybe have skipped the part of getting the source and running the command because I have the exact same problem with clicking the "Replace packages with source code" button.
So my questions are
Sorry if I was unclear in my previous question.
Ok it works now. The email was checked with permissions but some combination of loging into the main admin and checking the tenant user loggin of and into the tenant it self and restart made this all click. I really tried this all yesterday as well... strange.... but oh well...closing this
You are correct @abpVAndy.. I wanted validation on my code setup when working with related entities and if my usage of navigation properties were correct. I found the given answer sourly lacking so I gave up asking this question and just kept my code as it is above, correct or not...
I started out trying to be a DDD/abp.io-framework purist but there some many things to learn there I can´t let it stop me, so I just plow ahead and try to read and re-read the docs.
At some point when I have got every thing running (my style) I´m thinking of paying for a 1-on-1 code-review and coaching. Thats why my code is now littered with //REVIEW: comments where I need future validation.
One thing I like to add is that I find the docs missing more complex real-world examples.
That is why I have been asking lot of these kind of questions so others might gain from them. It would fantastic if they spur somebody on to create some.
And please excuse me if I'm telling you something you already know! Regradless this might help somebody else.
They are basically something (foreign keys relationship/association) to connect entities together.
Lets say you have some XXX entity that has these navigation properies e.g. like this public Guid? SomeRelatedEntityId { get; set; } and you use GetAsync(id) method (on XXX) you will get every property set on XXX but you will only get a guid to SomeRelatedEntity and not is properties.
If you then wanted to update the values in that entity/table you will have to call GetAsync(someRelatedEntityId) on the SomeRelatedEntityRepository to get the properties you need to update.
With GetXXXWithNavigationProperties methods you will get XXX with SomeRelatedEntity with all its properies filled out and ready to be displayed/used. You don´t need to call again (note that now you are getting more data and maybe not just what you need).
And now you enter the crux of my question: In Create/Update methods of XXXEntity, is it considered OK to mix the domain objects by calling insert/update on all the navigation properties like this here below.
public async Task<MyEventDto> CreateAsync(MyEventCreateAndUpdateDto input)
{
var newMyEvent = new MyEvent();
// Create navigation entities and map input data to their respected properties
var common = new Common(GuidGenerator.Create()){ Name = input.Name, etc... }
var settings = new Settings(GuidGenerator.Create()){ /*set properties*/ }
var geoRestriction = new GeoRestriction(GuidGenerator.Create()){ /*set properties*/}
// Insert everything to their respected repositories
// First I thought that this was incorrect "mixing" domains,
// but the CreateAsync is a Service so the domains are not mixing..correct?
var _common = await _commonRepository.InsertAsync(common, autoSave: autoSave);
var _settings = await _settingsRepository.InsertAsync(settings, autoSave: autoSave);
var _geo = await _geoRestrictionRepository.InsertAsync(geo, autoSave: autoSave);
// "Connect" the inserted id's to MyEvent
newMyEvent.CommonId = _common.Id;
newMyEvent.SettingsId = _settings.Id;
newMyEvent.GeoRestrictionId = _geoRestriction.Id;
newMyEvent.TenantId = CurrentTenant.Id;
// Save everything connected.. if anything fails above UnitOfWork will see to it that nothing gets saved.
var myEvent = await _MyEventRepository.InsertAsync(newMyEvent);
// Returns MyEvent that only has ids.
// You call GetXXXWithProperties again in your UI/Client code to get them.
// Or you could create another CreateAsync method that returns another object with everything,
// but that ofcourse depends on your use case
return ObjectMapper.Map<MyEvent, MyEventDto>(myEvent);
}
Is this the correct approach? It feels little too manual.. Should I maybe inject ICommonAppService, ISettingsAppService etc. into MyEventService and use their service methods to insert/update (like this example of MyService using the BookAppService)? But then you would loose UnitOfWork correct?
Awesome thank you! My "search-fu" was way off! I wanted to check this out way before my deadline but this will be up after 2-3 weeks for me so I´ll give you feedback then. Thanks again!
The code above works but what is the correct approach?
So after finally getting some code to look at Easy CRM example I see that I´m probably doing this incorrectly.
Here is what I got from that code.
This is one-to-one relationship
Create a MyEvent entity with the desired relations in the Domain
public class MyEvent : FullAuditedAggregateRoot<Guid>
{
public virtual Guid? TenantId { get; set; }
public virtual string Name { get; set; }
public virtual Guid? CommonId { get; set; }
public virtual Common Common { get; set; }
public virtual Guid? SettingsId { get; set; }
public virtual Settings Settings { get; set; }
public virtual Guid? GeoRestrictionId { get; set; }
public virtual GeoRestriction GeoRestriction { get; set; }
public virtual ICollection<Stuff> Stuff { get; set; }
}
an just for completness I´ll show one related entity
public class Common : FullAuditedAggregateRoot<Guid>, IMultiTenant
{
public virtual Guid? TenantId { get; set; }
public virtual string Address { get; set; }
public virtual MyEvent MyEvent { get; set; }
}
then in Contracts we have these two dto's where one is lighter with only ids and the other extended one with the dto-entities I'm working with
public class MyEventDto : FullAuditedEntityDto<Guid>
{
public virtual string Name { get; set; }
public Guid CommonId { get; set; }
public Guid SettingsId { get; set; }
public Guid GeoRestrictionId { get; set; }
}
public class MyEventCreateAndUpdateExtendedDto : OrderDto
{
public CommonDto Common { get; set; }
public SettingsDto Settings { get; set; }
public GeoRestrictionDto GeoRestriction { get; set; }
public List<StuffDto> Stuffs { get; set; }
}
I ended up not needing this relationship setup, but I´ll keep it here anyways
builder.Entity<MyEvent>(b =>
{
b.ToTable(MyEventConsts.DbTablePrefix + "MyEvent", MyEventConsts.DbSchema);
b.ConfigureByConvention();
b.Property(x => x.Name).IsRequired();
//This was not needed (because of EF picked up conventions used)
//b.HasOne(x => x.Common).WithOne(x => x.MyEvents).HasForeignKey(x => x.CommonId).IsRequired();
// b.HasOne(x => x.Settings).WithOne(x => x.MyEvents).HasForeignKey(x => x.SettingsId);
//b.HasOne(x => x.GeoRestriction).WithOne(x => x.MyEvents).HasForeignKey(x => x.GeoRestrictionId);
// b.HasMany(x => x.Stuff).WithOne(x => x.MyEvent).HasForeignKey(x => x.MyEventId);
});
then create MyEvent by call this method in the AppService
[Authorize(MyPermissions.MyEvents.Create)]
public virtual async Task<MyEventDto> CreateAsync(MyEventCreateAndUpdateExtendedDto input)
{
var newMyEvent = ObjectMapper.Map<MyEventCreateAndUpdateExtendedDto, MyEvent>(input);
newMyEvent.TenantId = CurrentTenant.Id;
var common = new Common(GuidGenerator.Create()){ Name = input.Name, etc... }
var settings = new Settings(GuidGenerator.Create()){ /*set properties*/ }
var geoRestriction = new GeoRestriction(GuidGenerator.Create()){ /*set properties*/}
newMyEvent.Common = common;
newMyEvent.Settings = settings;
newMyEvent.GeoRestriction = geoRestriction;
var myEvent = await _MyEventRepository.InsertAsync(newMyEvent);
await CurrentUnitOfWork.SaveChangesAsync();
return ObjectMapper.Map<MyEvent, MyEventDto>(myEvent);
}
And now I can use GetWithNavigationPropertiesAsync(id) to get them all back.
So could anybody chime in and tell me **what is the best approach **and if I´m on the right track or not?
Sorry for the long rant.. but if nobody answers me I just need to work this out for my self and learn something and hopefully help the next poor bastard that is trying to get this all to click :-)
Or would it make more sence to use Domain Service here?
I can see the value in using the Domain Service if I had some business logic (maybe using specifications) so I would contain my business logic there but not in the AppService. So I still think that I should use it like I did in my question but I would like a confirmation
[Authorize(MyPermissions.MyEvent.Create)]
public virtual async Task<MyEventDto> CreateAsync(MyEventCreateDto input)
{
var myEvent = ObjectMapper.Map<MyEventCreateDto, MyEvent>(input);
// Call the manager
myEvent = await myEventManager.CreateAsync(myEvent,common,settings,geoRestriction, autoSave: true);
myEventManager.Update(myEvent);
return ObjectMapper.Map<MyEvent, MyEventDto>(myEvent);
}
public class MyEventManager : DomainService
{
private readonly IRepository<MyEvent, Guid> myEventRepository;
private readonly IRepository<Common, Guid> commonRepository;
private readonly IRepository<Settings , Guid> settingsRepository;
private readonly IRepository<GeoRestriction , Guid> geoRestrictionRepository;
public IssueManager(IRepository<MyEvent, Guid> myEventRepository, Common, Guid> commonRepository, etc)
{
this.myEventRepository = myEventRepository;
this.commonRepository = commonRepository;
//etc.
}
public async Task CreateAsync(MyEvent myEvent, Common common, Settings settings, GeoRestriction geo, bool autoSave)
{
var _common = await _commonRepository.InsertAsync(common, autoSave: autoSave);
var _settings = await _settingsRepository.InsertAsync(settings, autoSave: autoSave);
var _geo = await _geoRestrictionRepository.InsertAsync(geo, autoSave: autoSave);
myEvent.CommonId = _common.Id;
myEvent.SettingsId = _settings.Id;
myEvent.GeoRestrictionId = _geoRestriction.Id;
myEvent.TenantId = CurrentTenant.Id;
}
}
I'm just writing this all down so I can better grasp DDD and hopefully help somebody else when they go down this route so please bear with me :-)