Skip to content

fix compatibility with IDEA 2016.1 - #181

Open
ulitink wants to merge 3 commits into
cefolger:masterfrom
ulitink:master
Open

fix compatibility with IDEA 2016.1#181
ulitink wants to merge 3 commits into
cefolger:masterfrom
ulitink:master

Conversation

@ulitink

@ulitink ulitink commented Mar 22, 2016

Copy link
Copy Markdown

Hi Chris!
I'm from WebStorm team. Are you planning to support your plugin? We have some questions from users about it.

@cefolger

Copy link
Copy Markdown
Owner

Thanks ulitink! Were you able to run the unit tests against this? Sorry I haven't looked at this in awhile, I'll run and pull this in over the weekend.

@ulitink

ulitink commented Mar 23, 2016

Copy link
Copy Markdown
Author

No, I haven't run tests, just fixed compilation. But my changes don't bring new behavior, so tests should be ok. Thanks for the great plugin!

@sergiogarciadev

Copy link
Copy Markdown

I'm a new to Plugins Development, but I tried this Pull Request with WebStorm 2016.1.3 and it's works pretty well.

There are currently 2 failing tests, probably due problems on MockFunction

java.lang.ArrayIndexOutOfBoundsException: 0

    at com.chrisfolger.needsmoredojo.core.amd.define.NearestAMDImportLocator.getIndexOfParameter(NearestAMDImportLocator.java:123)
    at com.chrisfolger.needsmoredojo.core.amd.define.NearestAMDImportLocator.getDefineLiteral(NearestAMDImportLocator.java:97)
    at com.chrisfolger.needsmoredojo.core.amd.define.TestNearestAMDImportLocator.testDefineLiteralIsRetrievedWhenCursorIsOverParameter(TestNearestAMDImportLocator.java:113)


java.lang.ArrayIndexOutOfBoundsException: 0

    at com.chrisfolger.needsmoredojo.core.amd.define.NearestAMDImportLocator.getParameter(NearestAMDImportLocator.java:60)
    at com.chrisfolger.needsmoredojo.core.amd.define.TestNearestAMDImportLocator.testParameterIsRetrievedWhenCursorIsOverDefineLiteral(TestNearestAMDImportLocator.java:131)

@simplicitytrade

simplicitytrade commented Jun 24, 2016

Copy link
Copy Markdown

@cefolger @ulitink @Sergio-Garcia Could you provide an (installable) JAR file of "needsmoredojo 0.71" for WebStorm/PHPStorm? My attempt unfortunately failed.

@sergiogarciadev

Copy link
Copy Markdown

@simplicitytrade sure, please download from this link.
https://www.dropbox.com/s/4xarjv0c3o4sr9y/needsmoredojo-0.7.1.jar?dl=0

@SergKam

SergKam commented Jun 27, 2016

Copy link
Copy Markdown

@Sergio-Garcia
I've tried this jar(from dropbox). It doesn't work for me. It shows errors in WebStorm
NoSuchMethodError: Worker exited due to exception: com.intellij.lang.javascript.psi.JSFunctionExpression.getParameters()[Lcom/intellij/lang/javascript/psi/JSParameterItemWithSource;

WebStorm 2016.2 EAP
Build #WS-162.1024.2, built on June 21, 2016
JRE: 1.8.0_76-release-b216 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

@sergiogarciadev

Copy link
Copy Markdown

Hi, I'm using WebStorm 2016.1.3, the lastest stable version.

Maybe on 2016.2 there are new API changes, @ulitink is it the case?

@ulitink

ulitink commented Jun 28, 2016

Copy link
Copy Markdown
Author

Hi. I haven't encountered new API changes which affect the plugin, but I somehow missed one place to fix. Please check this jar.

@SergKam

SergKam commented Jun 29, 2016

Copy link
Copy Markdown

@ulitink Thank you! It's finally working again!

@mattdoe

mattdoe commented Aug 12, 2016

Copy link
Copy Markdown

@ulitink Thank you! That works for me on version 2016.2.1.

@cefolger Will this be incorporated into an update of the plugin?

@wanghanbo

Copy link
Copy Markdown

Could give me another url for new jar?

@sindilevich

sindilevich commented Dec 1, 2016

Copy link
Copy Markdown

@cefolger, @ulitink, @LawfulHacker, is it possible you republish the fixed plugin? When the fixed version 0.7.1 is going to be publicly available through the JetBrains Plugin Repository?

Thank you in advance!

@sktguha

sktguha commented May 11, 2017

Copy link
Copy Markdown

where can i get the latest working jar file for webstorm 2016.3.3 ? i installed the latest plugin from the repository but none of the functionality is working

@sktguha

sktguha commented May 11, 2017

Copy link
Copy Markdown

the dropbox links are not working. could anyone be kind to reupload the working jar file again ?
@mattdoe @ulitink @SergKam

@sindilevich

Copy link
Copy Markdown

@sktguha, I can upload it to a publicly available storage, if you have one.

@sktguha

sktguha commented May 11, 2017

Copy link
Copy Markdown

err didn't get u . can u upload it to a google drive ? or here itself ?

@sindilevich

Copy link
Copy Markdown

@sktguha, try this one.

@sktguha

sktguha commented May 11, 2017

Copy link
Copy Markdown

thanks. it is working now. I must say it is a great plugin and tons of feautures. thanks @cefolger for making this. will try to contribute if possible in future :)

@sktguha

sktguha commented May 11, 2017

Copy link
Copy Markdown

would be helpful to update the main page with link to this updated jar file to help people with latest webstorm

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.

9 participants