JeroenG\GuestPass\GuestPassService

Complexity

Total Complexity

8

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	routes()	0	5	1
A	create()	0	17	2
A	isOwner()	0	4	1
A	getKeysOf()	0	7	1
A	findGuestPass()	0	9	1
A	getGuestPass()	0	4	1

<?php

namespace JeroenG\GuestPass;

use Illuminate\Database\Eloquent\Model;

class GuestPassService
{
    /**
     * Create a new instance.
     */
    public function __construct()
    {
        // constructor body
    }

    /**
     * Register the routes.
     * @return void
     */
    public function routes()
    {
        $router = app('router');
        $router->get('gp/{owner}/{key}', 'JeroenG\GuestPass\Controllers\Access');
    }

    /**
     * Create a new Guest Pass.
     * @param  lluminate\Database\Eloquent\Model $owner

Should the type for parameter $owner not be Model?

     * @param  lluminate\Database\Eloquent\Model $object

Should the type for parameter $object not be Model?

     * @param  string|null                       $view   Leave out the .blade.php extension.
     * @return bool|Exception
     */
    public function create(Model $owner, Model $object, $view = null)
    {
        try {
            GuestPass::create([

The method create() does not exist on JeroenG\GuestPass\GuestPass. Did you maybe mean created()?

                'key' => str_random(10),
                'owner_id' => $owner->id,
                'owner_model' => get_class($owner),
                'object_id' => $object->id,
                'object_model' => get_class($object),
                'view' => $view,
            ]);

            return true;
        } catch (Exception $e) {

The class JeroenG\GuestPass\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

            return false;
        }
    }

    /**
     * Check if the given user created the Guest Pass.
     * @param  lluminate\Database\Eloquent\Model $owner

Should the type for parameter $owner not be Model?

     * @param  lluminate\Database\Eloquent\Model $guestpass

Should the type for parameter $guestpass not be Model?

     * @return bool
     */
    public function isOwner(Model $owner, Model $guestpass)
    {
        return $owner->id == $guestpass->owner_id;
    }

    /**
     * Get all keys belonging to the owner.
     * @param  lluminate\Database\Eloquent\Model  $owner

Should the type for parameter $owner not be Model?

     * @return array
     */
    public function getKeysOf(Model $owner)
    {
        return GuestPass::where([
            'owner_id' => $owner->id,
            'owner_model' => get_class($owner),
        ])->get()->keyBy('key');
    }

    /**
     * Find (or fail) the Guest Pass belonging to an owner-object pairing.
     * @param  lluminate\Database\Eloquent\Model                $owner
     * @param  lluminate\Database\Eloquent\Model                $object

Should the type for parameter $object not be Model?

     * @return lluminate\Database\Eloquent\Collection|Exception
     */
    public function findGuestPass(Model $owner, Model $object)
    {
        return GuestPass::where([
            'owner_id' => $owner->id,
            'owner_model' => get_class($owner),
            'object_id' => $object->id,
            'object_model' => get_class($object),
        ])->firstOrFail();
    }

    /**
     * Get a Guest Pass by its key (or fail).
     * @param  string                                      $key
     * @return lluminate\Database\Eloquent\Model|Exception
     */
    public function getGuestPass($key)
    {
        return GuestPass::where('key', $key)->firstOrFail();
    }
}