Skip to content
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

Quick fix does not show up on diagnostic hover #60

Closed
kathrynkodama opened this issue Nov 25, 2020 · 7 comments
Closed

Quick fix does not show up on diagnostic hover #60

kathrynkodama opened this issue Nov 25, 2020 · 7 comments
Labels
5 Estimated story points bug Something isn't working diagnostic Diagnostic item medium priority
Milestone

Comments

@kathrynkodama
Copy link
Contributor

kathrynkodama commented Nov 25, 2020

Quick fixes show up on hover when you first launch an Eclipse application:
image

Once a change is made, the quick fix no longer appears on the hover of the diagnostic:
image

The quickfix is still accessible by clicking the warning on the left side bar, through the problems view, or right-clicking on the diagnostic and selecting "Quickfix"
image

@kathrynkodama kathrynkodama added bug Something isn't working diagnostic Diagnostic item labels Nov 25, 2020
@kathrynkodama kathrynkodama added this to New in 0.1.1 Project Plan via automation Sep 14, 2021
@kathrynkodama
Copy link
Contributor Author

Looks to be fixed with #26

0.1.1 Project Plan automation moved this from New to Done Nov 8, 2021
@kathrynkodama
Copy link
Contributor Author

Re-opening as this seems to be occurring with the latest from main
cc: @turkeylurkey

@kathrynkodama kathrynkodama reopened this Apr 8, 2022
@kathrynkodama kathrynkodama moved this from Done to New in 0.1.1 Project Plan Jul 11, 2022
@kathrynkodama kathrynkodama moved this from New to Planned in 0.1.1 Project Plan Jul 11, 2022
@kathrynkodama kathrynkodama added 5 Estimated story points medium priority labels Aug 16, 2022
@kathrynkodama kathrynkodama added this to the EOY 2022 GA milestone Aug 24, 2022
@mattbsox
Copy link
Contributor

Here's what we're seeing at the moment on Eclipse 2022-06 with the latest from main.

The quick fix shows up initially when the file is first opened:
Screen Shot 2022-09-14 at 2 44 31 PM

After a change is made the quick fix text shows Computing...
Screen Shot 2022-09-14 at 2 45 01 PM

When the same diagnostic is re-hovered the correct quick fix is displayed:
Screen Shot 2022-09-14 at 2 45 31 PM

Note that we only saw the Computing... message on Mac. Nothing is displayed for the quick fix message on Windows.

This seems to be the behavior on Eclipse versions 2022-03 and later.

We were able to reproduce the missing quick fix behavior on Eclipse versions 2021-12 and earlier.

@bensonnw
Copy link
Contributor

On my windows machine, when quickfix is missing, there is an exception from Eclipse language server which will terminate the running process, not sure what we can do with that exception:

!MESSAGE InterruptedException
!STACK 0
java.lang.InterruptedException
	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:385)
	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2022)
	at org.eclipse.lsp4e.operations.codeactions.LSPCodeActionMarkerResolution.checkMarkerResoultion(LSPCodeActionMarkerResolution.java:170)
	at org.eclipse.lsp4e.operations.codeactions.LSPCodeActionMarkerResolution.getResolutions(LSPCodeActionMarkerResolution.java:85)
	at org.eclipse.ui.internal.ide.registry.MarkerHelpRegistry.getResolutions(MarkerHelpRegistry.java:394)
	at org.eclipse.jdt.internal.ui.text.correction.JavaCorrectionProcessor.collectMarkerProposals(JavaCorrectionProcessor.java:318)
	at org.eclipse.jdt.internal.ui.text.correction.JavaCorrectionProcessor.collectProposals(JavaCorrectionProcessor.java:277)
	at org.eclipse.jdt.internal.ui.text.java.hover.ProblemHover$ProblemInfo.getMarkerAnnotationFixes(ProblemHover.java:154)
	at org.eclipse.jdt.internal.ui.text.java.hover.ProblemHover$ProblemInfo.computeCompletionProposals(ProblemHover.java:95)
	at org.eclipse.jdt.internal.ui.text.java.hover.ProblemHover$ProblemInfo.<init>(ProblemHover.java:74)
	at org.eclipse.jdt.internal.ui.text.java.hover.ProblemHover.createAnnotationInfo(ProblemHover.java:178)
	at org.eclipse.jdt.internal.ui.text.java.hover.AbstractAnnotationHover.getHoverInfo2(AbstractAnnotationHover.java:755)
	at org.eclipse.jdt.internal.ui.text.java.hover.BestMatchHover.getHoverInfo2(BestMatchHover.java:163)
	at org.eclipse.jdt.internal.ui.text.java.hover.BestMatchHover.getHoverInfo2(BestMatchHover.java:130)
	at org.eclipse.jdt.internal.ui.text.java.hover.JavaEditorTextHoverProxy.getHoverInfo2(JavaEditorTextHoverProxy.java:89)
	at org.eclipse.jface.text.TextViewerHoverManager$1.run(TextViewerHoverManager.java:155)

@kathrynkodama kathrynkodama moved this from Backlog to Sprint Backlog in 0.1.1 Project Plan Oct 3, 2022
@kathrynkodama kathrynkodama moved this from Sprint Backlog to Backlog in 0.1.1 Project Plan Oct 12, 2022
@TrevCraw
Copy link
Contributor

@bensonnw is no longer seeing the exception reported on Sep 15
@mattbsox to attempt to recreate the issue on Mac

@mattbsox
Copy link
Contributor

mattbsox commented Nov 2, 2022

Still seeing the same behavior as before.

@kathrynkodama
Copy link
Contributor Author

Closing as with the latest "Computing" is shown when the quick-fix is still loading

0.1.1 Project Plan automation moved this from Backlog to Sprint Closed Nov 9, 2022
@kathrynkodama kathrynkodama moved this from Sprint Closed to Closed in 0.1.1 Project Plan Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 Estimated story points bug Something isn't working diagnostic Diagnostic item medium priority
Projects
Development

No branches or pull requests

4 participants