Avoid NullReferenceException if context is null

[1d0n7kn0w]

No description provided.

[maliming]

hi @1d0n7kn0w
DataSeedContext will not be null by default. Can you explain?

[1d0n7kn0w]

Hi @maliming
I created a new solution, then I deleted all the things related to multitenancy
So in the class "src/{name}.Domain/Data/{name}DbMigrationService.cs" and specifically in the private method "SeedDataAsync()" I made this change from "await _dataSeeder.SeedAsync(tenant?.Id);" to "await _dataSeeder.SeedAsync(null);"
At first, I didn't know why the DbMigrator was throwing a NullReferenceException, after investigation with null "Volo.Abp.Data.IDataSeeder.SeedAsync(DataSeedContext context)" is used, so to avoid this I had to use the parameterless one so it uses "Volo.Abp.Data.DataSeederExtensions.SeedAsync(this IDataSeeder seeder, Guid? tenantId = null)"

[maliming]

hi

You can use this extension method instead of passing null.

abp/framework/src/Volo.Abp.Data/Volo/Abp/Data/DataSeederExtensions.cs

Line 8 in 42f37c5

public static Task SeedAsync(this IDataSeeder seeder, Guid? tenantId = null)

[1d0n7kn0w]

yes that's what I did as I mentioned in my comment, but to simplify things for other developers, passing null or nothing should have the same result I guess

[maliming]

Ok, can you also handle IdentityDataSeedContributor?

abp/modules/identity/src/Volo.Abp.Identity.Domain/Volo/Abp/Identity/IdentityDataSeedContributor.cs

Line 16 in ace4b55

public virtual Task SeedAsync(DataSeedContext context)

[1d0n7kn0w]

Done, with pleasure 😄