Propaganistas\LaravelFakeId\RoutesWithFakeIds

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getRouteKey()	0	4	1
A	resolveRouteBinding()	0	10	2

<?php namespace Propaganistas\LaravelFakeId;

use Illuminate\Support\Facades\App;
use Exception;

trait RoutesWithFakeIds
{
    /**
     * Get the value of the model's route key.
     *
     * @return mixed
     */
    public function getRouteKey()
    {
        return App::make('fakeid')->encode($this->getKey());

It seems like getKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Retrieve model for route model binding
     *
     * @param  mixed  $value
     * @return \Illuminate\Database\Eloquent\Model|null
     */
    public function resolveRouteBinding($value)
    {
        try {
            $value = App::make('fakeid')->decode($value);
        } catch (Exception $e) {
            return null;
        }

        return $this->where($this->getRouteKeyName(), $value)->first();

The method getRouteKeyName() does not exist on Propaganistas\LaravelFakeId\RoutesWithFakeIds. Did you maybe mean getRouteKey()?

It seems like where() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}