LikeDislikeController::store() - Code Metrics - deltcod/LaravelTube - Measure and Improve Code Quality continuously with Scrutinizer

LikeDislikeController::store() A
last analyzed 2017-07-28 07:27 UTC

↳ Parent: LikeDislikeController

Complexity

Conditions	3
Paths	3

Size

Total Lines	21
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	21
ccs	13
cts	13
cp	1
rs	9.3142
cc	3
eloc	14
nc	3
nop	1
crap	3

<?php

namespace App\Http\Controllers;

use App\Events\LikeDislikePush;
use App\Http\Requests\LikeDislikeStoreRequest;
use App\Repositories\LikeDislikeRepository as LikeDislike;
use App\Transformers\LikeDislikeTransformer;
use Chrisbjr\ApiGuard\Http\Controllers\ApiGuardController;

/**
 * Class LikeDislikeController.
 */
class LikeDislikeController extends ApiGuardController
{
    /**
     * @var LikeDislikeTransformer
     */
    protected $likeDislikeTransformer;

    /**
     * LikeDislike repository.
     *
     * @var LikeDislike
     */
    private $likeDislike;

    /**
     * @var array
     */
    protected $apiMethods = [
        'getLikes' => [
            'keyAuthentication' => false,
        ],
        'getDislikes' => [
            'keyAuthentication' => false,
        ],
        'getLikesCount' => [
            'keyAuthentication' => false,
        ],
        'getDislikesCount' => [
            'keyAuthentication' => false,
        ],
    ];

    /**
     * LikeDislikeController constructor.
     *
     * @param LikeDislike            $likeDislike
     * @param LikeDislikeTransformer $likeDislikeTransformer
     */
    public function __construct(LikeDislike $likeDislike, LikeDislikeTransformer $likeDislikeTransformer)
    {
        parent::__construct();

        $this->likeDislikeTransformer = $likeDislikeTransformer;
        $this->likeDislike = $likeDislike;
    }

    /**
     * Return likes.
     *
     * @param $id
     *
     * @return mixed
     */
    public function getLikes($id)
    {
        $likes = $this->likeDislike->getLikesAll($id);

        return $this->response->withCollection($likes, $this->likeDislikeTransformer);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * Return likes count.
     *
     * @param $id
     *
     * @return mixed
     */
    public function getLikesCount($id)
    {
        $count = $this->likeDislike->getLikesCount($id);

        return $count;
    }

    /**
     * Return dislikes.
     *
     * @param $id
     *
     * @return mixed
     */
    public function getDislikes($id)
    {
        $dislikes = $this->likeDislike->getDislikesAll($id);

        return $this->response->withCollection($dislikes, $this->likeDislikeTransformer);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * Return dislikes count.
     *
     * @param $id
     *
     * @return mixed
     */
    public function getDislikesCount($id)
    {
        $count = $this->likeDislike->getDislikesCount($id);

        return $count;
    }

    /**
     * Create and store like or dislike.
     *
     * @param LikeDislikeStoreRequest $request
     *
     * @return mixed
     */
    public function store(LikeDislikeStoreRequest $request)
    {
        $check = $this->likeDislike->checkIfDataExists($request);

        if ($check == 0) {
            $like = $this->likeDislike->create($request->all());
            $this->callEventPushLikeDislike($like);

            return $this->response->withItem($like, $this->likeDislikeTransformer);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        } else {
            $like = $this->likeDislike->findOrFail($check);
            $this->callEventPushLikeDislike($like);
            if ($request->input('type') != $like->type) {
                $like = $this->likeDislike->update($request->all(), $like->id);

                return $this->response->withItem($like, $this->likeDislikeTransformer);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            } else {
                $this->likeDislike->delete($like->id);
            }
        }
    }

    /**
     * Call event push like/dislike.
     *
     * @param $likeDislike
     */
    private function callEventPushLikeDislike($likeDislike)
    {
        event(new LikeDislikePush($likeDislike));
    }
}


1		<?php
2
3		namespace App\Http\Controllers;
4
5		use App\Events\LikeDislikePush;
6		use App\Http\Requests\LikeDislikeStoreRequest;
7		use App\Repositories\LikeDislikeRepository as LikeDislike;
8		use App\Transformers\LikeDislikeTransformer;
9		use Chrisbjr\ApiGuard\Http\Controllers\ApiGuardController;
10
11		/**
12		* Class LikeDislikeController.
13		*/
14		class LikeDislikeController extends ApiGuardController
15		{
16		/**
17		* @var LikeDislikeTransformer
18		*/
19		protected $likeDislikeTransformer;
20
21		/**
22		* LikeDislike repository.
23		*
24		* @var LikeDislike
25		*/
26		private $likeDislike;
27
28		/**
29		* @var array
30		*/
31		protected $apiMethods = [
32		'getLikes' => [
33		'keyAuthentication' => false,
34		],
35		'getDislikes' => [
36		'keyAuthentication' => false,
37		],
38		'getLikesCount' => [
39		'keyAuthentication' => false,
40		],
41		'getDislikesCount' => [
42		'keyAuthentication' => false,
43		],
44		];
45
46		/**
47		* LikeDislikeController constructor.
48		*
49		* @param LikeDislike $likeDislike
50		* @param LikeDislikeTransformer $likeDislikeTransformer
51		*/
52	7	public function __construct(LikeDislike $likeDislike, LikeDislikeTransformer $likeDislikeTransformer)
53		{
54	7	parent::__construct();
55
56	7	$this->likeDislikeTransformer = $likeDislikeTransformer;
57	7	$this->likeDislike = $likeDislike;
58	7	}
59
60		/**
61		* Return likes.
62		*
63		* @param $id
64		*
65		* @return mixed
66		*/
67	3	public function getLikes($id)
68		{
69	3	$likes = $this->likeDislike->getLikesAll($id);
70
71	3	return $this->response->withCollection($likes, $this->likeDislikeTransformer);
		0 ignored issues – show Documentation introduced 2017-07-27 08:37 UTC by Report Bug Copy Issue Report `$this->likeDislikeTransformer` is of type `object<App\Transformers\LikeDislikeTransformer>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
72		}
73
74		/**
75		* Return likes count.
76		*
77		* @param $id
78		*
79		* @return mixed
80		*/
81	1	public function getLikesCount($id)
82		{
83	1	$count = $this->likeDislike->getLikesCount($id);
84
85	1	return $count;
86		}
87
88		/**
89		* Return dislikes.
90		*
91		* @param $id
92		*
93		* @return mixed
94		*/
95	2	public function getDislikes($id)
96		{
97	2	$dislikes = $this->likeDislike->getDislikesAll($id);
98
99	2	return $this->response->withCollection($dislikes, $this->likeDislikeTransformer);
		0 ignored issues – show Documentation introduced 2017-07-27 08:37 UTC by Report Bug Copy Issue Report `$this->likeDislikeTransformer` is of type `object<App\Transformers\LikeDislikeTransformer>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
100		}
101
102		/**
103		* Return dislikes count.
104		*
105		* @param $id
106		*
107		* @return mixed
108		*/
109	1	public function getDislikesCount($id)
110		{
111	1	$count = $this->likeDislike->getDislikesCount($id);
112
113	1	return $count;
114		}
115
116		/**
117		* Create and store like or dislike.
118		*
119		* @param LikeDislikeStoreRequest $request
120		*
121		* @return mixed
122		*/
123	3	public function store(LikeDislikeStoreRequest $request)
124		{
125	3	$check = $this->likeDislike->checkIfDataExists($request);
126
127	3	if ($check == 0) {
128	1	$like = $this->likeDislike->create($request->all());
129	1	$this->callEventPushLikeDislike($like);
130
131	1	return $this->response->withItem($like, $this->likeDislikeTransformer);
		0 ignored issues – show Documentation introduced 2017-07-27 08:37 UTC by Report Bug Copy Issue Report `$this->likeDislikeTransformer` is of type `object<App\Transformers\LikeDislikeTransformer>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
132		} else {
133	2	$like = $this->likeDislike->findOrFail($check);
134	2	$this->callEventPushLikeDislike($like);
135	2	if ($request->input('type') != $like->type) {
136	1	$like = $this->likeDislike->update($request->all(), $like->id);
137
138	1	return $this->response->withItem($like, $this->likeDislikeTransformer);
		0 ignored issues – show Documentation introduced 2017-07-27 08:37 UTC by Report Bug Copy Issue Report `$this->likeDislikeTransformer` is of type `object<App\Transformers\LikeDislikeTransformer>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
139		} else {
140	1	$this->likeDislike->delete($like->id);
141		}
142		}
143	1	}
144
145		/**
146		* Call event push like/dislike.
147		*
148		* @param $likeDislike
149		*/
150	3	private function callEventPushLikeDislike($likeDislike)
151		{
152	3	event(new LikeDislikePush($likeDislike));
153	3	}
154		}
155

deltcod / LaravelTube

LikeDislikeController::store() A last analyzed 2017-07-28 07:27 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

LikeDislikeController::store() A
last analyzed 2017-07-28 07:27 UTC