Move hooks out of Waf_Runner class

projects/packages/waf/src/class-waf-initializer.php

@@ -42,13 +42,19 @@ public static function init() {
 		// Update the WAF after installing or upgrading a relevant Jetpack plugin
 		add_action( 'upgrader_process_complete', __CLASS__ . '::update_waf_after_plugin_upgrade', 10, 2 );
 
+		// Update the WAF after updating the rules settings
+		Waf_Rules_Manager::add_hooks();
+
 		// Check for compatibility updates
 		add_action( 'admin_init', __CLASS__ . '::check_for_updates' );
 
 		// WAF activation/deactivation hooks
 		add_action( 'jetpack_activate_module_waf', __CLASS__ . '::on_waf_activation' );
 		add_action( 'jetpack_deactivate_module_waf', __CLASS__ . '::on_waf_deactivation' );
 
+		// Update the WAF rule files on a schedule
+		Waf_Rules_Manager::schedule_rules_cron();
+
 		// Brute force protection activation/deactivation hooks
 		add_action( 'jetpack_activate_module_protect', __CLASS__ . '::on_brute_force_protection_activation' );
 		add_action( 'jetpack_deactivate_module_protect', __CLASS__ . '::on_brute_force_protection_deactivation' );

projects/packages/waf/src/class-waf-runner.php

@@ -37,12 +37,7 @@ public static function initialize() {
 		if ( ! self::is_allowed_mode( JETPACK_WAF_MODE ) ) {
 			return;
 		}
-		// Don't run if in standalone mode
-		if ( function_exists( 'add_action' ) ) {
-			self::add_hooks();
-			Waf_Rules_Manager::add_hooks();
-			Waf_Rules_Manager::schedule_rules_cron();
-		}
+
 		if ( ! self::did_run() ) {
 			self::run();
 		}
@@ -52,11 +47,10 @@ public static function initialize() {
 	 * Set action hooks
 	 *
 	 * @return void
+	 *
+	 * @deprecated $$next-version$$ Hooks have been moved to Waf_Initializer class.
 	 */
-	public static function add_hooks() {
-		// Register REST routes.
-		add_action( 'rest_api_init', array( new REST_Controller(), 'register_rest_routes' ) );
-	}
+	public static function add_hooks() {}
 
 	/**
 	 * Did the WAF run yet or not?