asudoh/dojox_mvc — comment on commit 5150504c50b1b08c8196eb2192e29a5fb10b379f
File: Bind.js line 152
By neonstalwart:

oh ok, i overlooked that this was code that existed before this commit.

it seems to me that for initialization, if the direction indicates that the view is observing the model, then the value from the model should be unconditionally copied to the view even if the value is `null` or `undefined` (ie the model holds the master copy of the initial value). for the case where the view is not observing the model but the model *is* observing the view then i would think that the value from the view should be unconditionally copied to the model (ie the view holds the master copy of the initial value).

i can't trace this code back past when `bindTwo` was introduced. is there some other code that existed before `bindTwo` that this is trying to match? maybe it would help us figure out if there is a reason for ignoring `null` or `undefined` values. i could believe that something like this might break some widgets or something like that so i'd like to figure out if there is a reason but if we can't find one then i think it might be better not to ignore `null` and `undefined`.