Skip to content

[change signature] support types defined in function signature #SCL-24118#717

Open
AdamJKing wants to merge 1 commit into
JetBrains:idea261.xfrom
AdamJKing:fix/scl-24118
Open

[change signature] support types defined in function signature #SCL-24118#717
AdamJKing wants to merge 1 commit into
JetBrains:idea261.xfrom
AdamJKing:fix/scl-24118

Conversation

@AdamJKing
Copy link
Copy Markdown

Fixes #SCL-24118. This issue was happening for two reasons;

  1. The search space for type paramters uses the fragment's context, which the change-signature dialog sets to the surrounding class and not the target function. This means it misses the function's type parameters.
  2. The code maps type resolution errors as Any, and forgets the previous type.

Using the function as the context instead and not setting a child (which isn't searched anyway) fixes the issue.

…4118

In the change signature dialog, update parameter / return-type fragments to use the target function as the fragment context instead of the parent class.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant