can-value handling null in addition to undefined for select elements

[erikjung]

I have a use case where I need to have a default, empty value <option> within a <select> that uses can-value for live-binding.

<script id="tpl" type="text/mustache">
  <select can-value="theValue">
    <option value="">Choose</option>
    <option value="1">One</option>
    <option value="2">Two</option>
    <option value="3">Three</option>
  </select>
</script>

The data passed to the template has null values by default on the properties that are bound to the <select>s.

can.$(document.body).append(
  can.view('tpl', {
    theValue: null
  })
);

It appears that can-live explicitly checks for undefined when determining whether or not to select an <option>

https://github.com/bitovi/canjs/blob/master/view/bindings/bindings.js#L182

It would be fantastic if this logic also accommodated null, which it currently does not due to typeof null evaluating as "object".

Code Pen example: http://codepen.io/erikjung/pen/Cvekz

[matthewp]

The commit where the undefined check was added is fdd650c. I agree that we should be doing a == null check here instead.

[justinbmeyer]

"" != null