Silk\Post\QueryBuilder

Complexity

Total Complexity

11

Size/Duplication

Total Lines	149
Duplicated Lines	7.38 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	6
Bugs	0	Features	2

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	make()	0	4	1
A	limit()	0	6	1
A	all()	0	4	1
A	order()	0	6	1
A	whereStatus()	0	6	1
A	whereSlug()	0	6	1
A	results()	0	8	2
A	collectModels()	11	11	1
A	set()	0	6	1

<?php

namespace Silk\Post;

use WP_Query;

Use classes must be in alphabetical order.

use Illuminate\Support\Collection;
use Silk\Query\Builder as BaseBuilder;

class QueryBuilder extends BaseBuilder
{

Opening brace of a class must be on the same line as the definition

    /**

4 spaces found, expected 1 tabs

     * WP_Query instance

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @var WP_Query

4 spaces found, expected 1 tabs

WP_Query => \WP_Query

     */

4 spaces found, expected 1 tabs

    protected $query;

4 spaces found, expected 1 tabs

Expected 1 blank line before member var; 0 found

    /**

4 spaces found, expected 1 tabs

     * Post Model instance

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @var Model

4 spaces found, expected 1 tabs

Invalid class name "Model"

     */

4 spaces found, expected 1 tabs

    protected $model;

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Builder constructor.

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @param WP_Query $query

4 spaces found, expected 1 tabs

WP_Query => \WP_Query

     */

4 spaces found, expected 1 tabs

    public function __construct(WP_Query $query)

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query = $query;

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Create a new instance.

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return static

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public static function make()

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        return new static(new WP_Query);

8 spaces found, expected 2 tabs

Calling class constructors must always include parentheses

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Limit the number of returned results

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @param integer $limit  The maximum number of results to return

4 spaces found, expected 1 tabs

integer => int

     *                        use -1 for no limit

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return $this

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public function limit($limit)

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query->set('posts_per_page', (int) $limit);

8 spaces found, expected 2 tabs

No whitespace should be between cast and variable.

        return $this;

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Return an unlimited number of results.

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return $this

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public function all()

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        return $this->limit(-1);

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Set the order for the query

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @param  string $order

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return $this

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public function order($order)

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query->set('order', strtoupper($order));

8 spaces found, expected 2 tabs

        return $this;

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Query by post status

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @param  string|array $status  the post status or stati to match

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return $this

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public function whereStatus($status)

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query->set('post_status', $status);

8 spaces found, expected 2 tabs

        return $this;

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Query by slug

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @param  string $slug  the post slug to query by

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return $this

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public function whereSlug($slug)

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query->set('name', $slug);

8 spaces found, expected 2 tabs

        return $this;

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Get the results as a collection

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return Collection

4 spaces found, expected 1 tabs

Collection => \Illuminate\Support\Collection

     */

4 spaces found, expected 1 tabs

    public function results()

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        if ($this->model) {

8 spaces found, expected 2 tabs

            return $this->collectModels();

12 spaces found, expected 3 tabs

        }

8 spaces found, expected 2 tabs

        return Collection::make($this->query->get_posts());

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Get the results as a collection of post model instances

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return Collection

4 spaces found, expected 1 tabs

Collection => \Illuminate\Support\Collection

     */

4 spaces found, expected 1 tabs

    protected function collectModels()

This method seems to be duplicated in your project.

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query->set('post_type', $this->model->post_type);

8 spaces found, expected 2 tabs

        $this->query->set('fields', ''); // as WP_Post objects

8 spaces found, expected 2 tabs

        $modelClass = get_class($this->model);

8 spaces found, expected 2 tabs

        return Collection::make($this->query->get_posts())

8 spaces found, expected 2 tabs

            ->map(function ($post) use ($modelClass) {

12 spaces found, expected 3 tabs

                return new $modelClass($post);

16 spaces found, expected 4 tabs

            });

12 spaces found, expected 3 tabs

    }

4 spaces found, expected 1 tabs

    /**

4 spaces found, expected 1 tabs

     * Set a query variable on the query

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @param string $var   Query variable key

4 spaces found, expected 1 tabs

     * @param mixed  $value Query value for key

4 spaces found, expected 1 tabs

     *

4 spaces found, expected 1 tabs

     * @return $this

4 spaces found, expected 1 tabs

     */

4 spaces found, expected 1 tabs

    public function set($var, $value)

4 spaces found, expected 1 tabs

    {

4 spaces found, expected 1 tabs

Opening brace of a function must be on the same line as the definition

        $this->query->set($var, $value);

8 spaces found, expected 2 tabs

        return $this;

8 spaces found, expected 2 tabs

    }

4 spaces found, expected 1 tabs

}

Closing brace of a class must have a new line between itself and the last content.