CODEWITHSUNDEEP
phplaravellaravel-5repository
user3732216
user3732216

Reputation: 1589

Repositories Not be Instantiated

I'm trying to find out why I'm receiving this error. I'm following along. However the only difference is that at the time of the recording it was done with Laravel 4.25 and I am now using Laravel 5.0.

Repositories and Inheritance

BindingResolutionException in Container.php line 785:

Target [App\Repositories\User\UserRepository] is not instantiable.

<?php 
    namespace App\Http\Controllers;

    use App\Http\Requests;
    use App\Http\Controllers\Controller;
    use App\Repositories\User\UserRepository;

    use Illuminate\Http\Request;

    class UsersController extends Controller {

        private $userRepository;

        public function __construct(UserRepository $userRepository) {
            $this->userRepository = $userRepository;
        }

        /**
         * Display a listing of the resource.
         *
         * @return Response
         */
        public function index() {
            $users = $this->userRepository->getAll();

            return $users;
        }
    }

<?php

    namespace App\Repositories\User;

    use App\Repositories\EloquentRepository;

    class EloquentUserRepository extends EloquentRepository implements UserRepository 
    {
        private $model;

        function __construct(User $model) {
            $this->model = $model;
        }
    }

<?php

    namespace App\Repositories\User;

    interface UserRepository {
        public function getAll();    
    }

<?php

    namespace App\Repositories;

    abstract class EloquentRepository {

        public function getAll() {
            return $this->model->all();
        }

        public function getById() {
            return $this->model->findOrFail($id);
        }
    }

Upvotes: 1

Views: 920

Answers (1)

Scopey
Scopey

Reputation: 6319

You are type hinting an interface, and not the class itself. This error is occurring because Laravel cannot bind an interface because the binding must be instantiable. Abstract classes or interfaces are not valid unless Laravel knows the concrete (instantiable) class to substitute in for the abstract class / interface.

You will need to bind the EloquentUserRepository to the interface:

App::bind('UserRepository', 'EloquentUserRepository');

Upvotes: 2

Related Questions