[Bug]: `jsx-no-leaked-render` report `&&` in prop

[AhmedBaset]

[Bug]: `jsx-no-leaked-render` report `&&` in prop

[AhmedBaset]

Yes.

Done in #3754

[ljharb]

i agree, this should be valid (altho !!firstCondition && !!anotherCondition would be more idiomatic)

[AhmedBaset]

One more thing is frustrating to me.

number && value might cause unintentional reslut but boolean && value won't.

Why the plugin requires ternary all the time even when no potential unexpected result? Could we use the TypeScrypt parser to only report the potential errors? && conditional rendering is a JS feature not something developer should avoid.

[ljharb]

@A7med3bdulBaset specifically because a 0 is falsy but will render a literal zero - that's the point of the rule, and indeed is why && conditional rendering is dangerous.

[AhmedBaset]

I understand well && and its cons/pros.

I know 0 && content will render 0. My point is why boolean && content, contentOrNullOrUndefinedOrBoolean && content is prevented?

[ljharb]

I agree that neither of those should be prevented when the type is known.

[AhmedBaset]

Thanks @akulsr0 for working on this

[AhmedBaset]

I know 0 && content will render 0. My point is why boolean && content, contentOrNullOrUndefinedOrBoolean && content is prevented?

I agree that neither of those should be prevented when the type is known.

@ljharb, should I create an issue for this?

[ljharb]

@AhmedBaset if, with the latest unreleased code, you still have an issue, please do file one.

[AhmedBaset]

<Comp prop={cond1 && cond2} /> was solved. But, dataOrUndefined && data... still an issue. IMO the rule should only complain if the left side is of type number.

[ljharb]

More specifically, it should only complain if the left side is a known type and is not a number.

This should also be an option, I'd think, because TS isn't actually "type safety" and folks may not want to rely on it.

[AhmedBaset]

More specifically, it should only complain if the left side is a known type and is not a number.

This should also be an option, I'd think, because TS isn't actually "type safety" and folks may not want to rely on it.

hmm. then, worth be an issue?

[ljharb]

Yes.