Fix connection string section load.

SenseNet · Jul 15, 2019 · 973bd37 · 973bd37
1 parent 8c9521a
commit 973bd37
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 0 deletions.
diff --git a/src/SenseNet.Tools.Tests/App.config b/src/SenseNet.Tools.Tests/App.config
@@ -6,6 +6,9 @@
     </sectionGroup>
     <section name="feature1" type="System.Configuration.NameValueSectionHandler" />
   </configSections>
+  <connectionStrings>
+    <add name="cs1" connectionString="csvalue1"/>
+  </connectionStrings>
   <appSettings>
     <add key="key1" value="value1" />
     <add key="key2" value="123" />

diff --git a/src/SenseNet.Tools.Tests/SnConfigTests.cs b/src/SenseNet.Tools.Tests/SnConfigTests.cs
@@ -173,6 +173,21 @@ void Test()
                 Test();
         }
 
+        //============================================================================== Legacy tests
+
+        [TestMethod]
+        public void SnConfig_Legacy_ConnectionString()
+        {
+            void Test()
+            {
+                Assert.AreEqual("csvalue1", SnConfig.GetValue<string>("connectionStrings", "cs1"));
+                Assert.AreEqual("csvalue1", SnConfig.GetValue<string>("ConnectionStrings", "cs1"));
+            }
+
+            using (new ConfigurationSwindler(new SnLegacyConfiguration()))
+                Test();
+        }
+
         //============================================================================== Error tests
 
         [TestMethod]

diff --git a/src/SenseNet.Tools/Configuration/SnLegacyConfiguration.cs b/src/SenseNet.Tools/Configuration/SnLegacyConfiguration.cs
@@ -78,6 +78,10 @@ public string this[string key]
         {
             get
             {
+                // special section in old configuration files
+                if (string.Compare(Key, "connectionStrings", StringComparison.InvariantCultureIgnoreCase) == 0)
+                    return ConfigurationManager.ConnectionStrings[key]?.ConnectionString;
+
                 var configValue = _configValues?[key];
 
                 // backward compatibility: fallback to the appsettings section