Skip to content

Commit

Permalink
Merge pull request #423 from onflow/bastian/398-fix-ls-add-missing-me…
Browse files Browse the repository at this point in the history
…mbers-quick-fix

[LS] Fix and improve add missing members quick fix
  • Loading branch information
turbolent authored Aug 31, 2024
2 parents 37db12b + 70f579d commit a62af3d
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions languageserver/server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -2549,7 +2549,7 @@ func maybeAddMissingMembersCodeActionResolver(

builder.WriteRune('\n')
builder.WriteString(indentation)
builder.WriteString(missingMember.Access.String())
builder.WriteString(missingMember.Access.QualifiedKeyword())
builder.WriteRune(' ')
builder.WriteString(newMemberSource)
builder.WriteRune('\n')
Expand Down Expand Up @@ -2614,8 +2614,14 @@ func formatNewMember(member *sema.Member, indentation string) string {

innerIndentation := strings.Repeat(" ", indentationCount)

var viewModifier string
if functionType.Purity == sema.FunctionPurityView {
viewModifier = "view "
}

return fmt.Sprintf(
"fun %s(%s)%s {\n%[4]s%[5]spanic(\"TODO\")\n%[4]s}",
"%sfun %s(%s)%s {\n%[5]s%[6]spanic(\"TODO\")\n%[5]s}",
viewModifier,
member.Identifier.Identifier,
parametersBuilder.String(),
returnType,
Expand Down

0 comments on commit a62af3d

Please sign in to comment.