Restore constructor props commits

[brunnre8]

Val's PR apparently fixes the issue introduced by e31c95e in #4710

Meaning we can re-introduce the commits that were reverted after merging it.

However, I don't understand why the commit referenced above exposes the breakage that was apparently already there in the first place.
I'd appreciate it if someone could educate me / point my brain in the right direction

[xPaw]

This is probably why:

var test = { a: 1, b: 2, c: 3 };

Object.assign(test, { b: undefined, c: null });

//test = {a: 1, b: undefined, c: null}

[xPaw]

Since this is a class now, why don't we assign all of these as defaults on the class properties instead of setting them in the constructors?

[xPaw]

I think a better fix would be make all of the parameters that chan creation uses explicit parameters of the constructor, such as constructor(name, type, key, muted) (and whatever else parameters can be set on creation.

Alternatively, it could be static helpers such as createNewQuery which returns an instance of new Chan that is set to type of query.

these comment also applies to the other models

[brunnre8]

Let's go with the approach outlined by xpaw there and avoid similar issues for now if we can, or at least log them.