Fixed issue microsoft#92 | Fails to handle Farms with multiple SQL Se…

…rver instances
kayodebristol · Nov 13, 2019 · 1f12f89 · 1f12f89
1 parent 349d60f
commit 1f12f89
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -68,3 +68,4 @@ Please refer to the follow Wiki page if you are interested in contributing to th
 
 ### Unreleased
 - Fixed issue Incorrect handling of Port in Read-SPWebapplications #90
+- Fails to handle Farms with multiple SQL Server instances #92
diff --git a/SharePointDSC.Reverse.ps1 b/SharePointDSC.Reverse.ps1
@@ -19,6 +19,8 @@
 * Fixed issue with Central admin Port retrieval;
 * Updated dependency to ReverseDSC 1.9.4.7;
 * Updated dependency to SharePointDSC 3.6.0.0;
+* Fixed issue #90 incorrect handling of Port in Read-SPWebapplications
+* Fixed issue #92  | Fails to handle Farms with multiple SQL Server instances
 
 #>
 
@@ -3435,10 +3437,7 @@ function Read-SPContentDatabase()
             $params.WebAppUrl = $spContentDB.WebApplication.Url
             $results = Get-TargetResource @params
             $results = Repair-Credentials -results $results
-
-            Add-ConfigurationDataEntry -Node "NonNodeData" -Key "DatabaseServer" -Value $results.DatabaseServer -Description "Name of the Database Server associated with the destination SharePoint Farm;"
-            $results.DatabaseServer = "`$ConfigurationData.NonNodeData.DatabaseServer"
-
+            $results.DatabaseServer = $spContentDB.Server
             $currentBlock = Get-DSCBlock -UseGetTargetResource -Params $results -ModulePath $module
             $currentBlock = Convert-DSCStringParamToVariable -DSCBlock $currentBlock -ParameterName "DatabaseServer"
             $Script:dscConfigContent += $currentBlock