I think relying on JavaScript "truthiness" can be a little unclear; for example I'm not sure what's intended here; does it mean `if (url.parameters.length > 0)` or `if (url.parameters[0] != undefined)`. Also, this is changed from the previous logic (which was ` if(home.relativeCrumbs().join('/') != bcs.rest.join('/'))`) and nothing in the commit message mentions such a change or why it would be made, so a future developer might think it was an accident or just not know.