Hard-deprecate the WordPressVIPMinimum.Functions.DynamicCalls sniff

Removing the sniff needs to wait for a new major, as it would be a breaking change, however, in the mean time, we can:
* Exclude the sniff from both rulesets. This will prevent PHP_CodeSniffer from throwing deprecation notices about the sniff.
* Deprecate the sniff itself, in case users of the VIP-Coding-Standards would be explicitly including the sniff from within their own ruleset.

Part of 517

Author: jrfnl

WordPress-VIP-Go/ruleset-test.inc

@@ -341,9 +341,9 @@ $my_theme_options = get_option( 'my_theme', false );
 if ( array_key_exists( 'key', $my_theme_options ) ) { } // Error.
 echo '<a href="' . esc_url( get_term_link( $var ) ) . '">My term link</a>'; // Error.
 
-// WordPressVIPMinimum.Functions.DynamicCalls
-$my_notokay_func = 'extract';
-$my_notokay_func(); // Error.
+
+
+
 
 // WordPressVIPMinimum.Functions.RestrictedFunctions
 

WordPress-VIP-Go/ruleset-test.php

@@ -36,7 +36,6 @@
 		337 => 1,
 		341 => 1,
 		342 => 1,
-		346 => 1,
 		350 => 1,
 		351 => 1,
 		352 => 1,

WordPress-VIP-Go/ruleset.xml

@@ -6,6 +6,7 @@
 	<!-- Include the base VIP Minimum ruleset -->
 	<rule ref="WordPressVIPMinimum">
 		<exclude name="WordPressVIPMinimum.JS"/>
+		<exclude name="WordPressVIPMinimum.Functions.DynamicCalls"/>
 	</rule>
 
 	<!-- Things that may be incompatible with the VIP Go infrastructure and needs a dev to review -->

WordPressVIPMinimum/Sniffs/Functions/DynamicCallsSniff.php

@@ -9,6 +9,7 @@
 
 namespace WordPressVIPMinimum\Sniffs\Functions;
 
+use PHP_CodeSniffer\Sniffs\DeprecatedSniff;
 use PHP_CodeSniffer\Util\Tokens;
 use PHPCSUtils\Utils\TextStrings;
 use WordPressVIPMinimum\Sniffs\Sniff;
@@ -26,7 +27,7 @@
  *
  * @link http://php.net/manual/en/migration71.incompatible.php
  */
-class DynamicCallsSniff extends Sniff {
+class DynamicCallsSniff extends Sniff implements DeprecatedSniff {
 
 	/**
 	 * Functions that should not be called dynamically.
@@ -194,4 +195,31 @@ private function find_dynamic_calls( $stackPtr ) {
 		$data    = [ $this->variables_arr[ $this->tokens[ $stackPtr ]['content'] ] ];
 		$this->phpcsFile->addError( $message, $stackPtr, 'DynamicCalls', $data );
 	}
+
+	/**
+	 * Provide the version number in which the sniff was deprecated.
+	 *
+	 * @return string
+	 */
+	public function getDeprecationVersion() {
+		return 'VIP-Coding-Standard v3.1.0';
+	}
+
+	/**
+	 * Provide the version number in which the sniff will be removed.
+	 *
+	 * @return string
+	 */
+	public function getRemovalVersion() {
+		return 'VIP-Coding-Standard v4.0.0';
+	}
+
+	/**
+	 * Provide a custom message to display with the deprecation.
+	 *
+	 * @return string
+	 */
+	public function getDeprecationMessage() {
+		return '';
+	}
 }

WordPressVIPMinimum/ruleset-test.inc

@@ -303,9 +303,9 @@ $my_theme_options = get_option( 'my_theme', false );
 if ( array_key_exists( 'key', $my_theme_options ) ) { } // Error.
 echo '<a href="' . esc_url( get_term_link( $var ) ) . '">My term link</a>'; // Error.
 
-// WordPressVIPMinimum.Functions.DynamicCalls
-$my_notokay_func = 'extract';
-$my_notokay_func(); // Error.
+
+
+
 
 // WordPressVIPMinimum.Functions.RestrictedFunctions
 

WordPressVIPMinimum/ruleset-test.php

@@ -81,7 +81,6 @@
 		299 => 1,
 		303 => 1,
 		304 => 1,
-		308 => 1,
 		312 => 1,
 		313 => 1,
 		314 => 1,

WordPressVIPMinimum/ruleset.xml

@@ -18,6 +18,11 @@
 		rule in which to place the exclusion, so this will have to do for now.
 		-->
 		<exclude name="WordPressVIPMinimum.JS"/>
+
+		<!--
+		Exclude other deprecated sniffs.
+		-->
+		<exclude name="WordPressVIPMinimum.Functions.DynamicCalls"/>
 	</rule>
 
 	<rule ref="Generic.PHP.Syntax"/>