From 12e11c60889f277c5b29f3e593905b226aea1503 Mon Sep 17 00:00:00 2001 From: Gary Jones Date: Tue, 18 Aug 2026 15:28:39 +0100 Subject: [PATCH] chore: remove obsolete doc-link TODO from RestrictedHooks The `upload_mimes` group carried a TODO asking for a VIP documentation link to be added to its warning, tracked in #235. That issue has now been closed without adding a link: docs.wpvip.com documents general topics rather than individual sniffs, and no page covers safely customising `upload_mimes` or the risk of allowing insecure upload types, so there is nothing suitable to link. Left in place, the TODO points at a closed, decided-against issue and would only invite the same investigation to be repeated. The warning is clear on its own, so drop the comment. --- WordPressVIPMinimum/Sniffs/Hooks/RestrictedHooksSniff.php | 1 - 1 file changed, 1 deletion(-) diff --git a/WordPressVIPMinimum/Sniffs/Hooks/RestrictedHooksSniff.php b/WordPressVIPMinimum/Sniffs/Hooks/RestrictedHooksSniff.php index 2ced308b..c18a2903 100644 --- a/WordPressVIPMinimum/Sniffs/Hooks/RestrictedHooksSniff.php +++ b/WordPressVIPMinimum/Sniffs/Hooks/RestrictedHooksSniff.php @@ -46,7 +46,6 @@ class RestrictedHooksSniff extends AbstractFunctionParameterSniff { */ private $restricted_hook_groups = [ 'upload_mimes' => [ - // TODO: This error message needs a link to the VIP Documentation, see https://github.com/Automattic/VIP-Coding-Standards/issues/235. 'type' => 'warning', 'msg' => 'Please ensure that the mimes being filtered do not include insecure types (i.e. SVG, SWF, etc.). Manual inspection required.', 'hooks' => [