replace diffAssoc with diff so the method is removed when there are parameters in front of it

[TomNealBorneman]

Fixes #2216

[tabuna]

Hi @TomNealBorneman, could you add some tests to test this case?

[TomNealBorneman]

When using Model bindings in your screen method (say the save function for example) you get a 404 when trying to save a new instance of a child relation:

This happens because it tries to use the method as a key for finding the model. And a model with the key 'save' does not exist. Changing the diffAssoc to diff makes it so the method is removed from the parameters regardless of its position within the collection. The following dumps (made after src\Screen\Screen.php:220) show the different results.

diff:

diffAssoc:

Dump:

This still does not work when you use extra url parts like http://laravel-cms.test/dashboard/users/38/test/create because then it will still use create as a key for the model binding. But in its current iteration the binding does not work at all.

This fix could cause issues if you have models with string keys that could overlap with methods, say if you have a model with the key save it would bind that instead of creating a new model as intended. But this seems like an edge case to me.

This has no effect on saving existing models like so: http://laravel-cms.test/dashboard/users/38/test/1 because the identifier is present and can be used for the binding.

edit: add clarification of parameters dumped in screenshots.

[madyanmalfi]

Changing to diff fixes the issue for me :). If this isn't going to get merged, then please provide anther solution.