Merge pull request #13 from theohbrothers/refactor/revise-new-itemlin…

…k-parameter-order Refactor: Revise New-ItemLink parameter order
theohbrothers · Aug 2, 2021 · 1585d1e · 1585d1e
2 parents 1cfcd4e + 67e82b3
commit 1585d1e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -51,7 +51,7 @@ Sample definition files can be found [here](docs/samples/definitions/links).
 #### Parameters
 
 ```powershell
-New-ItemLink [-Path] <string> [-ItemType] {HardLink | Junction | SymbolicLink} [-Value] <string> [-Force] [<CommonParameters>]
+New-ItemLink [-Path] <string> [-Value] <string> [-ItemType] {HardLink | Junction | SymbolicLink} [-Force] [<CommonParameters>]
 ```
 
 #### Examples

diff --git a/src/ItemLinkManagement/Public/New-ItemLink.ps1 b/src/ItemLinkManagement/Public/New-ItemLink.ps1
@@ -6,13 +6,13 @@ function New-ItemLink {
         [string]$Path
         ,
         [Parameter(Mandatory=$true, ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true)]
-        [ValidateSet('HardLink', 'Junction', 'SymbolicLink')]
-        [string]$ItemType
-        ,
-        [Parameter(Mandatory=$true, ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true)]
         [ValidateNotNullOrEmpty()]
         [string]$Value
         ,
+        [Parameter(Mandatory=$true, ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true)]
+        [ValidateSet('HardLink', 'Junction', 'SymbolicLink')]
+        [string]$ItemType
+        ,
         [Parameter(Mandatory=$false)]
         [switch]$Force
     )