laravel APi resource Call to undefined method Illuminate\Database\Query\Builder::mapInto()

Question

i have Post and User model with one to one relation and it works good:

//User.php

public function post(){
    return $this->hasOne(Post::class);
}


// Post.php

public function user() {
    return $this->belongsTo(User::class);
}

now i create API resources:

php artisan make:resource Post
php artisan make:resource User

I need to return all post with an api call then i set my route:

//web.php: /resource/posts

Route::get('/resource/posts', function () {
    return PostResource::collection(Post::all());
});

this is my Posts resource class:

<?php

namespace App\Http\Resources;
use Illuminate\Http\Resources\Json\Resource;
use App\Http\Resources\User as UserResource;

class Posts extends Resource
{
/**
 * Transform the resource into an array.
 *
 * @param  \Illuminate\Http\Request  $request
 * @return array
 */
public function toArray($request)
{
      return [
        'id' => $this->id,
        'title' => $this->title,
        'slug' => $this->slug,
        'bodys' => $this->body,
        'users' => UserResource::collection($this->user),
        'published' => $this->published,
        'created_at' => $this->created_at,
        'updated_at' => $this->updated_at,
    ];

}
}

this is the error:

Call to undefined method Illuminate\Database\Query\Builder::mapInto()

if i remove:

'users' => UserResource::collection($this->user),

it's work but i need to include relations in my api json, i have read and followed the doc at https://laravel.com/docs/5.5/collections.

this is my User resource class:

```

namespace App\Http\Resources;

use Illuminate\Http\Resources\Json\Resource;

class User extends Resource
{
/**
 * Transform the resource into an array.
 *
 * @param  \Illuminate\Http\Request  $request
 * @return array
 */
public function toArray($request)
{
   return [
       'user_id' => $this->user_id,
       'name' => $this->name,
       'lastname' => $this->lastname,
       'email' => $this->email
   ];
}
}

any ideas where am i wrong?

Maraboc · Answer 1 · 2018-03-05T08:55:16.547

141

The problem is that you use UserResource::collection($this->user) and you have just one element not a collection so you can replace it with new UserResource($this->user) like this :

return [
    'id' => $this->id,
    'title' => $this->title,
    'slug' => $this->slug,
    'bodys' => $this->body,
    'users' => new UserResource($this->user),
    'published' => $this->published,
    'created_at' => $this->created_at,
    'updated_at' => $this->updated_at,
];

edited Mar 05 '18 at 08:55

answered Dec 08 '17 at 09:07

Maraboc

10,550
3
37
48

1

In my case, I was using a collection in collection resource. – Payam Khaninejad Jun 26 '18 at 07:26
thank you so much . I chose to use it instead of using it in the array: https://paste.laravel.io/198efba2-9954-4c9c-856e-e053e0d1f584 – melih sahin May 28 '19 at 14:21
@runtimeTerror also mentioned the use of the static method `make` like so: `UserResource::make($this->user)`. I would like to know what's difference between the two aside from the later is using a static method? – basagabi Sep 07 '22 at 09:08

score 12 · Answer 2 · answered Sep 20 '21 at 17:39

12

In Laravel 8.5.*, you can use the static method make on the collection to get the same result. It would be like UserResource::make($this->user)

answered Sep 20 '21 at 17:39

runtimeTerror

398
3
14

score 11 · Answer 3 · edited Sep 22 '22 at 17:07

11

this problem is that you use UserResource::collection($this->user) it mean you have many users but and you have just one element for the a single resource and not a collection so you can replace it with new UserResource($this->user)

edited Sep 22 '22 at 17:07

runtimeTerror

398
3
14

answered Sep 21 '19 at 08:19

Amirex

241
3
9

elliot · Answer 4 · 2022-07-19T16:41:43.777

when you have just one element and not a collection use new Resouce

return [
        'id' => $this->id,
        'name' => $this->name,
        'description' => $this->description,
        'user' => new UserResource($this->whenLoaded('user')),
];

and give a try to use the eager loading for better perfomance not just this. using the method whenLoaded and put the relations name in ( ).

Resource::collection is used when the model has hasMany relation. If your model has hasOne relation use Resource::make(Book::all())

score 2 · Accepted Answer · answered Mar 22 '23 at 09:40

The issue is that you use UserResource::collection($this->user) and you have just one element come from database, not list of collection. It would be solved if you used new UserResource($this->user) like this:

<?php

namespace App\Http\Resources;
use Illuminate\Http\Resources\Json\Resource;
use App\Http\Resources\User as UserResource;

class Posts extends Resource
{
/**
 * Transform the resource into an array.
 *
 * @param  \Illuminate\Http\Request  $request
 * @return array
 */
public function toArray($request)
{
      return [
        'id' => $this->id,
        'title' => $this->title,
        'slug' => $this->slug,
        'bodys' => $this->body,
        'users' => new UserResource($this->user),
        'published' => $this->published,
        'created_at' => $this->created_at,
        'updated_at' => $this->updated_at,
    ];

}
}

laravel APi resource Call to undefined method Illuminate\Database\Query\Builder::mapInto()

5 Answers5

Linked