Skip to content

Commit 43e20a1

Browse files
authored
Merge pull request #24517 from abpframework/auto-merge/rel-10-0/4230
Merge branch rel-10.1 with rel-10.0
2 parents 463ba78 + c162867 commit 43e20a1

File tree

3 files changed

+18
-9
lines changed

3 files changed

+18
-9
lines changed

framework/src/Volo.Abp.Features/Volo/Abp/Features/FeatureChecker.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,12 @@ public FeatureChecker(
2828

2929
public override async Task<string?> GetOrNullAsync(string name)
3030
{
31-
var featureDefinition = await FeatureDefinitionManager.GetAsync(name);
31+
var featureDefinition = await FeatureDefinitionManager.GetOrNullAsync(name);
32+
if (featureDefinition == null)
33+
{
34+
return null;
35+
}
36+
3237
var providers = FeatureValueProviderManager.ValueProviders
3338
.Reverse();
3439

framework/src/Volo.Abp.Settings/Volo/Abp/Settings/SettingProvider.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,12 @@ public SettingProvider(
2323

2424
public virtual async Task<string?> GetOrNullAsync(string name)
2525
{
26-
var setting = await SettingDefinitionManager.GetAsync(name);
26+
var setting = await SettingDefinitionManager.GetOrNullAsync(name);
27+
if (setting == null)
28+
{
29+
return null;
30+
}
31+
2732
var providers = Enumerable
2833
.Reverse(SettingValueProviderManager.Providers);
2934

modules/setting-management/test/Volo.Abp.SettingManagement.Tests/Volo/Abp/SettingManagement/SettingManager_Basic_Tests.cs

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,10 @@ public SettingManager_Basic_Tests()
1818
}
1919

2020
[Fact]
21-
public async Task Should_Throw_Exception_When_Try_To_Get_An_Undefined_Setting()
21+
public async Task Should_Return_Null_When_Try_To_Get_An_Undefined_Setting()
2222
{
23-
await Assert.ThrowsAsync<AbpException>(
24-
async () => await _settingProvider.GetOrNullAsync("UndefinedSetting")
25-
);
23+
var value = await _settingProvider.GetOrNullAsync("UndefinedSetting");
24+
value.ShouldBeNull();
2625
}
2726

2827
[Fact]
@@ -64,15 +63,15 @@ public async Task Should_Set_Global_Value()
6463
(await _settingManager.GetOrNullGlobalAsync("MySetting1")).ShouldBe("43");
6564
(await _settingProvider.GetOrNullAsync("MySetting1")).ShouldBe("43");
6665
}
67-
66+
6867
[Fact]
6968
public async Task Set_Should_Throw_Exception_If_Provider_Not_Found()
7069
{
7170
var exception = await Assert.ThrowsAsync<AbpException>(async () =>
7271
{
7372
await _settingManager.SetAsync("MySetting1", "43", "UndefinedProvider", "Test");
7473
});
75-
74+
7675
exception.Message.ShouldBe("Unknown setting value provider: UndefinedProvider");
7776
}
78-
}
77+
}

0 commit comments

Comments
 (0)