insideout10 / wordlift-plugin

src/modules/dashboard/includes/Post_Entity_Match/Query_Builder.php

Spacing +5 added lines, -5 removed lines

@@ -61,16 +61,16 @@ 
 		global $wpdb;
 
 		// If a value has been provided and it's either 'draft' or 'publish', we add the related filter.
-		if ( is_string( $this->params['post_status'] ) && in_array(
+		if (is_string($this->params['post_status']) && in_array(
 			$this->params['post_status'],
 			array(
 				'publish',
 				'draft',
 			),
 			true
-		) ) {
+		)) {
 
-			$this->sql .= $wpdb->prepare( ' AND p.post_status = %s', $this->params['post_status'] );
+			$this->sql .= $wpdb->prepare(' AND p.post_status = %s', $this->params['post_status']);
 
 			return $this;
 		}
@@ -84,10 +84,10 @@ 
 	public function post_type() {
 		$post_types = $this->params['post_types'];
 
-		if ( ! isset( $post_types ) ) {
+		if ( ! isset($post_types)) {
 			return $this;
 		}
-		$post_types_sql = Escape::sql_array( $post_types );
+		$post_types_sql = Escape::sql_array($post_types);
 		$this->sql     .= " AND p.post_type IN ({$post_types_sql}) ";
 
 		return $this;

src/modules/dashboard/includes/Post_Entity_Match/Sort.php

Spacing +3 added lines, -3 removed lines

@@ -17,10 +17,10 @@
 	 * @return string
 	 */
 	public function get_field_name() {
-		$tmp_sort_field_name = substr( $this->sort, 1 );
-		if ( 'id' === $tmp_sort_field_name ) {
+		$tmp_sort_field_name = substr($this->sort, 1);
+		if ('id' === $tmp_sort_field_name) {
 			return 'p.ID';
-		} elseif ( 'date_modified_gmt' === $tmp_sort_field_name ) {
+		} elseif ('date_modified_gmt' === $tmp_sort_field_name) {
 			return 'p.post_modified_gmt';
 		} else {
 			return 'p.post_title';

src/modules/dashboard/includes/Term_Entity_Match/Term_Entity_Match_Rest_Controller.php

Spacing +36 added lines, -36 removed lines

@@ -28,12 +28,12 @@ 
 	 *
 	 * @param $match_service
 	 */
-	public function __construct( $match_service ) {
+	public function __construct($match_service) {
 		$this->match_service = $match_service;
 	}
 
 	public function register_hooks() {
-		add_action( 'rest_api_init', array( $this, 'register_routes' ) );
+		add_action('rest_api_init', array($this, 'register_routes'));
 	}
 
 	/**
@@ -46,13 +46,13 @@ 
 			'/term-matches',
 			array(
 				'methods'             => 'GET',
-				'callback'            => array( $this, 'get_term_matches' ),
+				'callback'            => array($this, 'get_term_matches'),
 				'args'                => array(
 					'cursor'                   => array(
 						'type'              => 'string',
 						'default'           => Cursor::EMPTY_CURSOR_AS_BASE64_STRING,
 						'validate_callback' => 'rest_validate_request_arg',
-						'sanitize_callback' => array( Cursor::class, 'rest_sanitize_request_arg' ),
+						'sanitize_callback' => array(Cursor::class, 'rest_sanitize_request_arg'),
 					),
 					'limit'                    => array(
 						'type'              => 'integer',
@@ -90,8 +90,8 @@ 
 						'validate_callback' => 'rest_validate_request_arg',
 					),
 				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -102,15 +102,15 @@ 
 			'/term-matches/(?P<term_id>\d+)/matches',
 			array(
 				'methods'             => 'POST',
-				'callback'            => array( $this, 'create_term_match' ),
+				'callback'            => array($this, 'create_term_match'),
 				'args'                => array(
 					'term_id' => array(
 						'required'          => true,
 						'validate_callback' => 'rest_validate_request_arg',
 					),
 				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -121,7 +121,7 @@ 
 			'/term-matches/(?P<term_id>\d+)/matches/(?P<match_id>\d+)',
 			array(
 				'methods'             => 'PUT',
-				'callback'            => array( $this, 'update_term_match' ),
+				'callback'            => array($this, 'update_term_match'),
 				'args'                => array(
 					'term_id'  => array(
 						'required'          => true,
@@ -132,8 +132,8 @@ 
 						'validate_callback' => 'rest_validate_request_arg',
 					),
 				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -146,27 +146,27 @@ 
 	 *
 	 * @throws \Exception If there was a problem getting the match.
 	 */
-	public function get_term_matches( $request ) {
+	public function get_term_matches($request) {
 
-		$cursor = $request->get_param( 'cursor' );
-		if ( $request->has_param( 'limit' ) ) {
-			$cursor['limit'] = $request->get_param( 'limit' );
+		$cursor = $request->get_param('cursor');
+		if ($request->has_param('limit')) {
+			$cursor['limit'] = $request->get_param('limit');
 		}
-		if ( $request->has_param( 'sort' ) ) {
-			$cursor['sort'] = $request->get_param( 'sort' );
+		if ($request->has_param('sort')) {
+			$cursor['sort'] = $request->get_param('sort');
 		}
-		if ( $request->has_param( 'taxonomies' ) ) {
-			$cursor['query']['taxonomies'] = $request->get_param( 'taxonomies' );
+		if ($request->has_param('taxonomies')) {
+			$cursor['query']['taxonomies'] = $request->get_param('taxonomies');
 		}
-		if ( $request->has_param( 'has_match' ) ) {
-			$cursor['query']['has_match'] = $request->get_param( 'has_match' );
+		if ($request->has_param('has_match')) {
+			$cursor['query']['has_match'] = $request->get_param('has_match');
 		}
-		if ( $request->has_param( 'ingredient_name_contains' ) ) {
-			$cursor['query']['ingredient_name_contains'] = $request->get_param( 'ingredient_name_contains' );
+		if ($request->has_param('ingredient_name_contains')) {
+			$cursor['query']['ingredient_name_contains'] = $request->get_param('ingredient_name_contains');
 		}
 
 		// Query.
-		$taxonomies = isset( $cursor['query']['taxonomies'] ) ? $cursor['query']['taxonomies'] : apply_filters(
+		$taxonomies = isset($cursor['query']['taxonomies']) ? $cursor['query']['taxonomies'] : apply_filters(
 			'wl_dashboard__post_entity_match__taxonomies',
 			array(
 				'post_tag',
@@ -174,8 +174,8 @@ 
 			)
 		);
 
-		$has_match                = isset( $cursor['query']['has_match'] ) ? $cursor['query']['has_match'] : null;
-		$ingredient_name_contains = isset( $cursor['query']['ingredient_name_contains'] ) ? $cursor['query']['ingredient_name_contains'] : null;
+		$has_match                = isset($cursor['query']['has_match']) ? $cursor['query']['has_match'] : null;
+		$ingredient_name_contains = isset($cursor['query']['ingredient_name_contains']) ? $cursor['query']['ingredient_name_contains'] : null;
 
 		$items = $this->match_service->list_items(
 			array(
@@ -211,15 +211,15 @@ 
 	 *
 	 * @throws \Exception If there was a problem creating the match.
 	 */
-	public function create_term_match( $request ) {
+	public function create_term_match($request) {
 
-		$term_id = $request->get_param( 'term_id' );
+		$term_id = $request->get_param('term_id');
 
 		// If we dont have a entry on the match table, then add one.
-		$content_id = Wordpress_Content_Id::create_term( $term_id );
-		if ( ! Wordpress_Content_Service::get_instance()->get_entity_id( $content_id ) ) {
-			$uri = Entity_Uri_Generator::create_uri( $content_id->get_type(), $content_id->get_id() );
-			Wordpress_Content_Service::get_instance()->set_entity_id( $content_id, $uri );
+		$content_id = Wordpress_Content_Id::create_term($term_id);
+		if ( ! Wordpress_Content_Service::get_instance()->get_entity_id($content_id)) {
+			$uri = Entity_Uri_Generator::create_uri($content_id->get_type(), $content_id->get_id());
+			Wordpress_Content_Service::get_instance()->set_entity_id($content_id, $uri);
 		}
 
 		$match_id = $this->match_service->get_id(
@@ -244,11 +244,11 @@ 
 	 *
 	 * @throws \Exception If there was a problem updating the match.
 	 */
-	public function update_term_match( $request ) {
+	public function update_term_match($request) {
 		return $this->match_service->set_jsonld(
-			$request->get_param( 'term_id' ),
+			$request->get_param('term_id'),
 			Object_Type_Enum::TERM,
-			$request->get_param( 'match_id' ),
+			$request->get_param('match_id'),
 			$request->get_json_params()
 		);
 	}

src/modules/dashboard/includes/Term_Entity_Match/Term_Entity_Match_Service.php

Spacing +14 added lines, -14 removed lines

@@ -21,7 +21,7 @@ 
 	 *
 	 * @throws \Exception If there was a problem generating the list items.
 	 */
-	public function list_items( $args ) {
+	public function list_items($args) {
 		global $wpdb;
 
 		$params = wp_parse_args(
@@ -42,24 +42,24 @@ 
 		/**
 		 * @var $sort Sort
 		 */
-		$sort = new Sort( $params['sort'] );
+		$sort = new Sort($params['sort']);
 
 		$query_builder = new Query_Builder(
 			$params,
 			$sort
 		);
-		$query         = $query_builder
+		$query = $query_builder
 			->get();
 
 		$items = $wpdb->get_results(
 		// Each function above is preparing `$sql` by using `$wpdb->prepare`.
 		// phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-			$wpdb->prepare( $query, Object_Type_Enum::TERM )
+			$wpdb->prepare($query, Object_Type_Enum::TERM)
 		);
 
-		$sort->apply( $items );
+		$sort->apply($items);
 
-		return $this->map( $items );
+		return $this->map($items);
 	}
 
 	/**
@@ -69,12 +69,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function map( array $items ) {
+	private function map(array $items) {
 		return array_map(
-			function ( $item ) {
-				$data              = json_decode( $item->match_jsonld, true );
-				$item->match_name  = $data && is_array( $data ) && array_key_exists( 'name', $data ) ? $data['name'] : null;
-				$item->occurrences = $this->get_term_occurrences( $item->id );
+			function($item) {
+				$data              = json_decode($item->match_jsonld, true);
+				$item->match_name  = $data && is_array($data) && array_key_exists('name', $data) ? $data['name'] : null;
+				$item->occurrences = $this->get_term_occurrences($item->id);
 
 				return $item;
 			},
@@ -89,10 +89,10 @@ 
 	 *
 	 * @return int
 	 */
-	private function get_term_occurrences( $term_id ) {
-		$term = get_term( $term_id );
+	private function get_term_occurrences($term_id) {
+		$term = get_term($term_id);
 
-		if ( ! is_wp_error( $term ) ) {
+		if ( ! is_wp_error($term)) {
 			return $term->count;
 		}
 

src/modules/dashboard/includes/Term_Entity_Match/Query_Builder.php

Spacing +4 added lines, -4 removed lines

@@ -53,10 +53,10 @@ 
 	public function taxonomy() {
 		$taxonomies = $this->params['taxonomies'];
 
-		if ( ! isset( $taxonomies ) ) {
+		if ( ! isset($taxonomies)) {
 			return $this;
 		}
-		$sql        = Escape::sql_array( $taxonomies );
+		$sql        = Escape::sql_array($taxonomies);
 		$this->sql .= " AND tt.taxonomy IN ($sql)";
 
 		return $this;
@@ -71,9 +71,9 @@ 
 		global $wpdb;
 
 		// If the ingredient_name_contains value is a non-empty string, add the filter
-		if ( is_string( $this->params['ingredient_name_contains'] ) && ! empty( $this->params['ingredient_name_contains'] ) ) {
+		if (is_string($this->params['ingredient_name_contains']) && ! empty($this->params['ingredient_name_contains'])) {
 			$ingredient_name_contains = $this->params['ingredient_name_contains'];
-			$this->sql               .= $wpdb->prepare( ' AND t.name LIKE %s', '%' . $ingredient_name_contains . '%' );
+			$this->sql               .= $wpdb->prepare(' AND t.name LIKE %s', '%'.$ingredient_name_contains.'%');
 		}
 
 		return $this;

src/modules/dashboard/includes/Post_Entity_Match/Post_Entity_Match_Service.php

Spacing +15 added lines, -15 removed lines

@@ -21,7 +21,7 @@ 
 	 *
 	 * @throws \Exception If there was a problem generating the list items.
 	 */
-	public function list_items( $args ) {
+	public function list_items($args) {
 		global $wpdb;
 
 		$params = wp_parse_args(
@@ -40,7 +40,7 @@ 
 		/**
 		 * @var $sort Sort
 		 */
-		$sort = new Sort( $params['sort'] );
+		$sort = new Sort($params['sort']);
 
 		$query_builder = new Query_Builder(
 			$params,
@@ -50,12 +50,12 @@ 
 		$items = $wpdb->get_results(
 		// Each function above is preparing `$sql` by using `$wpdb->prepare`.
 		// phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-			$wpdb->prepare( $query_builder->get(), Object_Type_Enum::POST )
+			$wpdb->prepare($query_builder->get(), Object_Type_Enum::POST)
 		);
 
-		$sort->apply( $items );
+		$sort->apply($items);
 
-		return $this->map( $items );
+		return $this->map($items);
 	}
 
 	/**
@@ -78,20 +78,20 @@ 
 	 *
 	 * @return array
 	 */
-	private function map( array $items ) {
+	private function map(array $items) {
 		return array_map(
-			function ( $item ) {
-				$data             = json_decode( $item->match_jsonld, true );
-				$item->match_name = $data && is_array( $data ) && array_key_exists( 'name', $data ) ? $data['name'] : null;
+			function($item) {
+				$data             = json_decode($item->match_jsonld, true);
+				$item->match_name = $data && is_array($data) && array_key_exists('name', $data) ? $data['name'] : null;
 
-				if ( $item->id ) {
-					$item->post_link    = get_edit_post_link( $item->id, 'ui' );
-					$item->view_link    = get_permalink( $item->id );
-					$item->preview_link = get_preview_post_link( $item->id );
+				if ($item->id) {
+					$item->post_link    = get_edit_post_link($item->id, 'ui');
+					$item->view_link    = get_permalink($item->id);
+					$item->preview_link = get_preview_post_link($item->id);
 				}
 
-				if ( $item->parent_post_id ) {
-					$item->parent_post_link = get_edit_post_link( $item->parent_post_id, 'ui' );
+				if ($item->parent_post_id) {
+					$item->parent_post_link = get_edit_post_link($item->parent_post_id, 'ui');
 				}
 
 				return $item;

src/modules/dashboard/includes/Term_Entity_Match/Sort.php

Spacing +5 added lines, -5 removed lines

@@ -17,14 +17,14 @@
 	 * @return string|null
 	 */
 	public function get_field_name() {
-		$tmp_sort_field_name = substr( $this->sort, 1 );
-		if ( 'id' === $tmp_sort_field_name ) {
+		$tmp_sort_field_name = substr($this->sort, 1);
+		if ('id' === $tmp_sort_field_name) {
 			return 't.term_id';
-		} elseif ( 'ingredient_term' === $tmp_sort_field_name ) {
+		} elseif ('ingredient_term' === $tmp_sort_field_name) {
 			return 't.name';
-		} elseif ( 'matched_ingredient' === $tmp_sort_field_name ) {
+		} elseif ('matched_ingredient' === $tmp_sort_field_name) {
 			return 't.match_name';
-		} elseif ( 'occurrences' === $tmp_sort_field_name ) {
+		} elseif ('occurrences' === $tmp_sort_field_name) {
 			// @todo This block will be filled in the future when occurrences handling is implemented
 			return null;
 		}

src/modules/dashboard/includes/Synchronization/Rest_Controller.php

Spacing +16 added lines, -16 removed lines

@@ -15,12 +15,12 @@ 
 	/**
 	 * @param Synchronization_Service $synchronization_service
 	 */
-	public function __construct( $synchronization_service ) {
+	public function __construct($synchronization_service) {
 		$this->synchronization_service = $synchronization_service;
 	}
 
 	public function register_hooks() {
-		add_action( 'rest_api_init', array( $this, 'rest_api_init' ) );
+		add_action('rest_api_init', array($this, 'rest_api_init'));
 	}
 
 	public function rest_api_init() {
@@ -29,7 +29,7 @@ 
 			'/synchronizations',
 			array(
 				'methods'             => 'POST',
-				'callback'            => array( $this, 'create_sync' ),
+				'callback'            => array($this, 'create_sync'),
 				'permission_callback' => '__return_true',
 			)
 		);
@@ -39,11 +39,11 @@ 
 			'/synchronizations',
 			array(
 				'methods'             => 'GET',
-				'callback'            => array( $this, 'list_syncs' ),
+				'callback'            => array($this, 'list_syncs'),
 				'permission_callback' => '__return_true',
 				'args'                => array(
 					'is_running' => array(
-						'description' => esc_html__( 'Filter the is_running', 'wordlift' ),
+						'description' => esc_html__('Filter the is_running', 'wordlift'),
 						'type'        => 'boolean',
 					),
 				),
@@ -55,7 +55,7 @@ 
 			'/synchronizations',
 			array(
 				'methods'             => 'DELETE',
-				'callback'            => array( $this, 'delete_syncs' ),
+				'callback'            => array($this, 'delete_syncs'),
 				'permission_callback' => '__return_true',
 			)
 		);
@@ -63,27 +63,27 @@ 
 
 	public function create_sync() {
 		try {
-			return rest_ensure_response( $this->synchronization_service->create() );
-		} catch ( \Exception $e ) {
+			return rest_ensure_response($this->synchronization_service->create());
+		} catch (\Exception $e) {
 			return new \WP_Error(
 				'wl_error_synchronization_running',
-				esc_html__( 'Another synchronization is already running.', 'wordlift' ),
-				array( 'status' => 409 )
+				esc_html__('Another synchronization is already running.', 'wordlift'),
+				array('status' => 409)
 			);
 		}
 	}
 
-	public function list_syncs( WP_REST_Request $request ) {
+	public function list_syncs(WP_REST_Request $request) {
 		$last_synchronization = $this->synchronization_service->load();
-		$is_running_all       = ! $request->has_param( 'is_running' );
-		if ( is_a( $last_synchronization, 'Wordlift\Modules\Dashboard\Synchronization\Synchronization' )
-			&& ( $is_running_all || $request->get_param( 'is_running' ) === $last_synchronization->is_running() ) ) {
-			$data = array( $last_synchronization );
+		$is_running_all       = ! $request->has_param('is_running');
+		if (is_a($last_synchronization, 'Wordlift\Modules\Dashboard\Synchronization\Synchronization')
+			&& ($is_running_all || $request->get_param('is_running') === $last_synchronization->is_running())) {
+			$data = array($last_synchronization);
 		} else {
 			$data = array();
 		}
 
-		return rest_ensure_response( array( 'items' => $data ) );
+		return rest_ensure_response(array('items' => $data));
 	}
 
 	public function delete_syncs() {

src/modules/dashboard/includes/Match/Match_Sort.php

Spacing +22 added lines, -22 removed lines

@@ -11,15 +11,15 @@ 
 	 */
 	protected $sort;
 
-	public function __construct( $sort ) {
+	public function __construct($sort) {
 		$this->sort = $sort;
 	}
 	abstract public function get_field_name();
 
 	public function property_name() {
-		$tmp_sort_field_name = substr( $this->sort, 1 );
+		$tmp_sort_field_name = substr($this->sort, 1);
 		// @todo: Need to discuss this code due to unit tests error - Undefined property: stdClass::$name.
-		if ( 'id' === $tmp_sort_field_name || str_contains( $tmp_sort_field_name, 'date_modified_gmt' ) ) {
+		if ('id' === $tmp_sort_field_name || str_contains($tmp_sort_field_name, 'date_modified_gmt')) {
 			return 'id';
 		} else {
 			return 'name';
@@ -27,47 +27,47 @@ 
 	}
 
 	public function is_ascending() {
-		return strpos( $this->sort, '-' ) !== 0;
+		return strpos($this->sort, '-') !== 0;
 	}
 
-	private function get_sort_order( $direction, $sort_ascending ) {
-		switch ( array( $sort_ascending, $direction ) ) {
-			case array( true, 'ASCENDING' ):
-			case array( false, 'DESCENDING' ):
+	private function get_sort_order($direction, $sort_ascending) {
+		switch (array($sort_ascending, $direction)) {
+			case array(true, 'ASCENDING'):
+			case array(false, 'DESCENDING'):
 				return 'ASC';
-			case array( true, 'DESCENDING' ):
-			case array( false, 'ASCENDING' ):
+			case array(true, 'DESCENDING'):
+			case array(false, 'ASCENDING'):
 				return 'DESC';
 		}
 
 		return 'ASC';
 	}
 
-	public function get_orderby_clause( $direction ) {
-		$sort_order = $this->get_sort_order( $direction, $this->is_ascending() );
+	public function get_orderby_clause($direction) {
+		$sort_order = $this->get_sort_order($direction, $this->is_ascending());
 		return " ORDER BY {$this->get_field_name()} $sort_order";
 	}
 
-	public function apply( &$items ) {
+	public function apply(&$items) {
 		$sort_ascending     = $this->is_ascending();
 		$sort_property_name = $this->property_name();
 		usort(
 			$items,
-			function ( $a, $b ) use ( $sort_ascending, $sort_property_name ) {
-				if ( $a->{$sort_property_name} === $b->{$sort_property_name} ) {
+			function($a, $b) use ($sort_ascending, $sort_property_name) {
+				if ($a->{$sort_property_name} === $b->{$sort_property_name} ) {
 					return 0;
 				}
 
-				switch ( array(
+				switch (array(
 					$sort_ascending,
 					$a->{$sort_property_name} > $b->{$sort_property_name},
-				) ) {
-					case array( true, true ):
-					case array( false, false ):
+				)) {
+					case array(true, true):
+					case array(false, false):
 						return 1;
-					case array( true, false ):
-					case array( false, true ):
-						return - 1;
+					case array(true, false):
+					case array(false, true):
+						return -1;
 				}
 
 				return 0;