kbwood / icalendar Goto Github PK

extendRemove() is redundant; it does nothing more than $.extend (see source, documentation).

While the comment in your source suggests jQuery extend now ignores nulls!, I believe you made the wrong assumption about the documentation, which states:

Arguments that are null or undefined are ignored.

Note that it says arguments, not the values within objects.

Further, if I test using Chrome's console:

var a = { foo1: null, foo2: 'bar1', foo5: null };
var b = { foo1: 'bar2', foo2: null, foo3: 'bar2', foo4: null };
$.extend(a, b);

Outputs:

Object { foo1: "bar2", foo2: null, foo5: null, foo3: "bar2", foo4: null }

As you can see, nulls are preserved in every case.

Is possible to use this plugin with require.js? I've tried to load it, but I get a onScriptError.

Thanks!