presenter.formatter.DateFormatter fixes

- PCTLIBRARY-1084: DateFormatter gets the timezone offset wrong if it contains half hours or quarter hours (@janh).
- PCTLIBRARY-1136 Bugfix: changed the if condition to check the date format to be an instance of Date (@jasonn).
PCTLIBRARY-1068: sub task 1112 Fix knockoutplugin bug which deals with 'click' events in verifier (@jasonn).
NO-JIRA: code style update with some changes to the actual code (checks for null, undefined, simplified if/else, ...) (@ewant).
PCTLIBRARY-1211 Only remove listeners on SelectionField options property

 - Removing listeners on SelectionField properties value, enabled, visible was unnecessary because they are instances of Property and were already being removed by removeChildListeners
 - options property is a NodeList, not a Property, so it doesn't get put in the list of Properties by PresentationNode.properties()

Also adds missing AT test file for the ExtJsComboBoxControl.

(@jeremyh, @janh)
Allow quoted arguments with spaces in them when running 'brjs' from a
UNIX machine.
Allow the `dep-insight -a` command to be used with partially defined
aliases by pointing the alias to the interface name that we do not yet
have a concrete definition for.
Show all source modules, regardless of whether they will be bundled or
not, if the source module being inspected will itself not be bundled,
since this represents a "Why is this thing not being bundled?" question,
which all requires unbundled dependencies to be shown.
Merge pull request #426 from BladeRunnerJS/support-arguments-with-spaces-on-unix

Allow quoted arguments with spaces in them when running 'brjs' from a
Merge pull request #425 from BladeRunnerJS/allow-dep-insight-to-be-used-with-partially-defined-aliases

Allow the `dep-insight -a` command to be used with partially defined
Allow quoted arguments with spaces in them when running 'brjs' from a
UNIX machine.
moving service/alias logic into bundleable node so anyone using aliases can talk about the service prefix - temporary workaround until we get a formalised mechanism)
Changing how we locate the app based on locateAncestorNodeOfClass so that our dashboard loads
Revert based on merge of 11a6b35891186123e654616805558783d632a0e4
Merge pull request #429 from BladeRunnerJS/allow-why-is-this-not-being-bundled-questions-to-be-answered

Allow why is this not being bundled questions to be answered
Removing class reference
Merge branch 'develop' of https://github.com/BladeRunnerJS/brjs into develop
Improved one of our error messages.
Move `JsCommentStrippingReader` class to 'brjs-core'.
Strip JavaScript comments before scanning for require() methods.
Strip JavaScript comments before processing with the trie.
Merge pull request #430 from BladeRunnerJS/strip-comments-before-processing-javascript

Strip comments before processing javascript