Level 61
Rename it to getFirstImageAttribute.
Jan 11, 2021
15
Level 4
Accessors not working in API
I have created an accessor to modify a database value
public function getImage1Attribute($value)
{
return $value ? asset('storage/hosts/images/' . $value) : null;
}
but I am still getting original value in JSON API like this
{
"message": "success",
"data": {
"id": 1,
"image_1": "n0rVxA1L-1610381013.jpg",
"created_at": "2021-01-07 18:02:56",
"updated_at": "2021-01-11 16:03:33"
}
}
I am passing response like this
return response([
'message' => Lang::get('api.success'),
'data' => HostDetail::find($id)
]);
I am not getting this problem with other models
I found out that the problem is with the column name image_1 but laravel is considering as image1. Is there any way around this?
Level 4
@bugsysha for this I have to replace it everywhere I used image_1 to first_image
Level 61
Not sure what is your approach but I never use numbers when naming stuff.
Have you tried naming it getImage_1Attribute?
Level 4
Me too. But this is an already existing application of one of the clients. Yes I tried getImage_1Attribute. But no luck
Level 4
getImage1Attribute It does work when I access it like this $host->image_1, but not when sending the json response
Level 61
How many models do have image_1 object property/database column?
Level 4
There are 2 of them
Level 61
That is then super easy change. Search and replace is your friend.
Level 24
will this work ?
protected $appends = ['image1'];
Level 4
No, it doesn't. It has the same problem as image_1
Level 24
even ?
protected $appends = ['image_1'];
1 like
Level 4
Yes appends only works for custom attributes. So this will give null
Level 24
search and replace in editor then xD in vscode it is very easy
Level 4
Creating ApisResource is also an alternate solution, or instead of returning the model returning the columns one by one in an array is also a solution. I think I will have to go with these
Level 24
well, actually you are try to add extra code to fix another code, this may cause problems later
just like @bugsysha say, i also never name the accessor like that xD
Please or to participate in this conversation.