laravel,how i can change the code to return an array?

Question

i have this controller which access the pivot table in many to many relationships. the columns in the pivot table are values and created_at. i want this controller to return me an array of values and created_at

public function status($s_type)
{

    $o_health_status = Status::where('name', 'health')->first();
    $o_speed_status = Status::where('name', 'speed')->first();

    if ($s_type === 'health'){

        $o_response = $this->statuses()->where('status_id', $o_health_status->id)
        ->select('values','created_at')->orderBy('created_at','ASC')->first();

        if($o_response === null){
            return 'unsigned';
        }
        $o_response = $o_response->values;
        return $o_response;         
    }else{

        $o_response = $this->statuses()->where('status_id', $o_speed_status->id)
        ->select('values','created_at')->orderBy('created_at', 'desc')->first();
        if($o_response === null){
            return 'unsigned';
        }

        $o_response = $o_response->values;
        return $o_response;


    }

It's very unclear what you are asking.

TheAlexLichter
– TheAlexLichter

2016-11-15 15:07:39 +00:00
Commented Nov 15, 2016 at 15:07 — TheAlexLichter
– TheAlexLichter, Commented Nov 15, 2016 at 15:07

Jorge Y. C. Rodriguez · Accepted Answer · 2016-11-17 19:55:23Z

1

You need to use pluck for this. this will pluck all required elements from a collection: so I clean up a bit and should looks something like this:

public function status($s_type)
{

    $o_health_status = Status::where('name', 'health')->first();
    $o_speed_status = Status::where('name', 'speed')->first();
    $o_response = $s_type === 'health' ? $this->statuses()->pluck('values','created_at')->where('status_id', $o_health_status->id)
            ->select('values', 'created_at')->orderBy('created_at', 'asc')->first() : $this->statuses()->pluck('values','created_at')->where('status_id', $o_speed_status->id)
            ->select('values', 'created_at')->orderBy('created_at', 'desc')->first();

    return ! $o_response ? 'unsigned' : $o_response->values;
}

edited Nov 17, 2016 at 19:55

answered Nov 15, 2016 at 15:11

Jorge Y. C. Rodriguez

3,4565 gold badges42 silver badges64 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

abraham foto Over a year ago

for same reason, your code looks perfect, but when i replaced my code with your code. it doesnt work at all.

Sachith Muhandiram Over a year ago

if its not working., why you marked as the answer? please check again.

Sachith Muhandiram · Accepted Answer · 2016-12-08 18:31:25Z

0

In laravel, if you need to return an array from the query, you can use pluck() instead of select()

$o_response = $this->statuses()->where('status_id', $o_health_status->id)
    ->pluck('values','created_at')->orderBy('created_at','ASC');// ->first();
//I think you no need to use ->first(); here.

Docs (scroll down to Retrieving A List Of Column Values)

edited Dec 8, 2016 at 18:31

user4627303

answered Nov 15, 2016 at 15:09

Sachith Muhandiram

2,97811 gold badges54 silver badges113 bronze badges

2 Comments

abraham foto Over a year ago

for same reason, when i wrote the code you suggested. it gave me error that says, Method orderBy does not exist! . are you allowed to use pluck and orderdBy at the same time?

Sachith Muhandiram Over a year ago

normally the resultant array comming according to the values having in the database. if they are ordered, then nothing to worry.

Collectives™ on Stack Overflow

laravel,how i can change the code to return an array?

2 Answers 2

2 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related