Fixes duplicate message getting printed by console.error

Line 839's logError was logging generic errors and since the error is
not caught it was also logged by defaultActionHandler.error at line 609.

Previously for a model hook syntax error I would get: "Error while
processing route: index" "foo is not defined" logged and immediately
after it "foo is not defined".

This works with and without error substates and when a "too many dynamic
segments" error is thrown.
[BUGFIX beta] paramsFor: don’t clobber falsy params

Fixes #5275

and other minor cleanup
[BUGFIX beta] controllers w query params are unit testable

Closes #5078
[BUGFIX beta] controllers have new QP values before setupController

New QP values weren't getting set until
finalizeQueryParamChanges, but they should have been
set before `setupController` is called, and this
change fixes that.

I commented out a test case for an extremely unlikely
corner case where the active class will be incorrectly
calculated if the user types in a non-Ember-generated,
unpruned URL, but I intend to fix this all in an active
class refactor shortly, but it's better to get this
fix in than delay.
[BUGFIX beta] use get() to fetch controller queryParams

Fixes #5102
[DOC beta] Mark Route#render options as optional
[BUGFIX beta] Fix initial render of {{input type=bound}} for checkboxes

Conflicts:
	packages/ember-handlebars/tests/controls/checkbox_test.js
[BUGFIX beta] makeBoundHelper supports unquoted bound property options - Fixes #5245

Conflicts:
	packages/ember-handlebars/lib/ext.js