Cleaned up help and parameters

Author: sanderstad

functions/Initialize-PDCVhdDisk.ps1

@@ -15,6 +15,17 @@ function Initialize-PDCVhdDisk {
 
     $scred = Get-Credential, then pass $scred object to the -Credential parameter.
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
+
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
 
@@ -46,7 +57,8 @@ function Initialize-PDCVhdDisk {
         $Credential,
         [ValidateSet('GPT', 'MBR')]
         [string]$PartitionStyle = 'GPT',
-        [int]$AllocationUnitSize = 64KB
+        [int]$AllocationUnitSize = 64KB,
+        [switch]$EnableException
     )
 
     begin {

functions/Invoke-PDCRepairClone.ps1

@@ -21,6 +21,17 @@ function Invoke-PDCRepairClone {
     Windows Authentication will be used if SqlCredential is not specified. SQL Server does not accept Windows credentials being passed as credentials.
     To connect as a different Windows user, run PowerShell as that user.
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
+
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
 
@@ -42,9 +53,9 @@ function Invoke-PDCRepairClone {
     param(
         [Parameter(Mandatory = $true)]
         [string[]]$HostName,
-
         [System.Management.Automation.PSCredential]
-        $SqlCredential
+        $SqlCredential,
+        [switch]$EnableException
     )
 
     begin {
@@ -87,10 +98,10 @@ function Invoke-PDCRepairClone {
             # Get the clones registered for the host
             try {
                 Write-PSFMessage -Message "Get the clones for host $hst" -Level Verbose
-                $results = Invoke-DbaSqlQuery -SqlInstance $ecDatabaseServer -Database $ecDatabaseName -Query $query
+                $results = Invoke-DbaSqlQuery -SqlInstance $pdcSqlInstance -Database $pdcDatabase -Query $query
             }
             catch {
-                Stop-PSFFunction -Message "Couldn't get the clones for $hst" -Target $ecDatabaseServer -ErrorRecord $_ -Continue
+                Stop-PSFFunction -Message "Couldn't get the clones for $hst" -Target $pdcSqlInstance -ErrorRecord $_ -Continue
             }
 
             # Loop through the results

functions/New-PDCClone.ps1

@@ -83,30 +83,20 @@ function New-PDCClone {
         [parameter(Mandatory = $true)]
         [ValidateNotNullOrEmpty()]
         [object[]]$SqlInstance,
-
         [System.Management.Automation.PSCredential]
         $SqlCredential,
-
         [System.Management.Automation.PSCredential]
         $Credential,
-
         [parameter(Mandatory = $true, ParameterSetName = "ByParent")]
         [string]$ParentVhd,
-
         [string]$Destination,
-
         [string]$CloneName,
-
         [parameter(Mandatory = $true, ParameterSetName = "ByLatest")]
         [string[]]$Database,
-
         [parameter(Mandatory = $true, ParameterSetName = "ByLatest")]
         [switch]$LatestImage,
-
         [switch]$Disabled,
-
         [switch]$Force,
-
         [switch]$EnableException
     )
 

functions/New-PDCImage.ps1

@@ -57,6 +57,17 @@ function New-PDCImage {
 .PARAMETER Force
     Forcefully execute commands when needed
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
+
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
 
@@ -83,35 +94,24 @@ function New-PDCImage {
         [parameter(Mandatory = $true)]
         [ValidateNotNullOrEmpty()]
         [object]$SourceSqlInstance,
-
         [System.Management.Automation.PSCredential]
         $SourceSqlCredential,
-
         [Parameter(Mandatory = $true)]
         [ValidateNotNullOrEmpty()]
         [object]$DestinationSqlInstance,
-
         [System.Management.Automation.PSCredential]
         $DestinationSqlCredential,
-
         [System.Management.Automation.PSCredential]
         $DestinationCredential,
-
         [Parameter(Mandatory = $true)]
         [ValidateNotNullOrEmpty()]
         [string]$ImageNetworkPath,
-
         [string]$ImageLocalPath,
-
         [Parameter(Mandatory = $true, ValueFromPipeline = $true)]
         [object[]]$Database,
-
         [switch]$CreateFullBackup,
-
         [switch]$UseLastFullBackup,
-
         [switch]$Force,
-
         [switch]$EnableException
     )
 

functions/New-PDCVhdDisk.ps1

@@ -28,6 +28,17 @@ function New-PDCVhdDisk {
 .PARAMETER Force
     Forcefully create the neccesary items
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
+
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
 
@@ -55,7 +66,8 @@ function New-PDCVhdDisk {
         [uint64]$Size = 64TB,
         [switch]$FixedSize,
         [switch]$ReadOnly,
-        [switch]$Force
+        [switch]$Force,
+        [switch]$EnableException
     )
 
     begin {

functions/Remove-PDCClone.ps1

@@ -34,6 +34,17 @@ function Remove-PDCClone {
 .PARAMETER ExcludeDatabase
     Allows to filter to exclude specific databases
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
+
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
 
@@ -66,19 +77,14 @@ function Remove-PDCClone {
         [parameter(Mandatory = $true)]
         [ValidateNotNullOrEmpty()]
         [string[]]$HostName,
-
         [System.Management.Automation.PSCredential]
         $SqlCredential,
-
         [System.Management.Automation.PSCredential]
         $Credential,
-
         [string[]]$Database,
-
         [string[]]$ExcludeDatabase,
-
-        [switch]$All
-
+        [switch]$All,
+        [switch]$EnableException
     )
 
     begin {

functions/Remove-PDCImage.ps1

@@ -35,11 +35,10 @@ function Remove-PDCImage {
         [parameter(Mandatory = $true)]
         [ValidateNotNullOrEmpty()]
         [string[]]$ImageLocation,
-
         [System.Management.Automation.PSCredential]
         $Credential,
-
-        [switch]$Force
+        [switch]$Force,
+        [switch]$EnableException
     )
 
     begin {

functions/Set-PDCConfiguration.ps1

@@ -27,6 +27,16 @@ function Set-PDCConfiguration {
 .PARAMETER Database
     Database to use to save all the information in
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
 
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
@@ -49,11 +59,10 @@ function Set-PDCConfiguration {
         [ValidateNotNullOrEmpty()]
         [Alias("ServerInstance", "SqlServerSqlServer")]
         [object]$SqlInstance,
-
         [System.Management.Automation.PSCredential]
         $SqlCredential,
-
-        [string]$Database
+        [string]$Database,
+        [switch]$EnableException
     )
 
     begin {

internal/functions/Convert-PDCLocalUncPathToLocalPath.ps1

@@ -10,6 +10,17 @@ function Convert-PDCLocalUncPathToLocalPath {
 .PARAMETER UncPath
     UNC path to convert to local path
 
+.PARAMETER EnableException
+    By default, when something goes wrong we try to catch it, interpret it and give you a friendly warning message.
+    This avoids overwhelming you with "sea of red" exceptions, but is inconvenient because it basically disables advanced scripting.
+    Using this switch turns this "nice by default" feature off and enables you to catch exceptions with your own try/catch.
+
+.PARAMETER WhatIf
+    If this switch is enabled, no actions are performed but informational messages will be displayed that explain what would happen if the command were to run.
+
+.PARAMETER Confirm
+    If this switch is enabled, you will be prompted for confirmation before executing any operations that change state.
+
 .NOTES
     Author: Sander Stad (@sqlstad, sqlstad.nl)
 
@@ -28,7 +39,8 @@ function Convert-PDCLocalUncPathToLocalPath {
     [CmdletBinding()]
     param(
         [Parameter(Mandatory = $true)]
-        [string]$UncPath
+        [string]$UncPath,
+        [switch]$EnableException
     )
 
     # Create the object

internal/functions/Test-PDCConfiguration.ps1

@@ -47,12 +47,9 @@ function Test-PDCConfiguration {
 
     param(
         [object]$SqlInstance,
-
         [System.Management.Automation.PSCredential]
         $SqlCredential,
-
         [string]$Database,
-
         [switch]$EnableException
     )