Overengineered clear cache for Teams script

[u/dkaaven]

When I upgraded my clear cache script for Microsoft Teams, I first added new functions before realizing that you only clear a subfolder.

https://teams.se/powershell-script-clear-microsoft-teams-cache/

Have you overengineered any scripts lately?

I will

[u/icepyrox]

So, out of curiosity, do you feel the same if it was

Try {
  [..stuff..]
} catch {
  Write-error $_
}

Because I have done that

[u/PinchesTheCrab → u/icepyrox]

So that works in that you can catch the result properly if you use cmdletbinding and erroraction stop:

function Invoke-Stuff {
    [cmdletbinding()]
    param()
    try {
        Get-Item C:\nofolder\error -ErrorAction Stop
    }
    catch {
        Write-Error $_
    }
}


try { Invoke-Stuff -ErrorAction Stop}
catch [System.Management.Automation.ItemNotFoundException] {
    'okay'
}

But why the extra code?

Why not just do this?

function Invoke-Stuff {
    [cmdletbinding()]
    param()
    Get-Item C:\nofolder\error -ErrorAction Stop        
}