insideout10 / wordlift-plugin

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

Indentation +171 added lines, -171 removed lines

@@ -5,51 +5,51 @@ 
 use WP_REST_Request;
 
 class Post_Query {
-	/**
-	 * @var WP_REST_Request
-	 */
-	private $request;
-	/**
-	 * @var mixed
-	 */
-	private $position;
-	/**
-	 * @var mixed
-	 */
-	private $element;
-	/**
-	 * @var mixed
-	 */
-	private $direction;
-
-	private $sort;
-
-	private $sortby;
-	/**
-	 * @var mixed
-	 */
-	private $limit;
-
-	/** @var Cursor_Sort $cursor_sort */
-	private $cursor_sort;
-
-	/**
-	 * @param WP_REST_Request $request
-	 * @param Cursor          $cursor
-	 */
-	public function __construct( $request, $cursor, $cursor_sort, $limit ) {
-		global $wpdb;
-
-		$this->request     = $request;
-		$this->position    = $cursor->get_position();
-		$this->element     = $cursor->get_element();
-		$this->direction   = $cursor->get_direction();
-		$this->limit       = $limit;
-		$this->cursor_sort = $cursor_sort;
-
-		$this->set_sort();
-
-		$this->sql = "
+    /**
+     * @var WP_REST_Request
+     */
+    private $request;
+    /**
+     * @var mixed
+     */
+    private $position;
+    /**
+     * @var mixed
+     */
+    private $element;
+    /**
+     * @var mixed
+     */
+    private $direction;
+
+    private $sort;
+
+    private $sortby;
+    /**
+     * @var mixed
+     */
+    private $limit;
+
+    /** @var Cursor_Sort $cursor_sort */
+    private $cursor_sort;
+
+    /**
+     * @param WP_REST_Request $request
+     * @param Cursor          $cursor
+     */
+    public function __construct( $request, $cursor, $cursor_sort, $limit ) {
+        global $wpdb;
+
+        $this->request     = $request;
+        $this->position    = $cursor->get_position();
+        $this->element     = $cursor->get_element();
+        $this->direction   = $cursor->get_direction();
+        $this->limit       = $limit;
+        $this->cursor_sort = $cursor_sort;
+
+        $this->set_sort();
+
+        $this->sql = "
 			SELECT p.ID as id,
 				p.post_title,
 				p.post_status,
@@ -62,131 +62,131 @@ 
 			WHERE 1=1 
 		";
 
-		$this->cursor();
-		$this->has_match();
-		$this->post_types();
-		$this->post_status();
-		$this->sort();
-		$this->limit();
-
-		// p.post_modified_gmt $condition %s
-		// AND p.post_types IN ( '" . implode( "','", $post_types ) . "' )
-		// AND p.post_status = %s
-		// ORDER BY p.post_modified_gmt $sort
-		// LIMIT %d
-	}
-
-	public function get_results() {
-		global $wpdb;
-
-		// The `sql` is prepared in each delegated function in this class.
-		// phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$items = $wpdb->get_results( $this->sql );
-
-		$sort = ( $this->sort === 'ASC' ? SORT_ASC : SORT_DESC );
-		array_multisort( array_column( $items, $this->cursor_sort->get_sort_property() ), $sort, $items );
-		$items = array_map( array( $this, 'map_item' ), $items );
-
-		return $items;
-	}
-
-	public function map_item( $item ) {
-		$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 );
-
-		return $item;
-	}
-
-	private function post_types() {
-		$post_types = $this->request->has_param( 'post_types' )
-			? (array) $this->request->get_param( 'post_types' )
-			: array( 'post', 'page' );
-		$value      = array_map( 'esc_sql', $post_types );
-		$this->sql .= " AND p.post_type IN ( '" . implode( "', '", $value ) . "' )";
-	}
-
-	private function limit() {
-		$value      = is_numeric( $this->limit ) ? $this->limit : 10;
-		$this->sql .= ' LIMIT ' . esc_sql( $value );
-	}
-
-	private function has_match() {
-		if ( ! $this->request->has_param( 'has_match' ) ) {
-			return;
-		}
-
-		$value = (bool) $this->request->get_param( 'has_match' );
-
-		if ( $value ) {
-			$this->sql .= ' AND e.about_jsonld IS NOT NULL';
-		} else {
-			$this->sql .= ' AND e.about_jsonld IS NULL';
-		}
-	}
-
-	private function sort() {
-		switch ( $this->direction . '$' . $this->sort ) {
-			case 'ASCENDING$ASC':
-			case 'DESCENDING$DESC':
-				$sort = 'ASC';
-				break;
-			case 'ASCENDING$DESC':
-			case 'DESCENDING$ASC':
-				$sort = 'DESC';
-				break;
-		}
-
-		$this->sql .= ' ORDER BY p.' . $this->sortby . ' ' . $sort;
-	}
-
-	private function post_status() {
-		if ( ! $this->request->has_param( 'post_status' ) ) {
-			$this->sql .= " AND p.post_status IN ( 'draft', 'publish' ) ";
-
-			return;
-		}
-
-		global $wpdb;
-		$value      = $this->request->get_param( 'post_status' );
-		$this->sql .= $wpdb->prepare( ' AND p.post_status = %s', $value );
-	}
-
-	private function cursor() {
-		if ( ! isset( $this->position ) ) {
-			return;
-		}
-
-		switch ( $this->direction . '$' . $this->sort ) {
-			case 'ASCENDING$ASC':
-			case 'DESCENDING$DESC':
-				$condition = '>';
-				break;
-			case 'ASCENDING$DESC':
-			case 'DESCENDING$ASC':
-				$condition = '<';
-				break;
-		}
-
-		$condition .= ( $this->element === 'INCLUDED' ? '=' : '' );
-		global $wpdb;
-		// We control the vars in this method.
-		// phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$this->sql .= $wpdb->prepare( ' AND p.' . esc_sql( $this->sortby ) . ' ' . $condition . ' %s', $this->position );
-	}
-
-	private function set_sort() {
-		$sortby_to_col = array(
-			'date_modified_gmt' => 'post_modified_gmt',
-		);
-
-		$value = $this->request->has_param( 'sort' )
-			? $this->request->get_param( 'sort' )
-			: '-date_modified_gmt';
-
-		$sortby       = substr( $value, 1 );
-		$this->sortby = isset( $sortby_to_col[ $sortby ] ) ? $sortby_to_col[ $sortby ] : $sortby;
-		$this->sort   = substr( $value, 0, 1 ) === '+' ? 'ASC' : 'DESC';
-	}
+        $this->cursor();
+        $this->has_match();
+        $this->post_types();
+        $this->post_status();
+        $this->sort();
+        $this->limit();
+
+        // p.post_modified_gmt $condition %s
+        // AND p.post_types IN ( '" . implode( "','", $post_types ) . "' )
+        // AND p.post_status = %s
+        // ORDER BY p.post_modified_gmt $sort
+        // LIMIT %d
+    }
+
+    public function get_results() {
+        global $wpdb;
+
+        // The `sql` is prepared in each delegated function in this class.
+        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+        $items = $wpdb->get_results( $this->sql );
+
+        $sort = ( $this->sort === 'ASC' ? SORT_ASC : SORT_DESC );
+        array_multisort( array_column( $items, $this->cursor_sort->get_sort_property() ), $sort, $items );
+        $items = array_map( array( $this, 'map_item' ), $items );
+
+        return $items;
+    }
+
+    public function map_item( $item ) {
+        $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 );
+
+        return $item;
+    }
+
+    private function post_types() {
+        $post_types = $this->request->has_param( 'post_types' )
+            ? (array) $this->request->get_param( 'post_types' )
+            : array( 'post', 'page' );
+        $value      = array_map( 'esc_sql', $post_types );
+        $this->sql .= " AND p.post_type IN ( '" . implode( "', '", $value ) . "' )";
+    }
+
+    private function limit() {
+        $value      = is_numeric( $this->limit ) ? $this->limit : 10;
+        $this->sql .= ' LIMIT ' . esc_sql( $value );
+    }
+
+    private function has_match() {
+        if ( ! $this->request->has_param( 'has_match' ) ) {
+            return;
+        }
+
+        $value = (bool) $this->request->get_param( 'has_match' );
+
+        if ( $value ) {
+            $this->sql .= ' AND e.about_jsonld IS NOT NULL';
+        } else {
+            $this->sql .= ' AND e.about_jsonld IS NULL';
+        }
+    }
+
+    private function sort() {
+        switch ( $this->direction . '$' . $this->sort ) {
+            case 'ASCENDING$ASC':
+            case 'DESCENDING$DESC':
+                $sort = 'ASC';
+                break;
+            case 'ASCENDING$DESC':
+            case 'DESCENDING$ASC':
+                $sort = 'DESC';
+                break;
+        }
+
+        $this->sql .= ' ORDER BY p.' . $this->sortby . ' ' . $sort;
+    }
+
+    private function post_status() {
+        if ( ! $this->request->has_param( 'post_status' ) ) {
+            $this->sql .= " AND p.post_status IN ( 'draft', 'publish' ) ";
+
+            return;
+        }
+
+        global $wpdb;
+        $value      = $this->request->get_param( 'post_status' );
+        $this->sql .= $wpdb->prepare( ' AND p.post_status = %s', $value );
+    }
+
+    private function cursor() {
+        if ( ! isset( $this->position ) ) {
+            return;
+        }
+
+        switch ( $this->direction . '$' . $this->sort ) {
+            case 'ASCENDING$ASC':
+            case 'DESCENDING$DESC':
+                $condition = '>';
+                break;
+            case 'ASCENDING$DESC':
+            case 'DESCENDING$ASC':
+                $condition = '<';
+                break;
+        }
+
+        $condition .= ( $this->element === 'INCLUDED' ? '=' : '' );
+        global $wpdb;
+        // We control the vars in this method.
+        // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+        $this->sql .= $wpdb->prepare( ' AND p.' . esc_sql( $this->sortby ) . ' ' . $condition . ' %s', $this->position );
+    }
+
+    private function set_sort() {
+        $sortby_to_col = array(
+            'date_modified_gmt' => 'post_modified_gmt',
+        );
+
+        $value = $this->request->has_param( 'sort' )
+            ? $this->request->get_param( 'sort' )
+            : '-date_modified_gmt';
+
+        $sortby       = substr( $value, 1 );
+        $this->sortby = isset( $sortby_to_col[ $sortby ] ) ? $sortby_to_col[ $sortby ] : $sortby;
+        $this->sort   = substr( $value, 0, 1 ) === '+' ? 'ASC' : 'DESC';
+    }
 
 }

Spacing +33 added lines, -33 removed lines

@@ -37,7 +37,7 @@ 
 	 * @param WP_REST_Request $request
 	 * @param Cursor          $cursor
 	 */
-	public function __construct( $request, $cursor, $cursor_sort, $limit ) {
+	public function __construct($request, $cursor, $cursor_sort, $limit) {
 		global $wpdb;
 
 		$this->request     = $request;
@@ -81,44 +81,44 @@ 
 
 		// The `sql` is prepared in each delegated function in this class.
 		// phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$items = $wpdb->get_results( $this->sql );
+		$items = $wpdb->get_results($this->sql);
 
-		$sort = ( $this->sort === 'ASC' ? SORT_ASC : SORT_DESC );
-		array_multisort( array_column( $items, $this->cursor_sort->get_sort_property() ), $sort, $items );
-		$items = array_map( array( $this, 'map_item' ), $items );
+		$sort = ($this->sort === 'ASC' ? SORT_ASC : SORT_DESC);
+		array_multisort(array_column($items, $this->cursor_sort->get_sort_property()), $sort, $items);
+		$items = array_map(array($this, 'map_item'), $items);
 
 		return $items;
 	}
 
-	public function map_item( $item ) {
-		$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 );
+	public function map_item($item) {
+		$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);
 
 		return $item;
 	}
 
 	private function post_types() {
-		$post_types = $this->request->has_param( 'post_types' )
-			? (array) $this->request->get_param( 'post_types' )
-			: array( 'post', 'page' );
-		$value      = array_map( 'esc_sql', $post_types );
-		$this->sql .= " AND p.post_type IN ( '" . implode( "', '", $value ) . "' )";
+		$post_types = $this->request->has_param('post_types')
+			? (array) $this->request->get_param('post_types')
+			: array('post', 'page');
+		$value      = array_map('esc_sql', $post_types);
+		$this->sql .= " AND p.post_type IN ( '".implode("', '", $value)."' )";
 	}
 
 	private function limit() {
-		$value      = is_numeric( $this->limit ) ? $this->limit : 10;
-		$this->sql .= ' LIMIT ' . esc_sql( $value );
+		$value      = is_numeric($this->limit) ? $this->limit : 10;
+		$this->sql .= ' LIMIT '.esc_sql($value);
 	}
 
 	private function has_match() {
-		if ( ! $this->request->has_param( 'has_match' ) ) {
+		if ( ! $this->request->has_param('has_match')) {
 			return;
 		}
 
-		$value = (bool) $this->request->get_param( 'has_match' );
+		$value = (bool) $this->request->get_param('has_match');
 
-		if ( $value ) {
+		if ($value) {
 			$this->sql .= ' AND e.about_jsonld IS NOT NULL';
 		} else {
 			$this->sql .= ' AND e.about_jsonld IS NULL';
@@ -126,7 +126,7 @@ 
 	}
 
 	private function sort() {
-		switch ( $this->direction . '$' . $this->sort ) {
+		switch ($this->direction.'$'.$this->sort) {
 			case 'ASCENDING$ASC':
 			case 'DESCENDING$DESC':
 				$sort = 'ASC';
@@ -137,27 +137,27 @@ 
 				break;
 		}
 
-		$this->sql .= ' ORDER BY p.' . $this->sortby . ' ' . $sort;
+		$this->sql .= ' ORDER BY p.'.$this->sortby.' '.$sort;
 	}
 
 	private function post_status() {
-		if ( ! $this->request->has_param( 'post_status' ) ) {
+		if ( ! $this->request->has_param('post_status')) {
 			$this->sql .= " AND p.post_status IN ( 'draft', 'publish' ) ";
 
 			return;
 		}
 
 		global $wpdb;
-		$value      = $this->request->get_param( 'post_status' );
-		$this->sql .= $wpdb->prepare( ' AND p.post_status = %s', $value );
+		$value      = $this->request->get_param('post_status');
+		$this->sql .= $wpdb->prepare(' AND p.post_status = %s', $value);
 	}
 
 	private function cursor() {
-		if ( ! isset( $this->position ) ) {
+		if ( ! isset($this->position)) {
 			return;
 		}
 
-		switch ( $this->direction . '$' . $this->sort ) {
+		switch ($this->direction.'$'.$this->sort) {
 			case 'ASCENDING$ASC':
 			case 'DESCENDING$DESC':
 				$condition = '>';
@@ -168,11 +168,11 @@ 
 				break;
 		}
 
-		$condition .= ( $this->element === 'INCLUDED' ? '=' : '' );
+		$condition .= ($this->element === 'INCLUDED' ? '=' : '');
 		global $wpdb;
 		// We control the vars in this method.
 		// phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-		$this->sql .= $wpdb->prepare( ' AND p.' . esc_sql( $this->sortby ) . ' ' . $condition . ' %s', $this->position );
+		$this->sql .= $wpdb->prepare(' AND p.'.esc_sql($this->sortby).' '.$condition.' %s', $this->position);
 	}
 
 	private function set_sort() {
@@ -180,13 +180,13 @@ 
 			'date_modified_gmt' => 'post_modified_gmt',
 		);
 
-		$value = $this->request->has_param( 'sort' )
-			? $this->request->get_param( 'sort' )
+		$value = $this->request->has_param('sort')
+			? $this->request->get_param('sort')
 			: '-date_modified_gmt';
 
-		$sortby       = substr( $value, 1 );
-		$this->sortby = isset( $sortby_to_col[ $sortby ] ) ? $sortby_to_col[ $sortby ] : $sortby;
-		$this->sort   = substr( $value, 0, 1 ) === '+' ? 'ASC' : 'DESC';
+		$sortby       = substr($value, 1);
+		$this->sortby = isset($sortby_to_col[$sortby]) ? $sortby_to_col[$sortby] : $sortby;
+		$this->sort   = substr($value, 0, 1) === '+' ? 'ASC' : 'DESC';
 	}
 
 }

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

Indentation +223 added lines, -223 removed lines

@@ -17,251 +17,251 @@
  */
 class Post_Entity_Match_Rest_Controller extends \WP_REST_Controller {
 
-	/**
-	 * @var Post_Entity_Match_Service
-	 */
-	private $match_service;
+    /**
+     * @var Post_Entity_Match_Service
+     */
+    private $match_service;
 
-	/**
-	 * Construct
-	 *
-	 * @param Post_Entity_Match_Service $match_service
-	 */
-	public function __construct( $match_service ) {
-		$this->match_service = $match_service;
-	}
+    /**
+     * Construct
+     *
+     * @param Post_Entity_Match_Service $match_service
+     */
+    public function __construct( $match_service ) {
+        $this->match_service = $match_service;
+    }
 
-	/**
-	 * Register hooks.
-	 */
-	public function register_hooks() {
-		add_action( 'rest_api_init', array( $this, 'register_routes' ) );
-	}
+    /**
+     * Register hooks.
+     */
+    public function register_hooks() {
+        add_action( 'rest_api_init', array( $this, 'register_routes' ) );
+    }
 
-	/**
-	 * Register the routes for the objects of the controller.
-	 */
-	public function register_routes() {
+    /**
+     * Register the routes for the objects of the controller.
+     */
+    public function register_routes() {
 
-		// Get term matches by taxonomy name
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/post-matches',
-			array(
-				'methods'             => 'GET',
-				'callback'            => array( $this, 'get_post_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' ),
-					),
-					'limit'       => array(
-						'type'              => 'integer',
-						'validate_callback' => 'rest_validate_request_arg',
-						'default'           => 10,
-						'minimum'           => 1,
-						'maximum'           => 100,
-						'sanitize_callback' => 'absint',
-					),
-					'post_types'  => array(
-						'type'              => 'array',
-						'validate_callback' => 'rest_validate_request_arg',
-					),
-					'has_match'   => array(
-						'type'              => 'boolean',
-						'required'          => false,
-						'validate_callback' => 'rest_validate_request_arg',
-					),
-					'post_status' => array(
-						'type'              => 'string',
-						'required'          => false,
-						'enum'              => array( 'publish', 'draft', 'all' ),
-						'validate_callback' => 'rest_validate_request_arg',
-					),
-					'sort'        => array(
-						'type'              => 'string',
-						'required'          => false,
-						'enum'              => array( '+date_modified_gmt', '-date_modified_gmt' ),
-						'validate_callback' => 'rest_validate_request_arg',
-						'default'           => '+date_modified_gmt',
-					),
-				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
+        // Get term matches by taxonomy name
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/post-matches',
+            array(
+                'methods'             => 'GET',
+                'callback'            => array( $this, 'get_post_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' ),
+                    ),
+                    'limit'       => array(
+                        'type'              => 'integer',
+                        'validate_callback' => 'rest_validate_request_arg',
+                        'default'           => 10,
+                        'minimum'           => 1,
+                        'maximum'           => 100,
+                        'sanitize_callback' => 'absint',
+                    ),
+                    'post_types'  => array(
+                        'type'              => 'array',
+                        'validate_callback' => 'rest_validate_request_arg',
+                    ),
+                    'has_match'   => array(
+                        'type'              => 'boolean',
+                        'required'          => false,
+                        'validate_callback' => 'rest_validate_request_arg',
+                    ),
+                    'post_status' => array(
+                        'type'              => 'string',
+                        'required'          => false,
+                        'enum'              => array( 'publish', 'draft', 'all' ),
+                        'validate_callback' => 'rest_validate_request_arg',
+                    ),
+                    'sort'        => array(
+                        'type'              => 'string',
+                        'required'          => false,
+                        'enum'              => array( '+date_modified_gmt', '-date_modified_gmt' ),
+                        'validate_callback' => 'rest_validate_request_arg',
+                        'default'           => '+date_modified_gmt',
+                    ),
+                ),
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
 
-		// Create a new match for a post
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/post-matches/(?P<post_id>\d+)/matches',
-			array(
-				'methods'             => 'POST',
-				'callback'            => array( $this, 'create_post_match' ),
-				'args'                => array(
-					'post_id' => array(
-						'required'          => true,
-						'validate_callback' => 'rest_validate_request_arg',
-						'type'              => 'integer',
-					),
-				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
+        // Create a new match for a post
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/post-matches/(?P<post_id>\d+)/matches',
+            array(
+                'methods'             => 'POST',
+                'callback'            => array( $this, 'create_post_match' ),
+                'args'                => array(
+                    'post_id' => array(
+                        'required'          => true,
+                        'validate_callback' => 'rest_validate_request_arg',
+                        'type'              => 'integer',
+                    ),
+                ),
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
 
-		// Update an existing post match
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/post-matches/(?P<post_id>\d+)/matches/(?P<match_id>\d+)',
-			array(
-				'methods'             => 'PUT',
-				'callback'            => array( $this, 'update_post_match' ),
-				'args'                => array(
-					'post_id'  => array(
-						'required'          => true,
-						'validate_callback' => 'rest_validate_request_arg',
-						'type'              => 'integer',
-					),
-					'match_id' => array(
-						'required'          => true,
-						'validate_callback' => 'rest_validate_request_arg',
-						'type'              => 'integer',
-					),
-				),
-				'permission_callback' => function () {
+        // Update an existing post match
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/post-matches/(?P<post_id>\d+)/matches/(?P<match_id>\d+)',
+            array(
+                'methods'             => 'PUT',
+                'callback'            => array( $this, 'update_post_match' ),
+                'args'                => array(
+                    'post_id'  => array(
+                        'required'          => true,
+                        'validate_callback' => 'rest_validate_request_arg',
+                        'type'              => 'integer',
+                    ),
+                    'match_id' => array(
+                        'required'          => true,
+                        'validate_callback' => 'rest_validate_request_arg',
+                        'type'              => 'integer',
+                    ),
+                ),
+                'permission_callback' => function () {
 
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-	}
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+    }
 
-	/**
-	 * @param \WP_REST_Request $request
-	 *
-	 * @return array|object|\stdClass[]|null
-	 * @throws Exception when some of the parameters are not in the accepted format.
-	 */
-	public function get_post_matches( $request ) {
-		$limit       = $request->has_param( 'limit' ) ? $request->get_param( 'limit' ) : 10;
-		$cursor      = $request->has_param( 'cursor' )
-			? Cursor::from_base64_string( $request->get_param( 'cursor' ) )
-			: Cursor::empty_cursor();
-		$cursor_sort = new Cursor_Sort(
-			$request->has_param( 'sort' ) ? $request->get_param( 'sort' ) : '-date_modified_gmt'
-		);
+    /**
+     * @param \WP_REST_Request $request
+     *
+     * @return array|object|\stdClass[]|null
+     * @throws Exception when some of the parameters are not in the accepted format.
+     */
+    public function get_post_matches( $request ) {
+        $limit       = $request->has_param( 'limit' ) ? $request->get_param( 'limit' ) : 10;
+        $cursor      = $request->has_param( 'cursor' )
+            ? Cursor::from_base64_string( $request->get_param( 'cursor' ) )
+            : Cursor::empty_cursor();
+        $cursor_sort = new Cursor_Sort(
+            $request->has_param( 'sort' ) ? $request->get_param( 'sort' ) : '-date_modified_gmt'
+        );
 
-		// If we're looking for recipes, we need the Recipe_Query.
-		if ( in_array( 'wprm_recipe', (array) $request->get_param( 'post_types' ), true ) ) {
-			$query = new Recipe_Query( $request, $cursor, $cursor_sort, $limit + 1 );
-		} else {
-			$query = new Post_Query( $request, $cursor, $cursor_sort, $limit + 1 );
-		}
+        // If we're looking for recipes, we need the Recipe_Query.
+        if ( in_array( 'wprm_recipe', (array) $request->get_param( 'post_types' ), true ) ) {
+            $query = new Recipe_Query( $request, $cursor, $cursor_sort, $limit + 1 );
+        } else {
+            $query = new Post_Query( $request, $cursor, $cursor_sort, $limit + 1 );
+        }
 
-		$results       = $query->get_results();
-		$items         = $cursor->get_direction() === 'ASCENDING'
-			? array_slice( $results, 0, min( count( $results ), $limit ) )
-			: array_slice( $results, count( $results ) > $limit ? 1 : 0 );
-		$position      = current( $items )->{$cursor_sort->get_sort_property()};
-		$next_position = end( $items )->{$cursor_sort->get_sort_property()};
+        $results       = $query->get_results();
+        $items         = $cursor->get_direction() === 'ASCENDING'
+            ? array_slice( $results, 0, min( count( $results ), $limit ) )
+            : array_slice( $results, count( $results ) > $limit ? 1 : 0 );
+        $position      = current( $items )->{$cursor_sort->get_sort_property()};
+        $next_position = end( $items )->{$cursor_sort->get_sort_property()};
 
-		$self  = $cursor;
-		$first = new Cursor( null, 'INCLUDED', 'ASCENDING' );
-		$last  = new Cursor( null, 'INCLUDED', 'DESCENDING' );
+        $self  = $cursor;
+        $first = new Cursor( null, 'INCLUDED', 'ASCENDING' );
+        $last  = new Cursor( null, 'INCLUDED', 'DESCENDING' );
 
-		return new Page(
-			$items,
-			$self->to_base64_string(),
-			$first->to_base64_string(),
-			$this->get_prev_cursor_as_base64_string( $cursor, $results, $limit, $position ),
-			$this->get_next_cursor_as_base64_string( $cursor, $results, $limit, $next_position ),
-			$last->to_base64_string()
-		);
-	}
+        return new Page(
+            $items,
+            $self->to_base64_string(),
+            $first->to_base64_string(),
+            $this->get_prev_cursor_as_base64_string( $cursor, $results, $limit, $position ),
+            $this->get_next_cursor_as_base64_string( $cursor, $results, $limit, $next_position ),
+            $last->to_base64_string()
+        );
+    }
 
-	/**
-	 * @param Cursor $cursor
-	 *
-	 * @return string|null
-	 * @throws Exception when one of the called functions throws an exception.
-	 */
-	private function get_prev_cursor_as_base64_string( $cursor, $results, $limit, $position ) {
-		if ( ( $cursor->get_direction() === 'ASCENDING' && $cursor->get_position() !== null ) || ( $cursor->get_direction() === 'DESCENDING' && count( $results ) > $limit ) ) {
-			$cursor = new Cursor( $position, 'EXCLUDED', 'DESCENDING' );
+    /**
+     * @param Cursor $cursor
+     *
+     * @return string|null
+     * @throws Exception when one of the called functions throws an exception.
+     */
+    private function get_prev_cursor_as_base64_string( $cursor, $results, $limit, $position ) {
+        if ( ( $cursor->get_direction() === 'ASCENDING' && $cursor->get_position() !== null ) || ( $cursor->get_direction() === 'DESCENDING' && count( $results ) > $limit ) ) {
+            $cursor = new Cursor( $position, 'EXCLUDED', 'DESCENDING' );
 
-			return $cursor->to_base64_string();
-		}
+            return $cursor->to_base64_string();
+        }
 
-		return null;
-	}
+        return null;
+    }
 
-	/**
-	 * @param Cursor $cursor
-	 *
-	 * @return string|null
-	 * @throws Exception when one of the called functions throws an exception.
-	 */
-	private function get_next_cursor_as_base64_string( $cursor, $results, $limit, $position ) {
-		if ( ( $cursor->get_direction() === 'DESCENDING' && $cursor->get_position() !== null ) || ( $cursor->get_direction() === 'ASCENDING' && count( $results ) > $limit ) ) {
-			$cursor = new Cursor( $position, 'EXCLUDED', 'ASCENDING' );
+    /**
+     * @param Cursor $cursor
+     *
+     * @return string|null
+     * @throws Exception when one of the called functions throws an exception.
+     */
+    private function get_next_cursor_as_base64_string( $cursor, $results, $limit, $position ) {
+        if ( ( $cursor->get_direction() === 'DESCENDING' && $cursor->get_position() !== null ) || ( $cursor->get_direction() === 'ASCENDING' && count( $results ) > $limit ) ) {
+            $cursor = new Cursor( $position, 'EXCLUDED', 'ASCENDING' );
 
-			return $cursor->to_base64_string();
-		}
+            return $cursor->to_base64_string();
+        }
 
-		return null;
-	}
+        return null;
+    }
 
-	/**
-	 * Create a new match for a post.
-	 *
-	 * @param  $request \WP_REST_Request
-	 *
-	 * @throws Exception If there was a problem creating the match.
-	 */
-	public function create_post_match( $request ) {
-		$post_id = $request->get_param( 'post_id' );
+    /**
+     * Create a new match for a post.
+     *
+     * @param  $request \WP_REST_Request
+     *
+     * @throws Exception If there was a problem creating the match.
+     */
+    public function create_post_match( $request ) {
+        $post_id = $request->get_param( 'post_id' );
 
-		// If we dont have a entry on the match table, then add one.
-		$content_id = Wordpress_Content_Id::create_post( $post_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 );
-		}
+        // If we dont have a entry on the match table, then add one.
+        $content_id = Wordpress_Content_Id::create_post( $post_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 = Wordpress_Dataset_Content_Service_Hooks::get_id_or_create(
-			$post_id,
-			Object_Type_Enum::POST
-		);
+        $match_id = Wordpress_Dataset_Content_Service_Hooks::get_id_or_create(
+            $post_id,
+            Object_Type_Enum::POST
+        );
 
-		return $this->match_service->set_jsonld(
-			$post_id,
-			Object_Type_Enum::POST,
-			$match_id,
-			$request->get_json_params()
-		);
-	}
+        return $this->match_service->set_jsonld(
+            $post_id,
+            Object_Type_Enum::POST,
+            $match_id,
+            $request->get_json_params()
+        );
+    }
 
-	/**
-	 * Update post match.
-	 *
-	 * @param  $request \WP_REST_Request
-	 *
-	 * @return Match_Entry
-	 *
-	 * @throws Exception If there was a problem updating the match.
-	 */
-	public function update_post_match( $request ) {
+    /**
+     * Update post match.
+     *
+     * @param  $request \WP_REST_Request
+     *
+     * @return Match_Entry
+     *
+     * @throws Exception If there was a problem updating the match.
+     */
+    public function update_post_match( $request ) {
 
-		return $this->match_service->set_jsonld(
-			$request->get_param( 'post_id' ),
-			Object_Type_Enum::POST,
-			$request->get_param( 'match_id' ),
-			$request->get_json_params()
-		);
-	}
+        return $this->match_service->set_jsonld(
+            $request->get_param( 'post_id' ),
+            Object_Type_Enum::POST,
+            $request->get_param( 'match_id' ),
+            $request->get_json_params()
+        );
+    }
 }

Spacing +44 added lines, -44 removed lines

@@ -27,7 +27,7 @@ 
 	 *
 	 * @param Post_Entity_Match_Service $match_service
 	 */
-	public function __construct( $match_service ) {
+	public function __construct($match_service) {
 		$this->match_service = $match_service;
 	}
 
@@ -35,7 +35,7 @@ 
 	 * Register hooks.
 	 */
 	public function register_hooks() {
-		add_action( 'rest_api_init', array( $this, 'register_routes' ) );
+		add_action('rest_api_init', array($this, 'register_routes'));
 	}
 
 	/**
@@ -49,7 +49,7 @@ 
 			'/post-matches',
 			array(
 				'methods'             => 'GET',
-				'callback'            => array( $this, 'get_post_matches' ),
+				'callback'            => array($this, 'get_post_matches'),
 				'args'                => array(
 					'cursor'      => array(
 						'type'              => 'string',
@@ -77,19 +77,19 @@ 
 					'post_status' => array(
 						'type'              => 'string',
 						'required'          => false,
-						'enum'              => array( 'publish', 'draft', 'all' ),
+						'enum'              => array('publish', 'draft', 'all'),
 						'validate_callback' => 'rest_validate_request_arg',
 					),
 					'sort'        => array(
 						'type'              => 'string',
 						'required'          => false,
-						'enum'              => array( '+date_modified_gmt', '-date_modified_gmt' ),
+						'enum'              => array('+date_modified_gmt', '-date_modified_gmt'),
 						'validate_callback' => 'rest_validate_request_arg',
 						'default'           => '+date_modified_gmt',
 					),
 				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -100,7 +100,7 @@ 
 			'/post-matches/(?P<post_id>\d+)/matches',
 			array(
 				'methods'             => 'POST',
-				'callback'            => array( $this, 'create_post_match' ),
+				'callback'            => array($this, 'create_post_match'),
 				'args'                => array(
 					'post_id' => array(
 						'required'          => true,
@@ -108,8 +108,8 @@ 
 						'type'              => 'integer',
 					),
 				),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -120,7 +120,7 @@ 
 			'/post-matches/(?P<post_id>\d+)/matches/(?P<match_id>\d+)',
 			array(
 				'methods'             => 'PUT',
-				'callback'            => array( $this, 'update_post_match' ),
+				'callback'            => array($this, 'update_post_match'),
 				'args'                => array(
 					'post_id'  => array(
 						'required'          => true,
@@ -133,9 +133,9 @@ 
 						'type'              => 'integer',
 					),
 				),
-				'permission_callback' => function () {
+				'permission_callback' => function() {
 
-					return current_user_can( 'manage_options' );
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -147,39 +147,39 @@ 
 	 * @return array|object|\stdClass[]|null
 	 * @throws Exception when some of the parameters are not in the accepted format.
 	 */
-	public function get_post_matches( $request ) {
-		$limit       = $request->has_param( 'limit' ) ? $request->get_param( 'limit' ) : 10;
-		$cursor      = $request->has_param( 'cursor' )
-			? Cursor::from_base64_string( $request->get_param( 'cursor' ) )
+	public function get_post_matches($request) {
+		$limit       = $request->has_param('limit') ? $request->get_param('limit') : 10;
+		$cursor      = $request->has_param('cursor')
+			? Cursor::from_base64_string($request->get_param('cursor'))
 			: Cursor::empty_cursor();
 		$cursor_sort = new Cursor_Sort(
-			$request->has_param( 'sort' ) ? $request->get_param( 'sort' ) : '-date_modified_gmt'
+			$request->has_param('sort') ? $request->get_param('sort') : '-date_modified_gmt'
 		);
 
 		// If we're looking for recipes, we need the Recipe_Query.
-		if ( in_array( 'wprm_recipe', (array) $request->get_param( 'post_types' ), true ) ) {
-			$query = new Recipe_Query( $request, $cursor, $cursor_sort, $limit + 1 );
+		if (in_array('wprm_recipe', (array) $request->get_param('post_types'), true)) {
+			$query = new Recipe_Query($request, $cursor, $cursor_sort, $limit + 1);
 		} else {
-			$query = new Post_Query( $request, $cursor, $cursor_sort, $limit + 1 );
+			$query = new Post_Query($request, $cursor, $cursor_sort, $limit + 1);
 		}
 
 		$results       = $query->get_results();
 		$items         = $cursor->get_direction() === 'ASCENDING'
-			? array_slice( $results, 0, min( count( $results ), $limit ) )
-			: array_slice( $results, count( $results ) > $limit ? 1 : 0 );
-		$position      = current( $items )->{$cursor_sort->get_sort_property()};
-		$next_position = end( $items )->{$cursor_sort->get_sort_property()};
+			? array_slice($results, 0, min(count($results), $limit))
+			: array_slice($results, count($results) > $limit ? 1 : 0);
+		$position      = current($items)->{$cursor_sort->get_sort_property()};
+		$next_position = end($items)->{$cursor_sort->get_sort_property()};
 
 		$self  = $cursor;
-		$first = new Cursor( null, 'INCLUDED', 'ASCENDING' );
-		$last  = new Cursor( null, 'INCLUDED', 'DESCENDING' );
+		$first = new Cursor(null, 'INCLUDED', 'ASCENDING');
+		$last  = new Cursor(null, 'INCLUDED', 'DESCENDING');
 
 		return new Page(
 			$items,
 			$self->to_base64_string(),
 			$first->to_base64_string(),
-			$this->get_prev_cursor_as_base64_string( $cursor, $results, $limit, $position ),
-			$this->get_next_cursor_as_base64_string( $cursor, $results, $limit, $next_position ),
+			$this->get_prev_cursor_as_base64_string($cursor, $results, $limit, $position),
+			$this->get_next_cursor_as_base64_string($cursor, $results, $limit, $next_position),
 			$last->to_base64_string()
 		);
 	}
@@ -190,9 +190,9 @@ 
 	 * @return string|null
 	 * @throws Exception when one of the called functions throws an exception.
 	 */
-	private function get_prev_cursor_as_base64_string( $cursor, $results, $limit, $position ) {
-		if ( ( $cursor->get_direction() === 'ASCENDING' && $cursor->get_position() !== null ) || ( $cursor->get_direction() === 'DESCENDING' && count( $results ) > $limit ) ) {
-			$cursor = new Cursor( $position, 'EXCLUDED', 'DESCENDING' );
+	private function get_prev_cursor_as_base64_string($cursor, $results, $limit, $position) {
+		if (($cursor->get_direction() === 'ASCENDING' && $cursor->get_position() !== null) || ($cursor->get_direction() === 'DESCENDING' && count($results) > $limit)) {
+			$cursor = new Cursor($position, 'EXCLUDED', 'DESCENDING');
 
 			return $cursor->to_base64_string();
 		}
@@ -206,9 +206,9 @@ 
 	 * @return string|null
 	 * @throws Exception when one of the called functions throws an exception.
 	 */
-	private function get_next_cursor_as_base64_string( $cursor, $results, $limit, $position ) {
-		if ( ( $cursor->get_direction() === 'DESCENDING' && $cursor->get_position() !== null ) || ( $cursor->get_direction() === 'ASCENDING' && count( $results ) > $limit ) ) {
-			$cursor = new Cursor( $position, 'EXCLUDED', 'ASCENDING' );
+	private function get_next_cursor_as_base64_string($cursor, $results, $limit, $position) {
+		if (($cursor->get_direction() === 'DESCENDING' && $cursor->get_position() !== null) || ($cursor->get_direction() === 'ASCENDING' && count($results) > $limit)) {
+			$cursor = new Cursor($position, 'EXCLUDED', 'ASCENDING');
 
 			return $cursor->to_base64_string();
 		}
@@ -223,14 +223,14 @@ 
 	 *
 	 * @throws Exception If there was a problem creating the match.
 	 */
-	public function create_post_match( $request ) {
-		$post_id = $request->get_param( 'post_id' );
+	public function create_post_match($request) {
+		$post_id = $request->get_param('post_id');
 
 		// If we dont have a entry on the match table, then add one.
-		$content_id = Wordpress_Content_Id::create_post( $post_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_post($post_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 = Wordpress_Dataset_Content_Service_Hooks::get_id_or_create(
@@ -255,12 +255,12 @@ 
 	 *
 	 * @throws Exception If there was a problem updating the match.
 	 */
-	public function update_post_match( $request ) {
+	public function update_post_match($request) {
 
 		return $this->match_service->set_jsonld(
-			$request->get_param( 'post_id' ),
+			$request->get_param('post_id'),
 			Object_Type_Enum::POST,
-			$request->get_param( 'match_id' ),
+			$request->get_param('match_id'),
 			$request->get_json_params()
 		);
 	}

src/modules/food-kg/includes/Main_Entity/Food_Kg_Recipe_Stats.php

Indentation +26 added lines, -26 removed lines

@@ -8,33 +8,33 @@ 
 
 class Food_Kg_Recipe_Stats {
 
-	public function register_hooks() {
-		add_filter( 'wl_dashboard__stats__settings', array( $this, 'dashboard__stats__settings' ) );
-	}
+    public function register_hooks() {
+        add_filter( 'wl_dashboard__stats__settings', array( $this, 'dashboard__stats__settings' ) );
+    }
 
-	public function dashboard__stats__settings( $arr ) {
+    public function dashboard__stats__settings( $arr ) {
 
-		$data = Recipe_Query::get_data(); // $this->get_data();
+        $data = Recipe_Query::get_data(); // $this->get_data();
 
-		$arr[] = new Stats_Settings(
-			__( 'Boosted Recipes are the ones WordLift matched with KG. Some Explanation how it helps them.', 'wordlift' ),
-			__( 'Lifted Recipes', 'wordlift' ),
-			__( 'Recipes', 'wordlift' ),
-			'#00c48c',
-			'../recipes',
-			(int) $data->total,
-			(int) $data->lifted
-		);
+        $arr[] = new Stats_Settings(
+            __( 'Boosted Recipes are the ones WordLift matched with KG. Some Explanation how it helps them.', 'wordlift' ),
+            __( 'Lifted Recipes', 'wordlift' ),
+            __( 'Recipes', 'wordlift' ),
+            '#00c48c',
+            '../recipes',
+            (int) $data->total,
+            (int) $data->lifted
+        );
 
-		return $arr;
-	}
+        return $arr;
+    }
 
-	public function get_data() {
-		global $wpdb;
+    public function get_data() {
+        global $wpdb;
 
-		return $wpdb->get_row(
-			$wpdb->prepare(
-				"
+        return $wpdb->get_row(
+            $wpdb->prepare(
+                "
 				SELECT COUNT(1) AS total, COUNT(e.about_jsonld) as lifted
 				FROM {$wpdb->prefix}posts p
 				LEFT JOIN {$wpdb->prefix}wl_entities e
@@ -42,10 +42,10 @@ 
 						AND e.content_type = %d 
 				WHERE p.post_type = %s
 				",
-				Object_Type_Enum::POST,
-				'wprm_recipe'
-			)
-		);
-	}
+                Object_Type_Enum::POST,
+                'wprm_recipe'
+            )
+        );
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -9,17 +9,17 @@
 class Food_Kg_Recipe_Stats {
 
 	public function register_hooks() {
-		add_filter( 'wl_dashboard__stats__settings', array( $this, 'dashboard__stats__settings' ) );
+		add_filter('wl_dashboard__stats__settings', array($this, 'dashboard__stats__settings'));
 	}
 
-	public function dashboard__stats__settings( $arr ) {
+	public function dashboard__stats__settings($arr) {
 
 		$data = Recipe_Query::get_data(); // $this->get_data();
 
 		$arr[] = new Stats_Settings(
-			__( 'Boosted Recipes are the ones WordLift matched with KG. Some Explanation how it helps them.', 'wordlift' ),
-			__( 'Lifted Recipes', 'wordlift' ),
-			__( 'Recipes', 'wordlift' ),
+			__('Boosted Recipes are the ones WordLift matched with KG. Some Explanation how it helps them.', 'wordlift'),
+			__('Lifted Recipes', 'wordlift'),
+			__('Recipes', 'wordlift'),
 			'#00c48c',
 			'../recipes',
 			(int) $data->total,

src/modules/food-kg/includes/Jsonld.php

Indentation +44 added lines, -44 removed lines

@@ -9,59 +9,59 @@
 
 class Jsonld {
 
-	/**
-	 * @var Content_Service $content_service
-	 */
-	private $content_service;
+    /**
+     * @var Content_Service $content_service
+     */
+    private $content_service;
 
-	public function __construct( Content_Service $content_service ) {
-		$this->content_service = $content_service;
-	}
+    public function __construct( Content_Service $content_service ) {
+        $this->content_service = $content_service;
+    }
 
-	public function register_hooks() {
-		add_filter( 'wprm_recipe_metadata', array( $this, '__recipe_metadata' ), 10, 2 );
-	}
+    public function register_hooks() {
+        add_filter( 'wprm_recipe_metadata', array( $this, '__recipe_metadata' ), 10, 2 );
+    }
 
-	/**
-	 * @param array       $metadata
-	 * @param WPRM_Recipe $recipe
-	 *
-	 * @return array
-	 */
-	public function __recipe_metadata( $metadata, $recipe ) {
+    /**
+     * @param array       $metadata
+     * @param WPRM_Recipe $recipe
+     *
+     * @return array
+     */
+    public function __recipe_metadata( $metadata, $recipe ) {
 
-		$ingredients = get_the_terms( $recipe->id(), 'wprm_ingredient' );
-		if ( ! $ingredients ) {
-			return $metadata;
-		}
-		$jsonlds = array_filter( array_map( array( $this, '__term_id_to_jsonld' ), $ingredients ) );
+        $ingredients = get_the_terms( $recipe->id(), 'wprm_ingredient' );
+        if ( ! $ingredients ) {
+            return $metadata;
+        }
+        $jsonlds = array_filter( array_map( array( $this, '__term_id_to_jsonld' ), $ingredients ) );
 
-		if ( empty( $jsonlds ) ) {
-			return $metadata;
-		}
+        if ( empty( $jsonlds ) ) {
+            return $metadata;
+        }
 
-		// We're embedding the full json-ld here because WL doesn't output its own markup, so it makes no sense
-		// to hook to wl_after_json_ld.
-		$metadata['mentions'] = isset( $metadata['mentions'] ) ? $metadata['mentions'] : array();
-		$metadata['mentions'] = array_merge( $metadata['mentions'], $jsonlds );
+        // We're embedding the full json-ld here because WL doesn't output its own markup, so it makes no sense
+        // to hook to wl_after_json_ld.
+        $metadata['mentions'] = isset( $metadata['mentions'] ) ? $metadata['mentions'] : array();
+        $metadata['mentions'] = array_merge( $metadata['mentions'], $jsonlds );
 
-		return $metadata;
-	}
+        return $metadata;
+    }
 
-	/**
-	 * @param WP_Term $ingredient
-	 *
-	 * @return array|null
-	 */
-	private function __term_id_to_jsonld( $ingredient ) {
-		$content_id = Wordpress_Content_Id::create_term( $ingredient->term_id );
-		$jsonld     = $this->content_service->get_about_jsonld( $content_id );
+    /**
+     * @param WP_Term $ingredient
+     *
+     * @return array|null
+     */
+    private function __term_id_to_jsonld( $ingredient ) {
+        $content_id = Wordpress_Content_Id::create_term( $ingredient->term_id );
+        $jsonld     = $this->content_service->get_about_jsonld( $content_id );
 
-		if ( ! is_string( $jsonld ) ) {
-			return null;
-		}
+        if ( ! is_string( $jsonld ) ) {
+            return null;
+        }
 
-		return json_decode( $jsonld, true );
-	}
+        return json_decode( $jsonld, true );
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -14,12 +14,12 @@ 
 	 */
 	private $content_service;
 
-	public function __construct( Content_Service $content_service ) {
+	public function __construct(Content_Service $content_service) {
 		$this->content_service = $content_service;
 	}
 
 	public function register_hooks() {
-		add_filter( 'wprm_recipe_metadata', array( $this, '__recipe_metadata' ), 10, 2 );
+		add_filter('wprm_recipe_metadata', array($this, '__recipe_metadata'), 10, 2);
 	}
 
 	/**
@@ -28,22 +28,22 @@ 
 	 *
 	 * @return array
 	 */
-	public function __recipe_metadata( $metadata, $recipe ) {
+	public function __recipe_metadata($metadata, $recipe) {
 
-		$ingredients = get_the_terms( $recipe->id(), 'wprm_ingredient' );
-		if ( ! $ingredients ) {
+		$ingredients = get_the_terms($recipe->id(), 'wprm_ingredient');
+		if ( ! $ingredients) {
 			return $metadata;
 		}
-		$jsonlds = array_filter( array_map( array( $this, '__term_id_to_jsonld' ), $ingredients ) );
+		$jsonlds = array_filter(array_map(array($this, '__term_id_to_jsonld'), $ingredients));
 
-		if ( empty( $jsonlds ) ) {
+		if (empty($jsonlds)) {
 			return $metadata;
 		}
 
 		// We're embedding the full json-ld here because WL doesn't output its own markup, so it makes no sense
 		// to hook to wl_after_json_ld.
-		$metadata['mentions'] = isset( $metadata['mentions'] ) ? $metadata['mentions'] : array();
-		$metadata['mentions'] = array_merge( $metadata['mentions'], $jsonlds );
+		$metadata['mentions'] = isset($metadata['mentions']) ? $metadata['mentions'] : array();
+		$metadata['mentions'] = array_merge($metadata['mentions'], $jsonlds);
 
 		return $metadata;
 	}
@@ -53,15 +53,15 @@ 
 	 *
 	 * @return array|null
 	 */
-	private function __term_id_to_jsonld( $ingredient ) {
-		$content_id = Wordpress_Content_Id::create_term( $ingredient->term_id );
-		$jsonld     = $this->content_service->get_about_jsonld( $content_id );
+	private function __term_id_to_jsonld($ingredient) {
+		$content_id = Wordpress_Content_Id::create_term($ingredient->term_id);
+		$jsonld     = $this->content_service->get_about_jsonld($content_id);
 
-		if ( ! is_string( $jsonld ) ) {
+		if ( ! is_string($jsonld)) {
 			return null;
 		}
 
-		return json_decode( $jsonld, true );
+		return json_decode($jsonld, true);
 	}
 
 }

src/modules/food-kg/includes/Main_Ingredient_Jsonld.php

Indentation +35 added lines, -35 removed lines

@@ -8,40 +8,40 @@
 
 class Main_Ingredient_Jsonld {
 
-	/**
-	 * @var Content_Service $content_service
-	 */
-	private $content_service;
-
-	public function __construct( Content_Service $content_service ) {
-		$this->content_service = $content_service;
-	}
-
-	public function register_hooks() {
-		add_filter( 'wprm_recipe_metadata', array( $this, '__recipe_metadata' ), 10, 2 );
-	}
-
-	/**
-	 * @param array       $metadata
-	 * @param WPRM_Recipe $recipe
-	 *
-	 * @return array
-	 */
-	public function __recipe_metadata( $metadata, $recipe ) {
-		$jsonld = $this->content_service->get_about_jsonld(
-			Wordpress_Content_Id::create_post( $recipe->id() )
-		);
-
-		if ( empty( $jsonld ) ) {
-			return $metadata;
-		}
-
-		// We're embedding the full json-ld here because WL doesn't output its own markup, so it makes no sense
-		// to hook to wl_after_json_ld.
-		$metadata['about'] = isset( $metadata['about'] ) ? $metadata['about'] : array();
-		$metadata['about'] = array_merge( $metadata['about'], array( json_decode( $jsonld, true ) ) );
-
-		return $metadata;
-	}
+    /**
+     * @var Content_Service $content_service
+     */
+    private $content_service;
+
+    public function __construct( Content_Service $content_service ) {
+        $this->content_service = $content_service;
+    }
+
+    public function register_hooks() {
+        add_filter( 'wprm_recipe_metadata', array( $this, '__recipe_metadata' ), 10, 2 );
+    }
+
+    /**
+     * @param array       $metadata
+     * @param WPRM_Recipe $recipe
+     *
+     * @return array
+     */
+    public function __recipe_metadata( $metadata, $recipe ) {
+        $jsonld = $this->content_service->get_about_jsonld(
+            Wordpress_Content_Id::create_post( $recipe->id() )
+        );
+
+        if ( empty( $jsonld ) ) {
+            return $metadata;
+        }
+
+        // We're embedding the full json-ld here because WL doesn't output its own markup, so it makes no sense
+        // to hook to wl_after_json_ld.
+        $metadata['about'] = isset( $metadata['about'] ) ? $metadata['about'] : array();
+        $metadata['about'] = array_merge( $metadata['about'], array( json_decode( $jsonld, true ) ) );
+
+        return $metadata;
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -13,12 +13,12 @@ 
 	 */
 	private $content_service;
 
-	public function __construct( Content_Service $content_service ) {
+	public function __construct(Content_Service $content_service) {
 		$this->content_service = $content_service;
 	}
 
 	public function register_hooks() {
-		add_filter( 'wprm_recipe_metadata', array( $this, '__recipe_metadata' ), 10, 2 );
+		add_filter('wprm_recipe_metadata', array($this, '__recipe_metadata'), 10, 2);
 	}
 
 	/**
@@ -27,19 +27,19 @@ 
 	 *
 	 * @return array
 	 */
-	public function __recipe_metadata( $metadata, $recipe ) {
+	public function __recipe_metadata($metadata, $recipe) {
 		$jsonld = $this->content_service->get_about_jsonld(
-			Wordpress_Content_Id::create_post( $recipe->id() )
+			Wordpress_Content_Id::create_post($recipe->id())
 		);
 
-		if ( empty( $jsonld ) ) {
+		if (empty($jsonld)) {
 			return $metadata;
 		}
 
 		// We're embedding the full json-ld here because WL doesn't output its own markup, so it makes no sense
 		// to hook to wl_after_json_ld.
-		$metadata['about'] = isset( $metadata['about'] ) ? $metadata['about'] : array();
-		$metadata['about'] = array_merge( $metadata['about'], array( json_decode( $jsonld, true ) ) );
+		$metadata['about'] = isset($metadata['about']) ? $metadata['about'] : array();
+		$metadata['about'] = array_merge($metadata['about'], array(json_decode($jsonld, true)));
 
 		return $metadata;
 	}