NotificationController::create() - Code Metrics - Inspection of "Merge pull request #138 from murrant/route_cleanup" - librenms/librenmsv2 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 744b64...0d51b0 )

by Tony

created 2016-05-27 17:49 UTC

NotificationController::create() A

↳ Parent: NotificationController

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	0
cts	0
cp	0
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	1
crap	2

<?php

namespace App\Http\Controllers;

use Dingo\Api\Http;
use Dingo\Api\Routing\Helpers;
use Illuminate\Http\Request;

class NotificationController extends Controller
{
    use Helpers;

    /**
     * Display a listing of the resource.
     *
     * @param null $type
     * @return \Illuminate\Http\Response

     */
    public function index($type = null)
    {

        $notifications = $this->api->be(auth()->user())->get('/api/notifications/'.$type);

        if ($type === 'archive') {
            $page = '';
            $button = 'Notifications';
            $bg = 'maroon';
        }
        else {
            $page = 'archive';
            $button = 'Archive';
            $bg = 'blue';
        }

        return view('notifications.list', compact(['notifications', 'page', 'button', 'bg', "type"]));
    }

    /**
     * Update a notifications status.
     *
     * @param $id
     * @param $action
     * @return \Illuminate\Http\JsonResponse
     */
    public function update($id, $action)
    {
        return response()->json($this->api->be(auth()->user())->patch('/api/notifications/'.$id.'/'.$action));

    }

    /**
     * Create a new notification
     *
     * @param Request $request
     * @return \Illuminate\Http\JsonResponse
     */
    public function create(Request $request)
    {
        $this->validate($request, [
            'title' => 'required|unique:notifications|max:255',
            'body'  => 'required',
        ]);
        return response()->json($this->api->be(auth()->user())->put('/api/notifications', ['title' => $request->title, 'body' => $request->body]));

    }

}


1		<?php
2
3		namespace App\Http\Controllers;
4
5		use Dingo\Api\Http;
6		use Dingo\Api\Routing\Helpers;
7		use Illuminate\Http\Request;
8
9		class NotificationController extends Controller
10		{
11		use Helpers;
12
13		/**
14		* Display a listing of the resource.
15		*
16	1	* @param null $type
17		* @return \Illuminate\Http\Response
		0 ignored issues – show Documentation introduced 2016-04-13 21:28 UTC by Report Bug Copy Issue Report Should the return type not be `\Illuminate\View\View\|\I...\Contracts\View\Factory`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
18	1	*/
19	1	public function index($type = null)
20		{
21
22		$notifications = $this->api->be(auth()->user())->get('/api/notifications/'.$type);
		0 ignored issues – show Bug introduced 2016-03-01 01:02 UTC by Report Bug Copy Issue Report The method `user()` does not seem to exist on `object<Illuminate\Contracts\Auth\Factory>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
23		if ($type === 'archive') {
24		$page = '';
25		$button = 'Notifications';
26		$bg = 'maroon';
27	1	}
28		else {
29		$page = 'archive';
30	1	$button = 'Archive';
31	1	$bg = 'blue';
32	1	}
33	1
34	1	return view('notifications.list', compact(['notifications', 'page', 'button', 'bg', "type"]));
35	1	}
36
37	1	/**
38	1	* Update a notifications status.
39	1	*
40		* @param $id
41		* @param $action
42	1	* @return \Illuminate\Http\JsonResponse
43		*/
44		public function update($id, $action)
45		{
46		return response()->json($this->api->be(auth()->user())->patch('/api/notifications/'.$id.'/'.$action));
		0 ignored issues – show Bug introduced 2016-03-01 01:02 UTC by Report Bug Copy Issue Report The method `user()` does not seem to exist on `object<Illuminate\Contracts\Auth\Factory>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
47		}
48
49		/**
50		* Create a new notification
51		*
52		* @param Request $request
53		* @return \Illuminate\Http\JsonResponse
54		*/
55		public function create(Request $request)
56		{
57		$this->validate($request, [
58		'title' => 'required\|unique:notifications\|max:255',
59		'body' => 'required',
60		]);
61		return response()->json($this->api->be(auth()->user())->put('/api/notifications', ['title' => $request->title, 'body' => $request->body]));
		0 ignored issues – show Bug introduced 2016-03-07 17:11 UTC by Report Bug Copy Issue Report The method `user()` does not seem to exist on `object<Illuminate\Contracts\Auth\Factory>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
62		}
63
64		}
65

librenms / librenmsv2

Push — master ( 744b64...0d51b0 )

NotificationController::create() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like