I need some help with the map function

[MrBean14]

Hi all,

in my project, I'm exporting a sorted Collection to Excel, working fine for the moment.

The result is a collection of cars with tasks/times. A car can have 1+ tasks/times.
After each car I would like to have an empty line containing a COUNT of the time worked on it.

I started to try the following, but the result is not what I expected. So I think I'm misunderstanding the map function.
Can somebody tell me if my idea is totally off and indeed I'm misunderstanding it?

PS the "0" or "1" I'm returning in last columns is only for test.

public function map($time): array
    {
        $dt = Carbon::parse($time->stop)->diffInSeconds(Carbon::parse($time->start));
        $duur = (is_null($time->stop)) ? 0 : $dt;


        $car = $time->car->brand . " " . $time->car->type . " (" . $time->car->license . ")";

        if (!isset($lastcar)){
            $lastcar = $car;
        }

        if ($lastcar != $car) {
            return [
                [
                    '-',
                    '-',
                    '1',
                ],
                [
                    $time->car->brand . " " . $time->car->type . " (" . $time->car->license . ")",
                    $time->task->name,
                    $time->employee->name,
                    $time->start,
                    $time->stop,
                    '=' . $duur . '/86400',
                    '0',
                ]
                ];
        }
        else {
            return [
                $time->car->brand . " " . $time->car->type . " (" . $time->car->license . ")",
                $time->task->name,
                $time->employee->name,
                $time->start,
                $time->stop,
                '=' . $duur . '/86400',
                '1',
            ];
        }

        if (isset($totTime[$car])) {
                $totTime[$car]['totaltime'] = $totTime[$car]['totaltime'] + $dt;
            }
            else {
                $totTime[$car]['totaltime'] = $dt;
            }

        $lastcar = $car;
    }

In case this is totally stupid, what would be the best to get this exported please ?

Thanks in advance.

Br,

Davy

[patrickbrouwers]

Hey Davy, this is a very implementation specific question that I currently don't have time to have a close look at. Perhaps a forum like Laracasts or Stackoverflow will be of quicker assistance.

[MrBean14]

Patrick, thanks for your answer.
The question is actually not implementation specific. I'm just trying to understand if the map function does allow what I'm trying to do, with the if/else construction?
Not understanding what "happens" in the MAP function, so it might be a very strange idea of me to think it would be possible while it is not at all... :-)

[patrickbrouwers]

Well I have to delve through a lot of lines of code to understand what you are asking me, so I call that pretty implementation specific :)

The map function works the same as array_map and collection->map(). You can do if-statements yes. Perhaps have another look at the documentation: https://docs.laravel-excel.com/3.1/exports/mapping.html#mapping-rows

[MrBean14]

Thanks again Patrick,

I looked many times at the documentation, but in the examples nowhere an if/else is used, that's the reason why I tried it (but concluded it is not doing what I'm asking :-) :-) ).
I'm actually just trying to do :

1. If option x : put one line
2. If option y : put two lines

[patrickbrouwers]

Try

return [
 [
                $time->car->brand . " " . $time->car->type . " (" . $time->car->license . ")",
                $time->task->name,
                $time->employee->name,
                $time->start,
                $time->stop,
                '=' . $duur . '/86400',
                '1',
      ]
];

I'm sorry, but I don't have more help to try debugging this for you. We are available on commercial basis if this is business critical for you: https://laravel-excel.com/commercial-support

[MrBean14]

No worries Patrick. Not business critical at all... :-)
Just testing and trying to get it working.
Thanks for your time and effort.

The above return is working as such btw, just the if/else part does not seem to be ok.
But I will try some more.