phpcs: Enable and fix Squiz.Operators.ValidLogicalOperators.NotAllowed

Change-Id: Ic0e5909c76e0d58f33c4e29b227cad9eb35f3f25

.phpcs.xml

@@ -34,7 +34,6 @@
 		<exclude name="PSR2.Classes.PropertyDeclaration.VarUsed" />
 		<exclude name="PSR2.Methods.MethodDeclaration.Underscore" />
 		<exclude name="Squiz.Classes.ValidClassName.NotCamelCaps" />
-		<exclude name="Squiz.Operators.ValidLogicalOperators.NotAllowed" />
 		<exclude name="Squiz.Scope.MemberVarScope.Missing" />
 		<exclude name="Squiz.Scope.MethodScope.Missing" />
 	</rule>

lessc.inc.php

@@ -213,15 +213,15 @@ public function cachedCompile( $in, $force = false ) {
 
 		if ( is_string( $in ) ) {
 			$root = $in;
-		} elseif ( is_array( $in ) and isset( $in['root'] ) ) {
-			if ( $force or !isset( $in['files'] ) ) {
+		} elseif ( is_array( $in ) && isset( $in['root'] ) ) {
+			if ( $force || !isset( $in['files'] ) ) {
 				// If we are forcing a recompile or if for some reason the
 				// structure does not contain any file information we should
 				// specify the root to trigger a rebuild.
 				$root = $in['root'];
-			} elseif ( isset( $in['files'] ) and is_array( $in['files'] ) ) {
+			} elseif ( isset( $in['files'] ) && is_array( $in['files'] ) ) {
 				foreach ( $in['files'] as $fname => $ftime ) {
-					if ( !file_exists( $fname ) or filemtime( $fname ) > $ftime ) {
+					if ( !file_exists( $fname ) || filemtime( $fname ) > $ftime ) {
 						// One of the files we knew about previously has changed
 						// so we should look at our incoming root again.
 						$root = $in['root'];