Get-DbaLogin - clean up old code causing issues

[jpomfret]

Type of Change

• [x] Bug fix (non-breaking change, fixes #9290
• [ ] New feature (non-breaking change, adds functionality, fixes # )
• [ ] Breaking change (affects multiple commands or functionality, fixes # )
• [x] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
• [ ] Adding code coverage to existing functionality
• [ ] Pester test is included
• [ ] If new file reference added for test, has is been added to github.com/dataplat/appveyor-lab ?
• [ ] Unit test is included
• [ ] Documentation
• [ ] Build system

Purpose

Fix an issue where the command didn't work properly if you had a variable in your session called $SqlLogins

Approach

Remove deprecated code - this used to be needed, but after the parameters were renamed it no longer is.

Commands to test

# this should work
Get-DbaLogin -SqlInstance $source |ft

# then set a variable and rerun - it'll only show sql logins - after this PR this works
$sqllogins = @(
"myDomain\UserA",
"myDomain\UserA")
Get-DbaLogin -SqlInstance $source |ft

Screenshots

Before - shows issue - second screenshot there are now Windows* logins

After - variable doesn't change results

tests passing

[jpomfret]

believe test failures are unrelated

[potatoqualitee]

thank you! set it to redo tests and will merge whent they pass 🙏🏼

[jpomfret]

Thanks @potatoqualitee 🐱