insideout10 / wordlift-plugin

src/wordlift/shipping-data/class-offer-structured-data.php

Indentation +28 added lines, -28 removed lines

@@ -4,43 +4,43 @@
 
 class Offer_Structured_Data {
 
-	/**
-	 * @var Shipping_Zones
-	 */
-	private $shipping_zones;
+    /**
+     * @var Shipping_Zones
+     */
+    private $shipping_zones;
 
-	/**
-	 * Offer_Structured_Data constructor.
-	 *
-	 * @param Shipping_Zones $shipping_zones
-	 */
-	public function __construct( $shipping_zones ) {
+    /**
+     * Offer_Structured_Data constructor.
+     *
+     * @param Shipping_Zones $shipping_zones
+     */
+    public function __construct( $shipping_zones ) {
 
-		$this->shipping_zones = $shipping_zones;
+        $this->shipping_zones = $shipping_zones;
 
-		add_filter( 'wl_entity_jsonld', array( $this, 'entity_jsonld' ), 10, 2 );
+        add_filter( 'wl_entity_jsonld', array( $this, 'entity_jsonld' ), 10, 2 );
 
-	}
+    }
 
-	public function entity_jsonld( $jsonld, $post_id ) {
+    public function entity_jsonld( $jsonld, $post_id ) {
 
-		// Bail out if it's not a Product or the offers property isn't set.
-		if ( ! in_array( 'Product', (array) $jsonld['@type'], true ) || ! isset( $jsonld['offers'] ) ) {
-			return $jsonld;
-		}
+        // Bail out if it's not a Product or the offers property isn't set.
+        if ( ! in_array( 'Product', (array) $jsonld['@type'], true ) || ! isset( $jsonld['offers'] ) ) {
+            return $jsonld;
+        }
 
-		if ( ! is_array( $jsonld['offers'] ) || ! is_numeric( key( $jsonld['offers'] ) ) ) {
-			$jsonld['offers'] = array( $jsonld['offers'] );
-		}
+        if ( ! is_array( $jsonld['offers'] ) || ! is_numeric( key( $jsonld['offers'] ) ) ) {
+            $jsonld['offers'] = array( $jsonld['offers'] );
+        }
 
-		$product = new Product( $post_id );
+        $product = new Product( $post_id );
 
-		foreach ( $jsonld['offers'] as &$offer ) {
-			$this->shipping_zones->add_available_delivery_method( $offer );
-			$this->shipping_zones->add_offer_shipping_details( $offer, $product );
-		}
+        foreach ( $jsonld['offers'] as &$offer ) {
+            $this->shipping_zones->add_available_delivery_method( $offer );
+            $this->shipping_zones->add_offer_shipping_details( $offer, $product );
+        }
 
-		return $jsonld;
-	}
+        return $jsonld;
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -14,30 +14,30 @@
 	 *
 	 * @param Shipping_Zones $shipping_zones
 	 */
-	public function __construct( $shipping_zones ) {
+	public function __construct($shipping_zones) {
 
 		$this->shipping_zones = $shipping_zones;
 
-		add_filter( 'wl_entity_jsonld', array( $this, 'entity_jsonld' ), 10, 2 );
+		add_filter('wl_entity_jsonld', array($this, 'entity_jsonld'), 10, 2);
 
 	}
 
-	public function entity_jsonld( $jsonld, $post_id ) {
+	public function entity_jsonld($jsonld, $post_id) {
 
 		// Bail out if it's not a Product or the offers property isn't set.
-		if ( ! in_array( 'Product', (array) $jsonld['@type'], true ) || ! isset( $jsonld['offers'] ) ) {
+		if ( ! in_array('Product', (array) $jsonld['@type'], true) || ! isset($jsonld['offers'])) {
 			return $jsonld;
 		}
 
-		if ( ! is_array( $jsonld['offers'] ) || ! is_numeric( key( $jsonld['offers'] ) ) ) {
-			$jsonld['offers'] = array( $jsonld['offers'] );
+		if ( ! is_array($jsonld['offers']) || ! is_numeric(key($jsonld['offers']))) {
+			$jsonld['offers'] = array($jsonld['offers']);
 		}
 
-		$product = new Product( $post_id );
+		$product = new Product($post_id);
 
-		foreach ( $jsonld['offers'] as &$offer ) {
-			$this->shipping_zones->add_available_delivery_method( $offer );
-			$this->shipping_zones->add_offer_shipping_details( $offer, $product );
+		foreach ($jsonld['offers'] as &$offer) {
+			$this->shipping_zones->add_available_delivery_method($offer);
+			$this->shipping_zones->add_offer_shipping_details($offer, $product);
 		}
 
 		return $jsonld;

src/wordlift/shipping-data/class-product.php

Indentation +39 added lines, -39 removed lines

@@ -4,58 +4,58 @@
 
 class Product {
 
-	/**
-	 * @var false|\WC_Product|null
-	 */
-	private $wc_product;
+    /**
+     * @var false|\WC_Product|null
+     */
+    private $wc_product;
 
-	/**
-	 * Product constructor.
-	 *
-	 * @param $post_id
-	 */
-	public function __construct( $post_id ) {
+    /**
+     * Product constructor.
+     *
+     * @param $post_id
+     */
+    public function __construct( $post_id ) {
 
-		$this->wc_product = wc_get_product( $post_id );
+        $this->wc_product = wc_get_product( $post_id );
 
-	}
+    }
 
-	public function add_handling_time( &$shipping_delivery_time ) {
+    public function add_handling_time( &$shipping_delivery_time ) {
 
-		// Bail out if there's no product.
-		if ( empty( $this->wc_product ) ) {
-			return;
-		}
+        // Bail out if there's no product.
+        if ( empty( $this->wc_product ) ) {
+            return;
+        }
 
-		$shipping_class_id = $this->wc_product->get_shipping_class_id();
+        $shipping_class_id = $this->wc_product->get_shipping_class_id();
 
-		$option = get_option( 'wpsso_options' );
+        $option = get_option( 'wpsso_options' );
 
-		if ( empty( $option[ "wcsdt_handling_c{$shipping_class_id}_unit_code" ] )
-			 || empty( $option[ "wcsdt_handling_c{$shipping_class_id}_minimum" ] )
-			 || empty( $option[ "wcsdt_handling_c{$shipping_class_id}_maximum" ] ) ) {
-			return;
-		}
+        if ( empty( $option[ "wcsdt_handling_c{$shipping_class_id}_unit_code" ] )
+             || empty( $option[ "wcsdt_handling_c{$shipping_class_id}_minimum" ] )
+             || empty( $option[ "wcsdt_handling_c{$shipping_class_id}_maximum" ] ) ) {
+            return;
+        }
 
-		$unit_code = $option[ "wcsdt_handling_c{$shipping_class_id}_unit_code" ];
-		$minimum   = $option[ "wcsdt_handling_c{$shipping_class_id}_minimum" ];
-		$maximum   = $option[ "wcsdt_handling_c{$shipping_class_id}_maximum" ];
+        $unit_code = $option[ "wcsdt_handling_c{$shipping_class_id}_unit_code" ];
+        $minimum   = $option[ "wcsdt_handling_c{$shipping_class_id}_minimum" ];
+        $maximum   = $option[ "wcsdt_handling_c{$shipping_class_id}_maximum" ];
 
-		if ( 'HUR' === $unit_code ) {
-			$minimum = floor( $minimum / 24.0 );
-			$maximum = ceil( $maximum / 24.0 );
-		}
+        if ( 'HUR' === $unit_code ) {
+            $minimum = floor( $minimum / 24.0 );
+            $maximum = ceil( $maximum / 24.0 );
+        }
 
-		$shipping_delivery_time['handlingTime'] = array(
-			'@type'    => 'QuantitativeValue',
-			'minValue' => $minimum,
-			'maxValue' => $maximum,
-		);
+        $shipping_delivery_time['handlingTime'] = array(
+            '@type'    => 'QuantitativeValue',
+            'minValue' => $minimum,
+            'maxValue' => $maximum,
+        );
 
-	}
+    }
 
-	private function get_handling_time() {
+    private function get_handling_time() {
 
-	}
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -14,36 +14,36 @@
 	 *
 	 * @param $post_id
 	 */
-	public function __construct( $post_id ) {
+	public function __construct($post_id) {
 
-		$this->wc_product = wc_get_product( $post_id );
+		$this->wc_product = wc_get_product($post_id);
 
 	}
 
-	public function add_handling_time( &$shipping_delivery_time ) {
+	public function add_handling_time(&$shipping_delivery_time) {
 
 		// Bail out if there's no product.
-		if ( empty( $this->wc_product ) ) {
+		if (empty($this->wc_product)) {
 			return;
 		}
 
 		$shipping_class_id = $this->wc_product->get_shipping_class_id();
 
-		$option = get_option( 'wpsso_options' );
+		$option = get_option('wpsso_options');
 
-		if ( empty( $option[ "wcsdt_handling_c{$shipping_class_id}_unit_code" ] )
-			 || empty( $option[ "wcsdt_handling_c{$shipping_class_id}_minimum" ] )
-			 || empty( $option[ "wcsdt_handling_c{$shipping_class_id}_maximum" ] ) ) {
+		if (empty($option["wcsdt_handling_c{$shipping_class_id}_unit_code"])
+			 || empty($option["wcsdt_handling_c{$shipping_class_id}_minimum"])
+			 || empty($option["wcsdt_handling_c{$shipping_class_id}_maximum"])) {
 			return;
 		}
 
-		$unit_code = $option[ "wcsdt_handling_c{$shipping_class_id}_unit_code" ];
-		$minimum   = $option[ "wcsdt_handling_c{$shipping_class_id}_minimum" ];
-		$maximum   = $option[ "wcsdt_handling_c{$shipping_class_id}_maximum" ];
+		$unit_code = $option["wcsdt_handling_c{$shipping_class_id}_unit_code"];
+		$minimum   = $option["wcsdt_handling_c{$shipping_class_id}_minimum"];
+		$maximum   = $option["wcsdt_handling_c{$shipping_class_id}_maximum"];
 
-		if ( 'HUR' === $unit_code ) {
-			$minimum = floor( $minimum / 24.0 );
-			$maximum = ceil( $maximum / 24.0 );
+		if ('HUR' === $unit_code) {
+			$minimum = floor($minimum / 24.0);
+			$maximum = ceil($maximum / 24.0);
 		}
 
 		$shipping_delivery_time['handlingTime'] = array(

src/wordlift/shipping-data/class-shipping-zones.php

Indentation +56 added lines, -56 removed lines

@@ -6,81 +6,81 @@
 
 class Shipping_Zones {
 
-	/**
-	 * @var Shipping_Zone[]
-	 */
-	private $zones;
+    /**
+     * @var Shipping_Zone[]
+     */
+    private $zones;
 
-	public function __construct() {
-		$this->zones = array();
-	}
+    public function __construct() {
+        $this->zones = array();
+    }
 
-	private function load_zones() {
+    private function load_zones() {
 
-		$zone_ids          = array_keys( WC_Shipping_Zones::get_zones() );
-		$wc_shipping_zones = array( WC_Shipping_Zones::get_zone( 0 ) );
-		foreach ( $zone_ids as $zone_id ) {
-			$wc_shipping_zones[] = WC_Shipping_Zones::get_zone( $zone_id );
-		}
+        $zone_ids          = array_keys( WC_Shipping_Zones::get_zones() );
+        $wc_shipping_zones = array( WC_Shipping_Zones::get_zone( 0 ) );
+        foreach ( $zone_ids as $zone_id ) {
+            $wc_shipping_zones[] = WC_Shipping_Zones::get_zone( $zone_id );
+        }
 
-		$zones = array();
-		foreach ( $wc_shipping_zones as $wc_shipping_zone ) {
-			$country_codes = $this->get_country_codes( $wc_shipping_zone->get_zone_locations() );
+        $zones = array();
+        foreach ( $wc_shipping_zones as $wc_shipping_zone ) {
+            $country_codes = $this->get_country_codes( $wc_shipping_zone->get_zone_locations() );
 
-			if ( empty( $country_codes ) ) {
-				$zones[] = Shipping_Zone::from_wc_shipping_zone( $wc_shipping_zone );
-			} else {
-				foreach ( $country_codes as $country_code ) {
-					$zones[] = Shipping_Zone::from_wc_shipping_zone( $wc_shipping_zone, $country_code );
-				}
-			}
-		}
+            if ( empty( $country_codes ) ) {
+                $zones[] = Shipping_Zone::from_wc_shipping_zone( $wc_shipping_zone );
+            } else {
+                foreach ( $country_codes as $country_code ) {
+                    $zones[] = Shipping_Zone::from_wc_shipping_zone( $wc_shipping_zone, $country_code );
+                }
+            }
+        }
 
-		$this->zones = $zones;
+        $this->zones = $zones;
 
-	}
+    }
 
-	public function add_available_delivery_method( &$jsonld ) {
+    public function add_available_delivery_method( &$jsonld ) {
 
-		$this->load_zones();
+        $this->load_zones();
 
-		foreach ( $this->zones as $zone ) {
-			$zone->add_available_delivery_method( $jsonld );
-		}
+        foreach ( $this->zones as $zone ) {
+            $zone->add_available_delivery_method( $jsonld );
+        }
 
-	}
+    }
 
-	/**
-	 * @param array   $jsonld
-	 * @param Product $product
-	 */
-	public function add_offer_shipping_details( &$jsonld, $product ) {
+    /**
+     * @param array   $jsonld
+     * @param Product $product
+     */
+    public function add_offer_shipping_details( &$jsonld, $product ) {
 
-		$this->load_zones();
+        $this->load_zones();
 
-		foreach ( $this->zones as $zone ) {
-			$zone->add_offer_shipping_details( $jsonld, $product );
-		}
+        foreach ( $this->zones as $zone ) {
+            $zone->add_offer_shipping_details( $jsonld, $product );
+        }
 
-	}
+    }
 
-	private function get_country_codes( $wc_shipping_zones ) {
-		$countries = array();
+    private function get_country_codes( $wc_shipping_zones ) {
+        $countries = array();
 
-		foreach ( $wc_shipping_zones as $wc_shipping_zone ) {
-			if ( ! isset( $wc_shipping_zone->type )
-				 || ( 'country' !== $wc_shipping_zone->type && 'state' !== $wc_shipping_zone->type ) ) {
-				continue;
-			}
+        foreach ( $wc_shipping_zones as $wc_shipping_zone ) {
+            if ( ! isset( $wc_shipping_zone->type )
+                 || ( 'country' !== $wc_shipping_zone->type && 'state' !== $wc_shipping_zone->type ) ) {
+                continue;
+            }
 
-			$country = substr( $wc_shipping_zone->code, 0, 2 );
+            $country = substr( $wc_shipping_zone->code, 0, 2 );
 
-			if ( ! in_array( $country, $countries, true ) ) {
-				$countries[] = $country;
-			}
-		}
+            if ( ! in_array( $country, $countries, true ) ) {
+                $countries[] = $country;
+            }
+        }
 
-		return $countries;
-	}
+        return $countries;
+    }
 
 }

Spacing +22 added lines, -22 removed lines

@@ -17,21 +17,21 @@ 
 
 	private function load_zones() {
 
-		$zone_ids          = array_keys( WC_Shipping_Zones::get_zones() );
-		$wc_shipping_zones = array( WC_Shipping_Zones::get_zone( 0 ) );
-		foreach ( $zone_ids as $zone_id ) {
-			$wc_shipping_zones[] = WC_Shipping_Zones::get_zone( $zone_id );
+		$zone_ids          = array_keys(WC_Shipping_Zones::get_zones());
+		$wc_shipping_zones = array(WC_Shipping_Zones::get_zone(0));
+		foreach ($zone_ids as $zone_id) {
+			$wc_shipping_zones[] = WC_Shipping_Zones::get_zone($zone_id);
 		}
 
 		$zones = array();
-		foreach ( $wc_shipping_zones as $wc_shipping_zone ) {
-			$country_codes = $this->get_country_codes( $wc_shipping_zone->get_zone_locations() );
+		foreach ($wc_shipping_zones as $wc_shipping_zone) {
+			$country_codes = $this->get_country_codes($wc_shipping_zone->get_zone_locations());
 
-			if ( empty( $country_codes ) ) {
-				$zones[] = Shipping_Zone::from_wc_shipping_zone( $wc_shipping_zone );
+			if (empty($country_codes)) {
+				$zones[] = Shipping_Zone::from_wc_shipping_zone($wc_shipping_zone);
 			} else {
-				foreach ( $country_codes as $country_code ) {
-					$zones[] = Shipping_Zone::from_wc_shipping_zone( $wc_shipping_zone, $country_code );
+				foreach ($country_codes as $country_code) {
+					$zones[] = Shipping_Zone::from_wc_shipping_zone($wc_shipping_zone, $country_code);
 				}
 			}
 		}
@@ -40,12 +40,12 @@ 
 
 	}
 
-	public function add_available_delivery_method( &$jsonld ) {
+	public function add_available_delivery_method(&$jsonld) {
 
 		$this->load_zones();
 
-		foreach ( $this->zones as $zone ) {
-			$zone->add_available_delivery_method( $jsonld );
+		foreach ($this->zones as $zone) {
+			$zone->add_available_delivery_method($jsonld);
 		}
 
 	}
@@ -54,28 +54,28 @@ 
 	 * @param array   $jsonld
 	 * @param Product $product
 	 */
-	public function add_offer_shipping_details( &$jsonld, $product ) {
+	public function add_offer_shipping_details(&$jsonld, $product) {
 
 		$this->load_zones();
 
-		foreach ( $this->zones as $zone ) {
-			$zone->add_offer_shipping_details( $jsonld, $product );
+		foreach ($this->zones as $zone) {
+			$zone->add_offer_shipping_details($jsonld, $product);
 		}
 
 	}
 
-	private function get_country_codes( $wc_shipping_zones ) {
+	private function get_country_codes($wc_shipping_zones) {
 		$countries = array();
 
-		foreach ( $wc_shipping_zones as $wc_shipping_zone ) {
-			if ( ! isset( $wc_shipping_zone->type )
-				 || ( 'country' !== $wc_shipping_zone->type && 'state' !== $wc_shipping_zone->type ) ) {
+		foreach ($wc_shipping_zones as $wc_shipping_zone) {
+			if ( ! isset($wc_shipping_zone->type)
+				 || ('country' !== $wc_shipping_zone->type && 'state' !== $wc_shipping_zone->type)) {
 				continue;
 			}
 
-			$country = substr( $wc_shipping_zone->code, 0, 2 );
+			$country = substr($wc_shipping_zone->code, 0, 2);
 
-			if ( ! in_array( $country, $countries, true ) ) {
+			if ( ! in_array($country, $countries, true)) {
 				$countries[] = $country;
 			}
 		}

src/wordlift/shipping-data/index.php

Indentation +6 added lines, -6 removed lines

@@ -4,16 +4,16 @@
 use Wordlift\Shipping_Data\Shipping_Zones;
 
 if ( ! class_exists( '\WC_Shipping_Zones' )
-	 || ! class_exists( '\WC_Shipping_Zone' )
-	 || ! class_exists( '\WC_Shipping_Method' )
-	 || ! function_exists( 'get_woocommerce_currency' )
-	 || ! function_exists( 'wc_get_product' ) ) {
-	return;
+     || ! class_exists( '\WC_Shipping_Zone' )
+     || ! class_exists( '\WC_Shipping_Method' )
+     || ! function_exists( 'get_woocommerce_currency' )
+     || ! function_exists( 'wc_get_product' ) ) {
+    return;
 }
 
 // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
 if ( ! apply_filters( 'wl_feature__enable__shipping-sd', false ) ) {
-	return;
+    return;
 }
 
 $shipping_zones = new Shipping_Zones();

Spacing +7 added lines, -7 removed lines

@@ -3,18 +3,18 @@
 use Wordlift\Shipping_Data\Offer_Structured_Data;
 use Wordlift\Shipping_Data\Shipping_Zones;
 
-if ( ! class_exists( '\WC_Shipping_Zones' )
-	 || ! class_exists( '\WC_Shipping_Zone' )
-	 || ! class_exists( '\WC_Shipping_Method' )
-	 || ! function_exists( 'get_woocommerce_currency' )
-	 || ! function_exists( 'wc_get_product' ) ) {
+if ( ! class_exists('\WC_Shipping_Zones')
+	 || ! class_exists('\WC_Shipping_Zone')
+	 || ! class_exists('\WC_Shipping_Method')
+	 || ! function_exists('get_woocommerce_currency')
+	 || ! function_exists('wc_get_product')) {
 	return;
 }
 
 // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-if ( ! apply_filters( 'wl_feature__enable__shipping-sd', false ) ) {
+if ( ! apply_filters('wl_feature__enable__shipping-sd', false)) {
 	return;
 }
 
 $shipping_zones = new Shipping_Zones();
-new Offer_Structured_Data( $shipping_zones );
+new Offer_Structured_Data($shipping_zones);

src/wordlift/common/editor/class-edit-post-loader.php

Indentation +9 added lines, -9 removed lines

@@ -10,19 +10,19 @@
  */
 abstract class Edit_Post_Loader {
 
-	public function init() {
+    public function init() {
 
-		add_action( 'admin_enqueue_scripts', array( $this, 'post_edit_screen' ) );
-	}
+        add_action( 'admin_enqueue_scripts', array( $this, 'post_edit_screen' ) );
+    }
 
-	public function post_edit_screen( $hook ) {
+    public function post_edit_screen( $hook ) {
 
-		if ( 'post.php' === $hook || 'post-new.php' === $hook ) {
-			$this->run_on_edit_post_screen();
-		}
+        if ( 'post.php' === $hook || 'post-new.php' === $hook ) {
+            $this->run_on_edit_post_screen();
+        }
 
-	}
+    }
 
-	abstract public function run_on_edit_post_screen();
+    abstract public function run_on_edit_post_screen();
 
 }

Spacing +3 added lines, -3 removed lines

@@ -12,12 +12,12 @@
 
 	public function init() {
 
-		add_action( 'admin_enqueue_scripts', array( $this, 'post_edit_screen' ) );
+		add_action('admin_enqueue_scripts', array($this, 'post_edit_screen'));
 	}
 
-	public function post_edit_screen( $hook ) {
+	public function post_edit_screen($hook) {
 
-		if ( 'post.php' === $hook || 'post-new.php' === $hook ) {
+		if ('post.php' === $hook || 'post-new.php' === $hook) {
 			$this->run_on_edit_post_screen();
 		}
 

src/wordlift/common/term-checklist/class-term-checklist.php

Indentation +24 added lines, -24 removed lines

@@ -10,36 +10,36 @@
  */
 class Term_Checklist {
 
-	/**
-	 * @param $input_name string The name of the input field assigned to checkbox.
-	 * @param $terms array<\WP_Term>
-	 * @param $selected_term_slugs array<string> The list of selected term slugs.
-	 *
-	 * @return string Html string to be rendered.
-	 */
-	public static function render( $input_name, $terms, $selected_term_slugs ) {
-
-		$terms_html = '';
-
-		$input_name = esc_html( $input_name );
-
-		foreach ( $terms as $term ) {
-
-			/**
-			 * @var $term \WP_Term
-			 */
-			$term_name   = esc_html( $term->name );
-			$checked     = in_array( $term->slug, $selected_term_slugs, true ) ? 'checked' : '';
-			$terms_html .= "<li id=\"wl_entity_type-{$term->term_id}\">
+    /**
+     * @param $input_name string The name of the input field assigned to checkbox.
+     * @param $terms array<\WP_Term>
+     * @param $selected_term_slugs array<string> The list of selected term slugs.
+     *
+     * @return string Html string to be rendered.
+     */
+    public static function render( $input_name, $terms, $selected_term_slugs ) {
+
+        $terms_html = '';
+
+        $input_name = esc_html( $input_name );
+
+        foreach ( $terms as $term ) {
+
+            /**
+             * @var $term \WP_Term
+             */
+            $term_name   = esc_html( $term->name );
+            $checked     = in_array( $term->slug, $selected_term_slugs, true ) ? 'checked' : '';
+            $terms_html .= "<li id=\"wl_entity_type-{$term->term_id}\">
 	<label class=\"selectit\">
 	<input value=\"{$term->slug}\" type=\"checkbox\" name=\"{$input_name}[]\" id=\"in-wl_entity_type-{$term->term_id}\" $checked>
 		${term_name}
 	</label>
 </li>";
 
-		}
+        }
 
-		return '<input type="text" id="wl_entity_type_search" placeholder="Search Entity Types.." /><ul id="wl-entity-type__ul">' . $terms_html . '</ul>';
-	}
+        return '<input type="text" id="wl_entity_type_search" placeholder="Search Entity Types.." /><ul id="wl-entity-type__ul">' . $terms_html . '</ul>';
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -17,19 +17,19 @@ 
 	 *
 	 * @return string Html string to be rendered.
 	 */
-	public static function render( $input_name, $terms, $selected_term_slugs ) {
+	public static function render($input_name, $terms, $selected_term_slugs) {
 
 		$terms_html = '';
 
-		$input_name = esc_html( $input_name );
+		$input_name = esc_html($input_name);
 
-		foreach ( $terms as $term ) {
+		foreach ($terms as $term) {
 
 			/**
 			 * @var $term \WP_Term
 			 */
-			$term_name   = esc_html( $term->name );
-			$checked     = in_array( $term->slug, $selected_term_slugs, true ) ? 'checked' : '';
+			$term_name   = esc_html($term->name);
+			$checked     = in_array($term->slug, $selected_term_slugs, true) ? 'checked' : '';
 			$terms_html .= "<li id=\"wl_entity_type-{$term->term_id}\">
 	<label class=\"selectit\">
 	<input value=\"{$term->slug}\" type=\"checkbox\" name=\"{$input_name}[]\" id=\"in-wl_entity_type-{$term->term_id}\" $checked>
@@ -39,7 +39,7 @@ 
 
 		}
 
-		return '<input type="text" id="wl_entity_type_search" placeholder="Search Entity Types.." /><ul id="wl-entity-type__ul">' . $terms_html . '</ul>';
+		return '<input type="text" id="wl_entity_type_search" placeholder="Search Entity Types.." /><ul id="wl-entity-type__ul">'.$terms_html.'</ul>';
 	}
 
 }

src/wordlift/common/background-process/class-background-process.php

Indentation +196 added lines, -196 removed lines

@@ -3,201 +3,201 @@
 namespace Wordlift\Common\Background_Process;
 
 abstract class Background_Process extends \Wordlift_Plugin_WP_Background_Process {
-	/**
-	 * @var \Wordlift_Log_Service
-	 */
-	private $log;
-	/**
-	 * @var Data_Source
-	 */
-	protected $data_source;
-
-	/**
-	 * Background_Process constructor.
-	 *
-	 * @param $data_source Data_Source
-	 */
-	public function __construct( $data_source ) {
-		parent::__construct();
-		$this->data_source = $data_source;
-		$this->log         = \Wordlift_Log_Service::get_logger( get_class() );
-		// Set value for action key.
-		$this->action = $this->get_action_key();
-	}
-
-	/**
-	 * The key which is used to store the Sync_State class for the current process
-	 *
-	 * @return string
-	 */
-	abstract protected function get_state_storage_key();
-
-	/**
-	 * The key which is used as prefix to store the options.
-	 *
-	 * @return string
-	 */
-	abstract protected function get_action_key();
-
-	/**
-	 * Check whether the process must cancel or not.
-	 *
-	 * @return bool Whether to cancel or not the process.
-	 */
-	private function must_cancel() {
-
-		return get_transient( "{$this->action}__cancel" );
-	}
-
-	/**
-	 * Check whether the provided state is `started` or not.
-	 *
-	 * @param Sync_State $state The {@link Sync_State}.
-	 *
-	 * @return bool True if the state is started.
-	 */
-	private function is_started( $state ) {
-		return $state instanceof Sync_State && 'started' === $state->state && 30 > ( time() - $state->last_update );
-	}
-
-	/**
-	 * Start the background processing.
-	 *
-	 * @return bool True if the process has been started, otherwise false.
-	 */
-	public function start() {
-		$action = $this->get_action_key();
-		$this->log->debug( "Trying to start  ${action}..." );
-		// Create a new Sync_Model state of `started`.
-		if ( ! $this->is_started( self::get_state() ) ) {
-			$this->log->debug( 'Starting...' );
-
-			$sync_state = new Sync_State( time(), 0, $this->data_source->count(), time(), 'started' );
-			update_option( $this->get_state_storage_key(), $sync_state, false );
-
-			$next = $this->data_source->next();
-
-			$this->push_to_queue( $next );
-			$this->save()->dispatch();
-
-			if ( $next && is_array( $next ) ) {
-				$this->log->debug( sprintf( 'Started with term IDs %s.', implode( ', ', $next ) ) );
-			}
-
-			return true;
-		}
-
-		return false;
-	}
-
-	/**
-	 * Cancels the current process.
-	 */
-	public function cancel() {
-
-		$action = $this->action;
-		$this->log->debug( "Cancelling ${action}..." );
-
-		// Cleanup the process data.
-		$this->cancel_process();
-
-		// Set the state to cancelled.
-		$state = $this->get_state();
-		$state->set_state( 'cancelled' );
-		update_option( $this->get_state_storage_key(), $state, false );
-
-		// Finally delete the transient.
-		delete_transient( "{$this->action}__cancel" );
-
-	}
-
-	/**
-	 * Get the sync state.
-	 *
-	 * @return Sync_State The {@link Sync_State}.
-	 */
-	public function get_state() {
-
-		try {
-			return get_option( $this->get_state_storage_key(), Sync_State::unknown() );
-		} catch ( \Exception $e ) {
-			return Sync_State::unknown();
-		}
-
-	}
-
-	/**
-	 * This function is called:
-	 *  - To start a new Synchronization, by passing a {@link Sync_Start_Message} instance.
-	 *  - To process a item, by passing a numeric ID.
-	 *
-	 * This function returns the parameter for the next call or NULL if there are no more items to process.
-	 *
-	 * @param int[] $items An array of item IDs.
-	 *
-	 * @return int[]|false The next IDs or false if there are no more.
-	 */
-	protected function task( $items ) {
-
-		// Check if we must cancel.
-		if ( $this->must_cancel() || ! $items ) {
-			$this->log->debug( 'Cancelling background process ' . $this->action . ' due to no items inside task() method' );
-			$this->cancel();
-
-			return false;
-		}
-
-		if ( $items && is_array( $items ) ) {
-			$this->log->debug( sprintf( 'Synchronizing items %s...', implode( ', ', $items ) ) );
-		}
-		// Sync the item.
-		if ( $this->process_items( $items ) ) {
-			// Update the process state, set the index in state in order
-			// to reflect the new items.
-			$this->update_batch_index();
-
-			// Get the next batch for processing.
-			return $this->get_next_batch();
-		} else {
-			// Return the failed term ids again.
-			return $items;
-		}
-	}
-
-	/**
-	 * Process all the items in the current batch.
-	 *
-	 * @param $items
-	 *
-	 * @return bool If all items are successfully processed.
-	 */
-	abstract protected function process_items( $items );
-
-	/**
-	 * Return next batch of items after processing.
-	 *
-	 * @return int[] or false
-	 */
-	protected function get_next_batch() {
-		return $this->data_source->next();
-	}
-
-	private function update_batch_index() {
-		$next = $this->data_source->next();
-
-		$next_state = isset( $next ) ? 'started' : 'ended';
-
-		/**
-		 * Update the synchronization meta data, by increasing the current index.
-		 *
-		 * @var Sync_State $sync The {@link Sync_State}.
-		 */
-		$state = self::get_state()
-					 ->increment_index( $this->data_source->get_batch_size() )
-					 ->set_state( $next_state );
-
-		update_option( $this->get_state_storage_key(), $state, false );
-
-	}
+    /**
+     * @var \Wordlift_Log_Service
+     */
+    private $log;
+    /**
+     * @var Data_Source
+     */
+    protected $data_source;
+
+    /**
+     * Background_Process constructor.
+     *
+     * @param $data_source Data_Source
+     */
+    public function __construct( $data_source ) {
+        parent::__construct();
+        $this->data_source = $data_source;
+        $this->log         = \Wordlift_Log_Service::get_logger( get_class() );
+        // Set value for action key.
+        $this->action = $this->get_action_key();
+    }
+
+    /**
+     * The key which is used to store the Sync_State class for the current process
+     *
+     * @return string
+     */
+    abstract protected function get_state_storage_key();
+
+    /**
+     * The key which is used as prefix to store the options.
+     *
+     * @return string
+     */
+    abstract protected function get_action_key();
+
+    /**
+     * Check whether the process must cancel or not.
+     *
+     * @return bool Whether to cancel or not the process.
+     */
+    private function must_cancel() {
+
+        return get_transient( "{$this->action}__cancel" );
+    }
+
+    /**
+     * Check whether the provided state is `started` or not.
+     *
+     * @param Sync_State $state The {@link Sync_State}.
+     *
+     * @return bool True if the state is started.
+     */
+    private function is_started( $state ) {
+        return $state instanceof Sync_State && 'started' === $state->state && 30 > ( time() - $state->last_update );
+    }
+
+    /**
+     * Start the background processing.
+     *
+     * @return bool True if the process has been started, otherwise false.
+     */
+    public function start() {
+        $action = $this->get_action_key();
+        $this->log->debug( "Trying to start  ${action}..." );
+        // Create a new Sync_Model state of `started`.
+        if ( ! $this->is_started( self::get_state() ) ) {
+            $this->log->debug( 'Starting...' );
+
+            $sync_state = new Sync_State( time(), 0, $this->data_source->count(), time(), 'started' );
+            update_option( $this->get_state_storage_key(), $sync_state, false );
+
+            $next = $this->data_source->next();
+
+            $this->push_to_queue( $next );
+            $this->save()->dispatch();
+
+            if ( $next && is_array( $next ) ) {
+                $this->log->debug( sprintf( 'Started with term IDs %s.', implode( ', ', $next ) ) );
+            }
+
+            return true;
+        }
+
+        return false;
+    }
+
+    /**
+     * Cancels the current process.
+     */
+    public function cancel() {
+
+        $action = $this->action;
+        $this->log->debug( "Cancelling ${action}..." );
+
+        // Cleanup the process data.
+        $this->cancel_process();
+
+        // Set the state to cancelled.
+        $state = $this->get_state();
+        $state->set_state( 'cancelled' );
+        update_option( $this->get_state_storage_key(), $state, false );
+
+        // Finally delete the transient.
+        delete_transient( "{$this->action}__cancel" );
+
+    }
+
+    /**
+     * Get the sync state.
+     *
+     * @return Sync_State The {@link Sync_State}.
+     */
+    public function get_state() {
+
+        try {
+            return get_option( $this->get_state_storage_key(), Sync_State::unknown() );
+        } catch ( \Exception $e ) {
+            return Sync_State::unknown();
+        }
+
+    }
+
+    /**
+     * This function is called:
+     *  - To start a new Synchronization, by passing a {@link Sync_Start_Message} instance.
+     *  - To process a item, by passing a numeric ID.
+     *
+     * This function returns the parameter for the next call or NULL if there are no more items to process.
+     *
+     * @param int[] $items An array of item IDs.
+     *
+     * @return int[]|false The next IDs or false if there are no more.
+     */
+    protected function task( $items ) {
+
+        // Check if we must cancel.
+        if ( $this->must_cancel() || ! $items ) {
+            $this->log->debug( 'Cancelling background process ' . $this->action . ' due to no items inside task() method' );
+            $this->cancel();
+
+            return false;
+        }
+
+        if ( $items && is_array( $items ) ) {
+            $this->log->debug( sprintf( 'Synchronizing items %s...', implode( ', ', $items ) ) );
+        }
+        // Sync the item.
+        if ( $this->process_items( $items ) ) {
+            // Update the process state, set the index in state in order
+            // to reflect the new items.
+            $this->update_batch_index();
+
+            // Get the next batch for processing.
+            return $this->get_next_batch();
+        } else {
+            // Return the failed term ids again.
+            return $items;
+        }
+    }
+
+    /**
+     * Process all the items in the current batch.
+     *
+     * @param $items
+     *
+     * @return bool If all items are successfully processed.
+     */
+    abstract protected function process_items( $items );
+
+    /**
+     * Return next batch of items after processing.
+     *
+     * @return int[] or false
+     */
+    protected function get_next_batch() {
+        return $this->data_source->next();
+    }
+
+    private function update_batch_index() {
+        $next = $this->data_source->next();
+
+        $next_state = isset( $next ) ? 'started' : 'ended';
+
+        /**
+         * Update the synchronization meta data, by increasing the current index.
+         *
+         * @var Sync_State $sync The {@link Sync_State}.
+         */
+        $state = self::get_state()
+                        ->increment_index( $this->data_source->get_batch_size() )
+                        ->set_state( $next_state );
+
+        update_option( $this->get_state_storage_key(), $state, false );
+
+    }
 
 }

Spacing +30 added lines, -30 removed lines

@@ -17,10 +17,10 @@ 
 	 *
 	 * @param $data_source Data_Source
 	 */
-	public function __construct( $data_source ) {
+	public function __construct($data_source) {
 		parent::__construct();
 		$this->data_source = $data_source;
-		$this->log         = \Wordlift_Log_Service::get_logger( get_class() );
+		$this->log         = \Wordlift_Log_Service::get_logger(get_class());
 		// Set value for action key.
 		$this->action = $this->get_action_key();
 	}
@@ -46,7 +46,7 @@ 
 	 */
 	private function must_cancel() {
 
-		return get_transient( "{$this->action}__cancel" );
+		return get_transient("{$this->action}__cancel");
 	}
 
 	/**
@@ -56,8 +56,8 @@ 
 	 *
 	 * @return bool True if the state is started.
 	 */
-	private function is_started( $state ) {
-		return $state instanceof Sync_State && 'started' === $state->state && 30 > ( time() - $state->last_update );
+	private function is_started($state) {
+		return $state instanceof Sync_State && 'started' === $state->state && 30 > (time() - $state->last_update);
 	}
 
 	/**
@@ -67,21 +67,21 @@ 
 	 */
 	public function start() {
 		$action = $this->get_action_key();
-		$this->log->debug( "Trying to start  ${action}..." );
+		$this->log->debug("Trying to start  ${action}...");
 		// Create a new Sync_Model state of `started`.
-		if ( ! $this->is_started( self::get_state() ) ) {
-			$this->log->debug( 'Starting...' );
+		if ( ! $this->is_started(self::get_state())) {
+			$this->log->debug('Starting...');
 
-			$sync_state = new Sync_State( time(), 0, $this->data_source->count(), time(), 'started' );
-			update_option( $this->get_state_storage_key(), $sync_state, false );
+			$sync_state = new Sync_State(time(), 0, $this->data_source->count(), time(), 'started');
+			update_option($this->get_state_storage_key(), $sync_state, false);
 
 			$next = $this->data_source->next();
 
-			$this->push_to_queue( $next );
+			$this->push_to_queue($next);
 			$this->save()->dispatch();
 
-			if ( $next && is_array( $next ) ) {
-				$this->log->debug( sprintf( 'Started with term IDs %s.', implode( ', ', $next ) ) );
+			if ($next && is_array($next)) {
+				$this->log->debug(sprintf('Started with term IDs %s.', implode(', ', $next)));
 			}
 
 			return true;
@@ -96,18 +96,18 @@ 
 	public function cancel() {
 
 		$action = $this->action;
-		$this->log->debug( "Cancelling ${action}..." );
+		$this->log->debug("Cancelling ${action}...");
 
 		// Cleanup the process data.
 		$this->cancel_process();
 
 		// Set the state to cancelled.
 		$state = $this->get_state();
-		$state->set_state( 'cancelled' );
-		update_option( $this->get_state_storage_key(), $state, false );
+		$state->set_state('cancelled');
+		update_option($this->get_state_storage_key(), $state, false);
 
 		// Finally delete the transient.
-		delete_transient( "{$this->action}__cancel" );
+		delete_transient("{$this->action}__cancel");
 
 	}
 
@@ -119,8 +119,8 @@ 
 	public function get_state() {
 
 		try {
-			return get_option( $this->get_state_storage_key(), Sync_State::unknown() );
-		} catch ( \Exception $e ) {
+			return get_option($this->get_state_storage_key(), Sync_State::unknown());
+		} catch (\Exception $e) {
 			return Sync_State::unknown();
 		}
 
@@ -137,21 +137,21 @@ 
 	 *
 	 * @return int[]|false The next IDs or false if there are no more.
 	 */
-	protected function task( $items ) {
+	protected function task($items) {
 
 		// Check if we must cancel.
-		if ( $this->must_cancel() || ! $items ) {
-			$this->log->debug( 'Cancelling background process ' . $this->action . ' due to no items inside task() method' );
+		if ($this->must_cancel() || ! $items) {
+			$this->log->debug('Cancelling background process '.$this->action.' due to no items inside task() method');
 			$this->cancel();
 
 			return false;
 		}
 
-		if ( $items && is_array( $items ) ) {
-			$this->log->debug( sprintf( 'Synchronizing items %s...', implode( ', ', $items ) ) );
+		if ($items && is_array($items)) {
+			$this->log->debug(sprintf('Synchronizing items %s...', implode(', ', $items)));
 		}
 		// Sync the item.
-		if ( $this->process_items( $items ) ) {
+		if ($this->process_items($items)) {
 			// Update the process state, set the index in state in order
 			// to reflect the new items.
 			$this->update_batch_index();
@@ -171,7 +171,7 @@ 
 	 *
 	 * @return bool If all items are successfully processed.
 	 */
-	abstract protected function process_items( $items );
+	abstract protected function process_items($items);
 
 	/**
 	 * Return next batch of items after processing.
@@ -185,7 +185,7 @@ 
 	private function update_batch_index() {
 		$next = $this->data_source->next();
 
-		$next_state = isset( $next ) ? 'started' : 'ended';
+		$next_state = isset($next) ? 'started' : 'ended';
 
 		/**
 		 * Update the synchronization meta data, by increasing the current index.
@@ -193,10 +193,10 @@ 
 		 * @var Sync_State $sync The {@link Sync_State}.
 		 */
 		$state = self::get_state()
-					 ->increment_index( $this->data_source->get_batch_size() )
-					 ->set_state( $next_state );
+					 ->increment_index($this->data_source->get_batch_size())
+					 ->set_state($next_state);
 
-		update_option( $this->get_state_storage_key(), $state, false );
+		update_option($this->get_state_storage_key(), $state, false);
 
 	}
 

src/wordlift/common/loader/class-loader.php

Indentation +6 added lines, -6 removed lines

@@ -9,11 +9,11 @@
 
 interface Loader {
 
-	/**
-	 * The loader should register the feature to feature registry.
-	 *
-	 * @return void
-	 */
-	public function init_feature();
+    /**
+     * The loader should register the feature to feature registry.
+     *
+     * @return void
+     */
+    public function init_feature();
 
 }

src/wordlift-editor.php

Indentation +25 added lines, -25 removed lines

@@ -9,32 +9,32 @@ 
  */
 function wordlift_mce_css( $mce_css ) {
 
-	/*
+    /*
 	 * Call the `wl_can_see_classification_box` filter to determine whether we can display the classification box.
 	 *
 	 * @since 3.20.3
 	 *
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/914
 	 */
-	if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
-		return $mce_css;
-	}
-
-	if ( ! empty( $mce_css ) ) {
-		$mce_css .= ',';
-	}
-
-	/**
-	 * Replacing the legacy `wordlift-reloaded.min.css` with tiny-mce.css.
-	 *
-	 * tiny-mce.css is generated using the new webpack project and its rules are shared with Gutenberg.
-	 *
-	 * @author David Riccitelli <david@wordlift.io>
-	 * @since 3.23.0
-	 */
-	$mce_css .= plugin_dir_url( __FILE__ ) . 'js/dist/tiny-mce.css';
-
-	return $mce_css;
+    if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
+        return $mce_css;
+    }
+
+    if ( ! empty( $mce_css ) ) {
+        $mce_css .= ',';
+    }
+
+    /**
+     * Replacing the legacy `wordlift-reloaded.min.css` with tiny-mce.css.
+     *
+     * tiny-mce.css is generated using the new webpack project and its rules are shared with Gutenberg.
+     *
+     * @author David Riccitelli <david@wordlift.io>
+     * @since 3.23.0
+     */
+    $mce_css .= plugin_dir_url( __FILE__ ) . 'js/dist/tiny-mce.css';
+
+    return $mce_css;
 }
 
 // hook the TinyMCE custom styles function.
@@ -49,13 +49,13 @@ 
  */
 function wordlift_filter_tiny_mce_before_init( $options ) {
 
-	if ( ! isset( $options['extended_valid_elements'] ) ) {
-		$options['extended_valid_elements'] = '';
-	}
+    if ( ! isset( $options['extended_valid_elements'] ) ) {
+        $options['extended_valid_elements'] = '';
+    }
 
-	$options['extended_valid_elements'] .= ',span[*]';
+    $options['extended_valid_elements'] .= ',span[*]';
 
-	return $options;
+    return $options;
 }
 
 add_filter( 'tiny_mce_before_init', 'wordlift_filter_tiny_mce_before_init', PHP_INT_MAX );

Spacing +8 added lines, -8 removed lines

@@ -7,7 +7,7 @@ 
  *
  * @return string The updated list of styles, including the custom style provided by WordLift.
  */
-function wordlift_mce_css( $mce_css ) {
+function wordlift_mce_css($mce_css) {
 
 	/*
 	 * Call the `wl_can_see_classification_box` filter to determine whether we can display the classification box.
@@ -16,11 +16,11 @@ 
 	 *
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/914
 	 */
-	if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
+	if ( ! apply_filters('wl_can_see_classification_box', true)) {
 		return $mce_css;
 	}
 
-	if ( ! empty( $mce_css ) ) {
+	if ( ! empty($mce_css)) {
 		$mce_css .= ',';
 	}
 
@@ -32,13 +32,13 @@ 
 	 * @author David Riccitelli <david@wordlift.io>
 	 * @since 3.23.0
 	 */
-	$mce_css .= plugin_dir_url( __FILE__ ) . 'js/dist/tiny-mce.css';
+	$mce_css .= plugin_dir_url(__FILE__).'js/dist/tiny-mce.css';
 
 	return $mce_css;
 }
 
 // hook the TinyMCE custom styles function.
-add_filter( 'mce_css', 'wordlift_mce_css' );
+add_filter('mce_css', 'wordlift_mce_css');
 
 /**
  * Set TinyMCE options, in particular enable microdata tagging.
@@ -47,9 +47,9 @@ 
  *
  * @return mixed
  */
-function wordlift_filter_tiny_mce_before_init( $options ) {
+function wordlift_filter_tiny_mce_before_init($options) {
 
-	if ( ! isset( $options['extended_valid_elements'] ) ) {
+	if ( ! isset($options['extended_valid_elements'])) {
 		$options['extended_valid_elements'] = '';
 	}
 
@@ -58,4 +58,4 @@ 
 	return $options;
 }
 
-add_filter( 'tiny_mce_before_init', 'wordlift_filter_tiny_mce_before_init', PHP_INT_MAX );
+add_filter('tiny_mce_before_init', 'wordlift_filter_tiny_mce_before_init', PHP_INT_MAX);