App\Modules\Notifications\Http\Controllers\PushNotificationDevicesController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	46
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	4	4	1
A	registerDevice()	8	8	1

<?php

namespace App\Modules\Notifications\Http\Controllers;

use Illuminate\Http\Request;
use App\Modules\Core\Http\Controllers\BaseApiController;
use \App\Modules\Notifications\Repositories\PushNotificationDeviceRepository;
use App\Modules\Core\Utl\CoreConfig;

class PushNotificationDevicesController extends BaseApiController

This class seems to be duplicated in your project.

{
    /**
     * List of all route actions that the base api controller
     * will skip permissions check for them.
     * @var array
     */
    protected $skipPermissionCheck = ['registerDevice'];

    /**
     * The validations rules used by the base api controller
     * to check before add.
     * @var array
     */
    protected $validationRules = [
    'device_token' => 'required|string|max:255',
    'user_id'      => 'required|exists:users,id'
    ];

    /**
     * Init new object.
     *
     * @param   PushNotificationDeviceRepository $repo
     * @param   CoreConfig                       $config
     * @return  void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(PushNotificationDeviceRepository $repo, CoreConfig $config)
    {
        parent::__construct($repo, $config, 'App\Modules\Notifications\Http\Resources\PushNotificationDevice');
    }

    /**
     * Register the given device to the logged in user.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function registerDevice(Request $request)
    {
        $this->validate($request, [
            'device_token' => 'required|string|max:255'
            ]);

        return new $this->modelResource($this->repo->registerDevice($request->all()));
    }
}