TaskManager\Services\ApiService

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getApi()	0	2	1
A	generateKey()	0	7	1
A	exists()	0	7	2

<?php
/**
 * Created by PhpStorm.
 * User: me
 * Date: 17.06.17
 * Time: 21:34
 */
namespace TaskManager\Services;

use TaskManager\ApiKeys;

class ApiService
{

    protected $api;
    function __construct(\TaskManager\ApiKeys $api)
    {
    $this->api = $api;
    }

    public function generateKey($user_id=null)

The parameter $user_id is not used and could be removed.

    {
        $this->api->key = uniqid();

The property key does not seem to exist on TaskManager\ApiKeys. Are you sure there is no database migration missing?

        $this->api->active = true;

The property active does not seem to exist on TaskManager\ApiKeys. Are you sure there is no database migration missing?

        $this->api->user_id = auth()->id();

The method id() does not exist on Illuminate\Contracts\Auth\Factory.

The property user_id does not exist on TaskManager\ApiKeys. Did you mean user?

        $this->api->save();
        return $this->api->key;
    }

    public function getApi(){
        return ApiKeys::where('user_id', auth()->id())->first();
    }


    public function exists()
    {
      // dd($this->api->where('user_id', auth()->id())->first());

70% of this comment could be valid code. Did you maybe forget this after debugging?

        if (count(ApiKeys::where('user_id', auth()->id())->first())) {
            return true;
        } else {
            return false;
        }

    }
}