Hyperapp: Undefined Parameter Causes Crash

[FlorianWendelborn]

Broken Code

const { h, app } = hyperapp
/** @jsx h */

const Example = ({a}) =>
    <div>
        Item {a}
    </div>

app({ 
    model: "Hi.", 
    view: model => <div><Example/>this should always render</div>
})

TypeError: undefined is not an object (evaluating 'node.data')

Working Code

const { h, app } = hyperapp
/** @jsx h */

const Example = ({a}) =>
    <div>
        Item {a}
    </div>

app({ 
    model: "Hi.", 
    view: model => <div><Example a=""/>this should always render</div>
})

---

Again, I'm not sure if this is intentional or a bug. Might be nice to show a better error if the ENV isn't production in case this is intended.

[jorgebucaran]

@dodekeract Couldn't repro. Both work fine. Neither one render anything.

It would help if you could provide broken examples via a CodePen/Fiddle.

[jorgebucaran]

@dodekeract https://jsfiddle.net/q6Lu62L1/

[FlorianWendelborn]

Okay, then the issue is weirder than I expected. I updated the original comment with links.

[jorgebucaran]

@dodekeract Thanks! I'll look into this. 💪

[jorgebucaran]

@dodekeract Fortunately this is somewhat related to https://github.com/hyperapp/hyperapp/issues/92, so fixing this might kill two birds with one stone.

[tunnckoCore]

Btw reference https://facebook.github.io/react/docs/jsx-in-depth.html#booleans-null-and-undefined-are-ignored

[jorgebucaran]

@tunnckoCore Thanks, that's really useful. I was looking just for that a couple of days ago.

[FlorianWendelborn]

@tunnckoCore thanks for sharing, wasn't aware of the .length "bug"/issue.