-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix. Unhandled ArrayStoreException when string template has [] operator inside #513
Bugfix. Unhandled ArrayStoreException when string template has [] operator inside #513
Conversation
### What's done: * Fixed bugs
Codecov Report
@@ Coverage Diff @@
## master #513 +/- ##
============================================
- Coverage 81.90% 81.88% -0.03%
- Complexity 1633 1635 +2
============================================
Files 78 78
Lines 4118 4123 +5
Branches 1300 1302 +2
============================================
+ Hits 3373 3376 +3
Misses 219 219
- Partials 526 528 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|| node.treeNext.elementType == CLOSING_QUOTE | ||
} else { | ||
|| node.treeNext.elementType == CLOSING_QUOTE) | ||
} else if(!isArrayAccessExpression) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you will need to check for more corner cases - this can be not the only problem here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
What's done:
This pull request closes #406