Hook in icns and fonts auto-updates to settings

Author: ravichdev

plugin/php/admin/class-admin-updates.php

@@ -71,14 +71,8 @@ public function run_updates() {
 	 * @throws \Exception Exception.
 	 */
 	public function update_fonts( $write_response = true ) {
-
-		$expired = get_transient( Update_Fonts::TRANSIENT );
-		if ( false !== $expired ) {
-			return false;
-		}
-
 		$fonts = new Update_Fonts( false );
-		$fonts->get_fonts( $write_response );
+		$fonts->update( $write_response );
 
 		return true;
 	}
@@ -91,14 +85,8 @@ public function update_fonts( $write_response = true ) {
 	 * @return bool
 	 */
 	public function update_icons( $write_response = true ) {
-
-		$expired = get_transient( Update_Icons::TRANSIENT );
-		if ( false !== $expired ) {
-			return false;
-		}
-
 		$icons = new Update_Icons( false );
-		$icons->get_icons( $write_response );
+		$icons->update( $write_response );
 
 		return true;
 	}

plugin/php/api/class-update-fonts.php

@@ -197,6 +197,20 @@ public function get_http_response() {
 		return ! empty( $json ) ? $json : false;
 	}
 
+	/**
+	 * Update fonts from Google Font API.
+	 *
+	 * @param boolean $write_response Shoud write the reponse to file.
+	 * @return mixed
+	 */
+	public function update( $write_response = false ) {
+		if ( $this->should_check_for_updates() ) {
+			return $this->get_fonts( $write_response );
+		}
+
+		return false;
+	}
+
 	/**
 	 * Returns error message
 	 *

plugin/php/api/class-update-icons.php

@@ -170,6 +170,20 @@ public function get_http_response( $write_response = true ) {
 		return $codepoints;
 	}
 
+	/**
+	 * Update icons from source.
+	 *
+	 * @param boolean $write_response Shoud write the reponse to file.
+	 * @return mixed
+	 */
+	public function update( $write_response = false ) {
+		if ( $this->should_check_for_updates() ) {
+			return $this->get_icons( $write_response );
+		}
+
+		return false;
+	}
+
 	/**
 	 * Get last updated timestamp
 	 *

plugin/php/api/class-updates-api-base.php

@@ -61,6 +61,14 @@ public function __construct() {
 	 */
 	abstract public function get_http_response();
 
+	/**
+	 * Child class must provide this method
+	 *
+	 * @param boolean $write_response Shoud write the reponse to file.
+	 * @return mixed
+	 */
+	abstract public function update( $write_response );
+
 	/**
 	 * Wrapper function to accommodate tests.
 	 *
@@ -75,4 +83,29 @@ abstract public function get_http_response();
 	public function file_get_contents( $filename, $use_include_path = false, $context = null, $offset = 0, $length = null ) {
 		return apply_filters( 'material_design_file_get_contents', file_get_contents( $filename, $use_include_path, $context, $offset, $length ) ); //phpcs:ignore WordPressVIPMinimum.Performance.FetchingRemoteData.FileGetContentsUnknown
 	}
+
+	/**
+	 * Determine if check for auto-updates should happen.
+	 *
+	 * @return boolean
+	 */
+	public function should_check_for_updates() {
+		$expired = get_transient( static::TRANSIENT );
+
+		if ( false !== $expired ) {
+			return false;
+		}
+
+		// Check if auto-update is enabled.
+		return static::is_auto_update_enabled();
+	}
+
+	/**
+	 * Determine if auto-update is enabled.
+	 *
+	 * @return boolean
+	 */
+	public function is_auto_update_enabled() {
+		return ! empty( get_option( static::AUTO_UPDATE_SLUG, false ) );
+	}
 }