nosql4idea icon indicating copy to clipboard operation
nosql4idea copied to clipboard

Published 20 hours ago • verified publisher icon dboissier

Could help with the development

Open judepereira opened this issue 9 years ago • 11 comments

Hey, I love both IDEA, and mongo. I'd love to write some code for your plugin. Do you have a list of features planned? I can fork, develop and send you a pull request then.

Thanks!

judepereira avatar Dec 12 '15 01:12 judepereira

Hi,

Great news and thanks for supporting it.

About Mongo, many features are in the backlog:

  • Complete migration to the mongo-java-driver-3.0 (some methods or classes are deprecated)
  • Implementing a lazing loading with Cursor returned by the driver.
  • an interesting requested feature
  • Show Index infos of a collection
  • Show the number of results returned by the query (there is a lot of space at the right of the toolbar)

Tell me which of them you would like to code.

Reards

dboissier avatar Dec 22 '15 13:12 dboissier

Thanks for the update, I'll start on migrating mongo to the 3.0 driver. That'll give me a good overlay of your code structure.

judepereira avatar Dec 22 '15 18:12 judepereira

I've managed to upgrade the entire core to the new driver and classes.

Somehow the JPanel classes seem to fail running their tests. For example, line 30 of RedisAuthenticationPanel throws a null pointer exception. It used to work before. Any idea why so?

judepereira avatar Dec 26 '15 15:12 judepereira

HI,

You should create a fork from the github project. So I will be able to have a look on your modifications and help you on the issue you encounter.

Regards

dboissier avatar Dec 27 '15 12:12 dboissier

Regarding the code structure, there is still some dirty area I have to refactor. If you have any suggestion, please fell free to submit ;)

dboissier avatar Dec 27 '15 12:12 dboissier

You can see my changes here: https://github.com/dboissier/nosql4idea/compare/master...judepereira:Upgrading-to

I've refactored the names of your classes MongoCollection and and MongoClient to SingleMongoCollection and SingleMongoClient respectively, as the 3.x driver has similarly named classes.

judepereira avatar Dec 27 '15 13:12 judepereira

The most weird part is that the redis swing test worked even after my changes, but then suddenly stopped working. Not sure why.

judepereira avatar Dec 27 '15 13:12 judepereira

Funny, a project rebuilt solved the issue. I'm facing only one test case issue now, will let you know if I can't update it correctly.

judepereira avatar Dec 27 '15 19:12 judepereira

@dboissier Looks like I've managed to update it all to the latest driver :) All the test cases are passing now, except for one - test displayTreeWithASimpleArray in MongoResultPanelTest. This one attempts to create a MongoResult object with an array. Does mongo ever return an array to any query?

My update to MongoResult class only accepts Document objects. In your test there, you're looking to load strings into the MongoResult object.

judepereira avatar Dec 31 '15 06:12 judepereira

@dboissier have you got a chance to look at that test case?

judepereira avatar Jan 09 '16 04:01 judepereira

Hi,

sorry for being idle these weeks (prefessional matters). Here a proposal:

  • Disable the test case by adding @Ignored annotation
  • When you sumbit the pull request, I will have a look on it.

Regards,

David

dboissier avatar Jan 11 '16 08:01 dboissier