Breaks conditional extends by removing parenthesis

[johanjisu]

Expected Behavior

This should work after obfuscation:

export default Target => class Events extends (Target || Base) {

}

Current Behavior

Obfuscation removes () around Target || Base, which breaks it.

(the code above is used for mixins, Base is an existing class)

Your Environment

ES6

[sanex3339]

Hi, i'll look into this issue but currently i take a 1-2 months timeout with this project to doing renovations on my flat. So, until october i'll have no time to look into this.

If i'll have time - i'll fix it.

[johanjisu]

We currently work around it by regexing the parentheses back in place after obfuscation, so we are ok anyway. But would be nice to have it fixed when you have the time. Good luck with the renovation and thanks for a great utility!

[asifshfq]

We currently work around it by regexing the parentheses back in place after obfuscation, so we are ok anyway. But would be nice to have it fixed when you have the time. Good luck with the renovation and thanks for a great utility!

Can you please elaborate this, because I am also facing the same issue.

[sanex3339]

Hi. This is escodegen problem. Will try to create a PR to the escodegen

[sanex3339]

I created the PR with the fix for escodegen
It is better to ping this PR, because maintainers are not so active:
estools/escodegen#404

[sanex3339]

Fixed in 0.23.2 release