App\Http\Controllers\ApiKeyController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes	3
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	makeKey()	0	16	1
A	revokeKey()	0	4	1
A	getKey()	0	4	1
A	deleteKey()	0	7	1

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Http\Request;
use Chrisbjr\ApiGuard\Models\ApiKey;
use App\Http\Requests;

class ApiKeyController extends Controller
{
    protected $apiKey;

    /**
     * ApiKeyController constructor.
     * @param ApiKey $apiKey
     */
    public function __construct(ApiKey $apiKey)
    {
        $this->middleware('auth');
        $this->middleware('lang');

        $this->apiKey = $apiKey;
    }

    /**
     * Create a new api token.
     *
     * TODO: implement session to the view.
     * TODO: Build api key limit that the user can have max. 5 keys.
     * @param Request $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function makeKey(Request $request)
    {
        $id         = auth()->user()->id;

The method user() does not seem to exist on object<Illuminate\Contracts\Auth\Factory>.

        $create     = $this->apiKey->make($id);
        $insertedId = $create->id;

        $serviceInsert          = $this->apiKey->find($insertedId);
        $serviceInsert->service = $request->get('service');
        $serviceInsert->save();

        $token = $this->apiKey->find($insertedId);

        session()->flash('message', 'Token created');
        session()->flash('token', $token->key);
        return redirect()->back(302);
    }

    public function revokeKey()
    {

    }

    /**
     * Get the apikey
     *
     * @param int $id the database id off the api key.
     * @return \Illuminate\Http\RedirectResponse
     */
    public function getKey($id)

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

    {
        return redirect()->back(302);
    }

    /**
     * Delete a api key out off the system.
     *
     * @param int $id the database id off the api key.
     * @return \Illuminate\Http\RedirectResponse
     */
    public function deleteKey($id)
    {
        $this->apiKey->destroy($id);
        session()->flash('message', 'The api key is deleted.');

        return redirect()->back(302);
    }

}