Merge pull request #2848 from PeteSral/dev

BrentOzar · web-flow · commit 0319c3b0ccc3 · 2021-04-10T07:53:41.000Z
#2847 sp_BlitzCache: Fix negative CHARINDEX result when ) precedes ( in compile_time_value.
diff --git a/sp_BlitzCache.sql b/sp_BlitzCache.sql
@@ -3982,7 +3982,7 @@ SET    s.variable_datatype = CASE WHEN s.variable_datatype LIKE '%(%)%'
 	   s.compile_time_value = CASE WHEN s.compile_time_value LIKE '%(%)%' 
                                    THEN SUBSTRING(s.compile_time_value, 
 												  CHARINDEX('(', s.compile_time_value) + 1,
-												  CHARINDEX(')', s.compile_time_value) - 1 - CHARINDEX('(', s.compile_time_value)
+												  CHARINDEX(')', s.compile_time_value, CHARINDEX('(', s.compile_time_value) + 1) - 1 - CHARINDEX('(', s.compile_time_value)
 												  )
 									WHEN variable_datatype NOT IN ('bit', 'tinyint', 'smallint', 'int', 'bigint') 
 									AND s.variable_datatype NOT LIKE '%binary%' 

Original file line number	Diff line number	Diff line change
`@@ -3982,7 +3982,7 @@ SET s.variable_datatype = CASE WHEN s.variable_datatype LIKE '%(%)%'`
`3982`	`3982`	`s.compile_time_value = CASE WHEN s.compile_time_value LIKE '%(%)%'`
`3983`	`3983`	`THEN SUBSTRING(s.compile_time_value,`
`3984`	`3984`	`CHARINDEX('(', s.compile_time_value) + 1,`
`3985`		`- CHARINDEX(')', s.compile_time_value) - 1 - CHARINDEX('(', s.compile_time_value)`
	`3985`	`+ CHARINDEX(')', s.compile_time_value, CHARINDEX('(', s.compile_time_value) + 1) - 1 - CHARINDEX('(', s.compile_time_value)`
`3986`	`3986`	`)`
`3987`	`3987`	`WHEN variable_datatype NOT IN ('bit', 'tinyint', 'smallint', 'int', 'bigint')`
`3988`	`3988`	`AND s.variable_datatype NOT LIKE '%binary%'`