From cac65d8a546f00eb933bacb0d0f781cdd5e59c8c Mon Sep 17 00:00:00 2001
From: jrfnl <jrfnl@users.noreply.github.com>
Date: Tue, 27 Feb 2018 10:44:25 +0100
Subject: [PATCH] :wrench: PHPCS: enable checking for globals being prefixed

Everything which is defined in the global namespace should be prefixed to prevent conflicts with other themes and plugins.

This snippet enables the sniff which verifies this and tells it which prefix to look for.
---
 phpcs.xml.dist | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/phpcs.xml.dist b/phpcs.xml.dist
index 2c4eabf2..66317a71 100644
--- a/phpcs.xml.dist
+++ b/phpcs.xml.dist
@@ -86,6 +86,14 @@
 		</properties>
 	</rule>
 
+	<!-- Verify that everything in the global namespace is prefixed with a theme specific prefix.
+		 Multiple valid prefixes can be provided as a comma-delimited list. -->
+	<rule ref="WordPress.NamingConventions.PrefixAllGlobals">
+		<properties>
+			<property name="prefixes" type="array" value="_s" />
+		</properties>
+	</rule>
+
 
 	<!--
 	#############################################################################
@@ -93,6 +101,7 @@
 	#############################################################################
 	-->
 
+
 	<config name="testVersion" value="5.2-99.0"/>
 	<rule ref="PHPCompatibility"/>
 </ruleset>