Improve ANSI Settings Check Logic to Distinguish Best Practices from Legacy Defaults (issue #658)

Author: gflpcantor31

sp_PerfCheck/sp_PerfCheck.sql

@@ -4338,29 +4338,29 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
             database_name = d.name,
             details =
                 N'One or more ANSI settings differ from recommended best practices: ' +
-                CASE WHEN d.is_ansi_null_default_on = 0 THEN N'ANSI_NULL_DEFAULT OFF (recommended ON), ' ELSE N'' END +
-                CASE WHEN d.is_ansi_nulls_on = 0 THEN N'ANSI_NULLS OFF (recommended ON), ' ELSE N'' END +
-                CASE WHEN d.is_ansi_padding_on = 0 THEN N'ANSI_PADDING OFF (recommended ON), ' ELSE N'' END +
-                CASE WHEN d.is_ansi_warnings_on = 0 THEN N'ANSI_WARNINGS OFF (recommended ON), ' ELSE N'' END +
-                CASE WHEN d.is_arithabort_on = 0 THEN N'ARITHABORT OFF (recommended ON in many contexts), ' ELSE N'' END +
-                CASE WHEN d.is_concat_null_yields_null_on = 0 THEN N'CONCAT_NULL_YIELDS_NULL OFF (recommended ON), ' ELSE N'' END +
-                CASE WHEN d.is_numeric_roundabort_on = 1 THEN N'NUMERIC_ROUNDABORT ON (recommended OFF), ' ELSE N'' END +
-                CASE WHEN d.is_quoted_identifier_on = 0 THEN N'QUOTED_IDENTIFIER OFF (recommended ON), ' ELSE N'' END +
+                      CASE WHEN d.is_ansi_null_default_on = 0 THEN N'ANSI_NULL_DEFAULT OFF (recommended ON), ' ELSE N'' END +
+                      CASE WHEN d.is_ansi_nulls_on = 0 THEN N'ANSI_NULLS OFF (recommended ON), ' ELSE N'' END +
+                      CASE WHEN d.is_ansi_padding_on = 0 THEN N'ANSI_PADDING OFF (recommended ON), ' ELSE N'' END +
+                      CASE WHEN d.is_ansi_warnings_on = 0 THEN N'ANSI_WARNINGS OFF (recommended ON), ' ELSE N'' END +
+                      CASE WHEN d.is_arithabort_on = 0 THEN N'ARITHABORT OFF (recommended ON in many contexts), ' ELSE N'' END +
+                      CASE WHEN d.is_concat_null_yields_null_on = 0 THEN N'CONCAT_NULL_YIELDS_NULL OFF (recommended ON), ' ELSE N'' END +
+                      CASE WHEN d.is_numeric_roundabort_on = 1 THEN N'NUMERIC_ROUNDABORT ON (recommended OFF), ' ELSE N'' END +
+                      CASE WHEN d.is_quoted_identifier_on = 0 THEN N'QUOTED_IDENTIFIER OFF (recommended ON), ' ELSE N'' END +
                 N'These settings may lead to inconsistent behavior, reduced feature compatibility, or unexpected query results ' +
                 N'if they do not align with recommended best practices.',
             url = N'https://erikdarling.com/sp_PerfCheck#ANSISettings'
         FROM #databases AS d
         WHERE d.database_id = @current_database_id
         AND
         (
-               d.is_ansi_null_default_on = 0
-            OR d.is_ansi_nulls_on = 0
-            OR d.is_ansi_padding_on = 0
-            OR d.is_ansi_warnings_on = 0
-            OR d.is_arithabort_on = 0
-            OR d.is_concat_null_yields_null_on = 0
-            OR d.is_numeric_roundabort_on = 1
-            OR d.is_quoted_identifier_on = 0
+             d.is_ansi_null_default_on = 0
+          OR d.is_ansi_nulls_on = 0
+          OR d.is_ansi_padding_on = 0
+          OR d.is_ansi_warnings_on = 0
+          OR d.is_arithabort_on = 0
+          OR d.is_concat_null_yields_null_on = 0
+          OR d.is_numeric_roundabort_on = 1
+          OR d.is_quoted_identifier_on = 0
         );
 
         /* Check Query Store Status */