Ensure Ziggy.defaults is always an object

[bakerkretzmar]

This PR fixes #366 by ensuring that the JavaScript Ziggy.defaults property is always an object, even when it's empty.

The bug in #366 happens because previously, when defaults was an empty PHP array, it was encoded as an empty JavaScript array too. JavaScript arrays of course have several built-in methods, like shift() and reduce(), and so if you had a route parameter name that matched one of those methods, and you tried to access it on an array, it would return that method:

const defaults = [];
const name1 = 'users';
const name2 = 'shift';

!!defaults[name1]; // false, because it evaluates to defaults['users'] which doesn't exist
!!defaults[name2]; // true, because it evaluates to defaults['shift'] which DOES exist and is a function

If there are default parameters, defaults will be an object, which avoids this issue, but if there aren't, it would have been an array, so any route parameter names matching built-in JavaScript array methods would cause errors like this.

[sbine]

🏆