rikvdkleij / intellij-haskell

IntelliJ plugin for Haskell
https://rikvdkleij.github.io/intellij-haskell/
Apache License 2.0
1.31k stars 94 forks source link

double-qualifying autocompleted identifiers in beta57? #497

Closed aryairani closed 4 years ago

aryairani commented 4 years ago

Given:

import qualified Unison.HashQualified'         as HQ'

image

See how the autocomplete is HQ'.HQ'.requalify? But it should just be HQ'.requalify as the longer name doesn't exist.

Here's a crazier example: given

import qualified Unison.Codebase.Editor.Output.BranchDiff as OBranchDiff

image

P.S. This may have been in beta55 too, I'm not sure.

rikvdkleij commented 4 years ago

Thanks for reporting!

I can not reproduce this issue. I tried several scenarios.

What are your code completion settings and at which location do you request code completion?

aryairani commented 4 years ago

Oops, where can I find my code completion settings?

As a test just now, I did

git clone --recursive https://github.com/unisonweb/unison

did "New project from existing sources..." "Haskell Tool Stack 1.9.3 (2.1.3 )"

Navigate -> File -> HandleInput.hs:206:49 and did Ctrl+Space (Code Completion -> Basic).

image

The first completion is HQ'.Unison.HashQualified'.NameOnly, which doesn't exist; it should be HQ'.NameOnly.

The import was

import qualified Unison.HashQualified'         as HQ'
rikvdkleij commented 4 years ago

Oops, where can I find my code completion settings?

If you go to settings and then search for "code completion" you will find them. For me Match case and First letter only are enabled.

aryairani commented 4 years ago

image

rikvdkleij commented 4 years ago

Eventually I could reproduce your issue and fix it.

aryairani commented 4 years ago

❤️

rikvdkleij commented 4 years ago

Should be fixed in beta59

aryairani commented 4 years ago

Thanks Rik!

rikvdkleij commented 4 years ago

@aryairani I wonder if you experience IDE freezes with beta59

aryairani commented 4 years ago

I'll keep you posted! Probably safe to close IDE freeze tickets with earlier builds, if you want?

rikvdkleij commented 4 years ago

If beta61 solves the issue we can close them all 😄