Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Two rows selected in document list if first column is not id #16

Closed
smakela opened this issue Sep 23, 2013 · 6 comments
Closed

Two rows selected in document list if first column is not id #16

smakela opened this issue Sep 23, 2013 · 6 comments

Comments

@smakela
Copy link

smakela commented Sep 23, 2013

If you have something else as first column than id and rows have same values in that column, two of those rows with same value is selected (as show in attached picture)

screen shot 2013-09-23 at 3 13 34 pm

Here's start of the object.

{
"_class" : "OperationFormTemplate",
"_id" : ObjectId("523fe4e43004b4e685be238d"),
}

Actions:

mj edit OperationFormTemplate

So it seems that the id column position is hard coded.

This prevents lot of the functionalities (like show row in JSON)

@smakela
Copy link
Author

smakela commented Sep 23, 2013

@nikomakela
Copy link

I can confirm this issue. I faced the same bug in one of my collections.

@Kanatoko
Copy link
Owner

Thanks for reporting. I'll fix this in few days.

@Kanatoko
Copy link
Owner

1.0.16 is released. Maybe this issue is fixed.

I can not reproduce those issues, maybe because my environment is Ubuntu.

I added changes to codes, to always set the position of the "_id" columns to left.

Thanks you very much for your reports.

@nikomakela
Copy link

Seems like my issues are solved!

The issue was quite critical since I could not get documents' JSON nor use document editor to view the content of documents, because multiple documents (rows in editor) were selected at once. Now when _id is the first column in editor, it seems to work ok.

Thank you for your fast contribution and fix.

@smakela
Copy link
Author

smakela commented Sep 30, 2013

Fix seems to work!

Thanks for quick fix.

@smakela smakela closed this as completed Sep 30, 2013
Kanatoko pushed a commit that referenced this issue Sep 30, 2013
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

No branches or pull requests

3 participants