Support ability to pass array data to root template about inertia-laravel

[georgehanson]

@njoguamos that makes sense, passing an array would probably be a good feature to have.

[njoguamos]

@georgehanson Thanks for noting that. It was a typo and have updated my comment.

However, I dug deeper to Inertia and found out that withViewData does not work array

    public function withViewData($key, $value)
    {
        $this->viewData[$key] = $value;
        return $this;
    }

Instead it should be as suggested by this PR

    public function withViewData($key, $value = null)
    {
        if (is_array($key)) {
            foreach ($key as $k => $value) {
                $this->viewData[$k] = $value;
            }
        } else {
            $this->viewData[$key] = $value;
        }
        
        return $this;
    }

I hope am clear 😄

[nckrtl]

Hi @njoguamos

You can already pass an array, withViewData is not needed.

public function index() {
    return Inertia::render('Contact/Index', [
        'title'       => 'Contact Us | My App',
        'description' => 'Contact us with your comments, questions, and suggestions',
        'meta'        => 'Some other SEO meta'
    ]);
}

If you'll define the props (title, description and meta) in your Index.vue the values will become available.

[connecteev]

+1 on this...unless I'm missing a way to update the content in the tag for SEO?

[enzonotario]

@connecteev check this: https://github.com/inertiajs/inertia-laravel#accessing-data-in-root-template

[njoguamos]

I believe the idea of @reinink having withViewData is to be able to pass variables to Laravel blade template.

@nckrtl the solution you provided works only if one want to pass data to vue (which by the way is okay) but I want the array data to be accessible to my app.blade.php

[nckrtl]

aha, in that case I misunderstood your question.

[njoguamos]

😄 Not really, I am the one who did not explain well.

[georgehanson]

@njoguamos I noticed there is a comma after specifying the Vue Component name. The render method on the ResponseFactory has the second argument default to an empty array if it has not been provided, so this exception shouldn't be thrown. Have you tried changing it from this:

public function index() {
    return Inertia::render('Contact/Index',)->withViewData([
        'title'       => 'Contact Us | My App',
        'description' => 'Contact us with your comments, questions, and suggestions',
        'meta'        => 'Some other SEO meta'
    ]);
}

To this:

public function index() {
    return Inertia::render('Contact/Index')->withViewData([
        'title'       => 'Contact Us | My App',
        'description' => 'Contact us with your comments, questions, and suggestions',
        'meta'        => 'Some other SEO meta'
    ]);
}

[connecteev]

@enzonotario thanks

[njoguamos]

@reinink has updated withViewData to support array see commit