From 1167c78acd28f70eaaa6ddb8a5da7b61030db5d1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Halil=20=C4=B0brahim=20Kalkan?= <hi_kalkan@yahoo.com>
Date: Fri, 15 Jul 2022 11:05:11 +0300
Subject: [PATCH] Prevent duplicate providers in the list.

---
 .../Volo.Abp.Settings/Volo/Abp/Settings/SettingDefinition.cs  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/framework/src/Volo.Abp.Settings/Volo/Abp/Settings/SettingDefinition.cs b/framework/src/Volo.Abp.Settings/Volo/Abp/Settings/SettingDefinition.cs
index ee96d1c160..21527c0ff2 100644
--- a/framework/src/Volo.Abp.Settings/Volo/Abp/Settings/SettingDefinition.cs
+++ b/framework/src/Volo.Abp.Settings/Volo/Abp/Settings/SettingDefinition.cs
@@ -39,7 +39,7 @@ public class SettingDefinition
     /// A list of allowed providers to get/set value of this setting.
     /// An empty list indicates that all providers are allowed.
     /// </summary>
-    public List<string> Providers { get; } //TODO: Rename to AllowedProviders
+    public List<string> Providers { get; }
 
     /// <summary>
     /// Is this setting inherited from parent scopes.
@@ -98,7 +98,7 @@ public class SettingDefinition
     {
         if (!providers.IsNullOrEmpty())
         {
-            Providers.AddRange(providers);
+            Providers.AddIfNotContains(providers);
         }
 
         return this;