#3350 DBCC DBinfo fires incorrectly

[Montro1981]

Fix for issue #3350

The old code had an logical error in it:

Check 233 is not in #BlitzResults
Check 2 is not in #SkipChecks
Check 68 is in #SkipChecks

It became:
IF True (Check 233 not in #BlitzResults)
AND
(
    True (Check 2 not in #SkipChecks)
    OR
    False (Check 68 in #SkipChecks)
)

So the IF returned true and the check fired.
Basically the OR needed to be an AND, this allowed the code to be simplified to:

IF NOT EXISTS
(
	SELECT	1/0 
	FROM	#BlitzResults 
	WHERE	CheckID = 223 AND URL = 'https://aws.amazon.com/rds/sqlserver/'
) AND NOT EXISTS
(
	SELECT  1/0
	FROM    #SkipChecks
	WHERE   DatabaseName IS NULL AND CheckID IN (2, 68)
)

Easier to read as well.

[BrentOzar]

Thanks for the pull request! Wow, the logic in the original one makes my head hurt. Merging into the dev branch, will be in the next release with credit to you in the release notes.