Commit 268459d

committed

briefly improve the ux, we will move this task to a different branch

1 parent 24f8ed2 commit 268459dCopy full SHA for 268459d

File tree

+15

-6

lines changed

+15

-6

lines changed

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -51,11 +51,13 @@ function Remove-EmptyFolder {`
`51`	`51`	`# Use ShouldProcess for confirmation`
`52`	`52`	`if ($PSCmdlet.ShouldProcess("$folderPath", "Remove empty folder")) {`
`53`	`53`	`Remove-Item -Path $folderPath -Force`
`54`		- Write-Host "Removed empty folder:`n$folderPath" -ForegroundColor Red
	`54`	+ Write-Host "Removed empty folder:`n$folderPath" `
	`55`	`+ -ForegroundColor Red`
`55`	`56`	`}`
`56`	`57`	`}`
`57`	`58`	`else {`
`58`		- Write-Host "Skipped non-empty folder:`n$folderPath" -ForegroundColor Magenta
	`59`	+ Write-Host "Skipped non-empty folder:`n$folderPath" `
	`60`	`+ -ForegroundColor DarkYellow`
`59`	`61`	`continue`
`60`	`62`	`}`
`61`	`63`	`}`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ function Resolve-PathwErr {`
`41`	`41`	`try {`
`42`	`42`	`$outputPath = (Resolve-Path -Path $inputFolderPath -ErrorAction Stop).Path`
`43`	`43`	`Write-Verbose "Resolve operation successful for '$inputFolderPath',"`
`44`		- Write-Verbose "output path: '$outputPath'.`n"
	`44`	`+ Write-Verbose "Output path: '$outputPath'."`
`45`	`45`	`}`
`46`	`46`	`catch {`
`47`	`47`	`$outputPath = $null`

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,8 @@ function Confirm-FolderArray {`
`18`	`18`	`contains only unique entries.`
`19`	`19`
`20`	`20`	`.INPUTS`
`21`		`- [string[]] Accepts an array of folder paths as input.`
	`21`	`+ [System.Collections.Generic.List[string]]`
	`22`	`+ Accepts an array of folder paths as input.`
`22`	`23`
`23`	`24`	`.OUTPUTS`
`24`	`25`	`None. This function does not return output objects, but may write warning or verbose messages`
`@@ -40,13 +41,15 @@ function Confirm-FolderArray {`
`40`	`41`	`)`
`41`	`42`
`42`	`43`	`# Checking input folder array for null`
	`44`	`+ Write-Verbose "Checking for null or empty input..."`
`43`	`45`	`if (-not $folderPathsArray -or $folderPathsArray.Count -eq 0) {`
`44`	`46`	Show-ErrorMsg `
`45`	`47`	`-FunctionName $MyInvocation.MyCommand.Name`
`46`	`48`	`-CustomMessage "No folders provided."`
`47`	`49`	`}`
`48`	`50`
`49`	`51`	`# Checking for duplication in a simple manner through counting`
	`52`	`+ Write-Verbose "Checking for duplicate items..."`
`50`	`53`	`if ($folderPathsArray.Count -ne ($folderPathsArray \| Select-Object -Unique).Count) {`
`51`	`54`	`# Current behavior, function continues and warns user about the skipping`
`52`	`55`	`Write-Warning "Duplicate folders detected in the input."`

Lines changed: 6 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -67,14 +67,17 @@ function Move-ChildItemUp {`
`67`	`67`
`68`	`68`	`# After collecting all pipeline input, process the array`
`69`	`69`	`end {`
`70`		`- Write-Host "Program starts..." -ForegroundColor Green`
	`70`	+ Write-Host "Program starts...`n" `
	`71`	`+ -ForegroundColor Green`
`71`	`72`
`72`	`73`	`# Step 1:`
`73`	`74`	`# Validate input array only, not the path`
	`75`	`+ Write-Verbose "Start input folder array check..."`
`74`	`76`	`Confirm-FolderArray $folderPathsArray`
`75`	`77`
`76`	`78`	`# Step 2:`
`77`	`79`	`# Assemble the folder object array for ease of process`
	`80`	`+ Write-Verbose "Start collecting folder relevant info..."`
`78`	`81`	`[System.Collections.Generic.List[pscustomobject]]$folderObjArray =`
`79`	`82`	`Get-FolderInfo $folderPathsArray`
`80`	`83`
`@@ -86,6 +89,7 @@ function Move-ChildItemUp {`
`86`	`89`	`# Remove-Item -Path $source -Recurse -Force`
`87`	`90`	`Remove-EmptyFolder $folderObjArray`
`88`	`91`
`89`		`- Write-Host "Program complete." -ForegroundColor Green`
	`92`	+ Write-Host "`nAction complete." `
	`93`	`+ -ForegroundColor Green`
`90`	`94`	`}`
`91`	`95`	`}`

Whitespace-only changes.

Comments

(0)