VS Code show arguments in wrong order

Neil_Burkholder · February 10, 2021, 4:16pm

I just noticed that documentation popup in vscode shows the arguments in the wrong order.

This just threw me for a loop.

Is there anyway to fix this? @thomthom

DanRathbun · February 10, 2021, 7:54pm

Looking at the API stubs repo, and at this method, it’s parameters are in the correct order.

Have you downloaded the most recently published stub files ?
…

github.com

SketchUp/ruby-api-stubs/blob/master/lib/sketchup-api-stubs/stubs/Sketchup/Model.rb#L1477-L1544


# The {#start_operation} method is used to notify SketchUp that a new
# operation (which can be undone) is starting.
#
# The +op_name+ argument is a description for the operation that is displayed
# adjacent to the Edit > Undo menu item. Make sure to provide a user friendly
# name for your operation.
#
# Starting with SketchUp 7.0, there are three additional booleans that one can
# pass in when starting an operation. It is recommended to always set
# +disable_ui+ to +true+. It's left to +false+ for default for compatibility
# reasons.
#
# @example Observer Operation since SU2016
#   class MyDefinitionsObserver < Sketchup::DefinitionObserver
#     def onComponentAdded(definitions, definition)
#       return if definition.deleted?
#       # The operation name won't be displayed when the fourth argument is
#       # +true+. It will absorb into the previous operation.
#       definition.model.start_operation('Tag It', true, false, true)
#       definition.set_attribute('MyExtension', 'Tag', 'You are it')

This file has been truncated. show original

Neil_Burkholder · February 10, 2021, 8:14pm

The stubs aren’t the problem. Actually I noticed it in one of my own methods first.

In fact pressing Ctrl while hovering shows the def preview with the parameters in correct order.

Also Ctrl+Clicking the method shows the stub is correct.

DanRathbun · February 10, 2021, 8:22pm

Well it should be that the stubs are the only way that an IDE editor knows an API’s methods and parameters.

If it cant be the stubs, then it must then be an error in the VS Code Ruby plugin. Have you checked the VS Code forums for reports of similar errors ?

Neil_Burkholder · February 10, 2021, 8:28pm

Must be a bug in either Solargraph or VSCode.

This is the only thing that turned up in my search, and I couldn’t tell if it was related.

thomthom · February 11, 2021, 11:16am

I’m not seeing that re-ordering:

This is certainly an issue with Solargraph or VSCode.

What version of the Solargraph VSCode extension and gem do you have?

Also, I’m currently trying to figure out why YRI doesn’t always generate the YRI database for the stubs. I think it might relate to using Bundler instead of gem install. Look into why stubs are not always available to Solargraph · Issue #76 · SketchUp/ruby-api-stubs (github.com)

Could be worth to try to run yard gems sketchup-api-stubs to explicitly re-generate the YRI database.

Neil_Burkholder · February 11, 2021, 1:49pm

I’m using v0.21.1 according to VS Code

If I run gem server, it reports two versions installed.

I ran that but it didn’t seem to do anything.

tt_su · February 11, 2021, 1:58pm

Latest gem for the stubs are 0.7.6. But I don’t think that will matter.

I’d recommend you open an issue with the Ruby Solargraph repo: castwide/vscode-solargraph: A Visual Studio Code extension for Solargraph. (github.com)

Neil_Burkholder · February 12, 2021, 3:39pm

Now I really botched things up. I can’t even get solargraph to run.

I uninstalled ruby, and vscode, but I noticed that all my settings persisted. I probably have some settings wrong. I have this problem of just changing a bunch of settings without actually knowing what I’m doing.

Neil_Burkholder · February 12, 2021, 4:52pm

Whew that was frustrating!! I figured out that my original problem was that I didn’t have the stubs setup right. I copied the stubs into my repository so solargraph was finding them that way.

Fortunately I finally have things right!

thomthom · February 26, 2021, 7:17pm

You no longer have the stubs copied into your project folder, right?

Neil_Burkholder · February 26, 2021, 7:21pm

That’s right, but some of my own methods still have the parameters in the wrong order. I’ve decided to live with it.

thomthom · March 1, 2021, 3:58pm

That is strange. Very strange. I’d report that to the solargraph project. The maintainer is usually pretty responsive.

Neil_Burkholder · March 1, 2021, 4:30pm

Ok we’ll see if they have any answers.

Neil_Burkholder · March 1, 2021, 5:02pm

You were definitely right about that!!

https://github.com/castwide/vscode-solargraph/issues/211

The fix will be published in gem version 0.40.4 later this week.

Neil_Burkholder · March 3, 2021, 1:18pm

Fix released in gem v0.40.4.