Refactor validation for brevity

Automattic · Jun 24, 2024 · 157ec5c · 157ec5c
1 parent a540e18
commit 157ec5c
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/modules/notifications/notifications.php b/modules/notifications/notifications.php
@@ -1416,11 +1416,7 @@ public function settings_validate( $new_options ) {
 			}
 
 			// White list validation for the 'slack_webhook_url' option
-			if ( ! isset( $new_options['webhook_url'] ) ) {
-				$new_options['webhook_url'] = '';
-			}
-
-			if ( esc_url_raw( $new_options['webhook_url'] ) !== $new_options['webhook_url'] ) {
+			if ( ! isset( $new_options['webhook_url'] ) || esc_url_raw( $new_options['webhook_url'] ) !== $new_options['webhook_url'] ) {
 				$new_options['webhook_url'] = '';
 			} else {
 				$new_options['webhook_url'] = esc_url_raw( $new_options['webhook_url'] );