Skip to content

Component analyzer does not detect external increment/decrement operations. #12543

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.

Sign up for GitHub

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

Jump to bottom
Closed
NTaylorMullen opened this issue Jul 24, 2019 · 0 comments
Closed

Component analyzer does not detect external increment/decrement operations. #12543

NTaylorMullen opened this issue Jul 24, 2019 · 0 comments
Assignees
@NTaylorMullen
Labels
area-blazor Includes: Blazor, Razor Components bug This issue describes a behavior which is not expected - a bug. Done This issue has been fixed
Milestone
3.0.0-preview8

Comments

@NTaylorMullen
Copy link

image

In the above image there should be a warning on IncrementAmount.
@NTaylorMullen NTaylorMullen added bug This issue describes a behavior which is not expected - a bug. 1 - Ready labels Jul 24, 2019
@NTaylorMullen NTaylorMullen self-assigned this Jul 24, 2019
@NTaylorMullen NTaylorMullen mentioned this issue Jul 24, 2019
Closed
@NTaylorMullen NTaylorMullen changed the title Component analyzer does not detect private increment/decrement operations. Component analyzer does not detect external increment/decrement operations. Jul 24, 2019
NTaylorMullen pushed a commit that referenced this issue Jul 24, 2019
Add support for increment/decrement [Parameter] usage detection.
c6c1212 
- Didn't realize there was an increment or decrement operation kind.
- Added tests to cover this scenario

#12543
@NTaylorMullen NTaylorMullen added this to the 3.0.0-preview9 milestone Jul 24, 2019
@NTaylorMullen NTaylorMullen mentioned this issue Jul 24, 2019
Merged
NTaylorMullen pushed a commit that referenced this issue Jul 25, 2019
Add support for increment/decrement [Parameter] usage detection.
c76cb92 
- Didn't realize there was an increment or decrement operation kind.
- Added tests to cover this scenario

#12543
@NTaylorMullen NTaylorMullen added Done This issue has been fixed and removed Working labels Jul 25, 2019
@Pilchie Pilchie added the area-blazor Includes: Blazor, Razor Components label Jul 25, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NTaylorMullen NTaylorMullen

Labels
area-blazor Includes: Blazor, Razor Components bug This issue describes a behavior which is not expected - a bug. Done This issue has been fixed
Projects
None yet
Milestone
3.0.0-preview8
Development

No branches or pull requests
2 participants
@Pilchie @NTaylorMullen