Ziggy does not find the id key in object about ziggy

[ankurk91]

Some insights -

// routes/web.php
Route::resource(
        'users', 'UserController', ['only' => [
            'index', 'store', 'edit', 'update'
        ]]
    );

Ziggy example

route('users.edit',{user:1}).url()

Result -

"http://app.local/users/1/edit"

It does not look for id but user property because the route requires {user}.

$ php artisan route:list | grep users.edit

|        | GET|HEAD  | users/{user}/edit                   | users.edit              | App\Http\Controllers\UserController@edit                               | web,auth,role:admin,can:manage-users,user                 |

Notice the {user} in url

[DanielCoulbourne]

Yeah that test is passing...
@lamberttraccard if you can give me a combination of a route definition and a Ziggy usage that makes this break, please re-open.

I'd love a failing test

[the94air]

It is not merged yet as Daniel said

It may have gotten lost in the big rewrite. I'll go back and look to see if it's there.

@DanielCoulbourne can you look at @avrahamappel test

[fridzema]

I had this error a few times now.
Make sure that the value you are passing to object is defined/not empty!

[DanielCoulbourne]

Thanks I’ll look into this today.

[DanielCoulbourne]

@lamberttraccard can you show me the code that caused this error?

[lamberttraccard]

@DanielCoulbourne

[lamberttraccard]

Thanks @ankurk91, yes it's seems that Ziggy doesn't work with route model binding.

[DanielCoulbourne]

I know we did support looking up by generic ID at one point. I believe @jesseleite wrote that.

It may have gotten lost in the big rewrite. I'll go back and look to see if it's there.

[jesseleite]

Yessir 👌 https://github.com/tightenco/ziggy/blob/master/tests/js/test.route.js#L101-L113

[avrahamappel]

@DanielCoulbourne
I'm having this issue as well.

route('contracts.revise', {id:1}).url()

I get

Uncaught Error: Ziggy Error: 'contract' key is required for route 'contracts.revise'

Route definition:

Route::post('contracts/{contract}/revise', 'ContractController@revise')->name('contracts.revise');

I don't have anything filtered right now.

[the94air]

Try this @avrahamappel

route('contracts.revise', {contract:1}).url()

[avrahamappel]

I understand, but that was my test case. In reality I'm passing a JavaScript representation of an Eloquent Model in there, and according to what I understood from the above discussion, Ziggy should be able to find the id field in the object.

[DanielCoulbourne]

Will do! May be a day or two unfortunately but thanks for the heads up.