search

openpsa / jsgrid

Fork of last jqGrid version before license change
http://openpsa.github.io/jsgrid/
Other
28 stars 12 forks source link

usage of key:true in colModel cause row selections to be wrong #96

Closed bchr02 closed 9 years ago

bchr02 commented 9 years ago

@OlegKi fixed this issue in his repo. Here is the link:

https://github.com/OlegKi/jqGrid/commit/e1913ff3204b87d334ca0c4ac89e26b3fcf609cf

Maybe he would be willing to fix it here to?

flack commented 9 years ago

Thanks for the report! We can always try to pull changes from Oleg's repo, but AFAICT he refactored the readers quite a bit, so maybe this is only a fix for a problem introduced in his refactoring. Did you try with our js file, or do you have some example code to reproduce?

bchr02 commented 9 years ago

Yes, I verified that the issue exists with your repo as well.

Here is a test site: https://www.tesllc.aero/temp/jqgrid/

bchr02 commented 9 years ago

The issue occurs only when (key: true) and (subgrids: true).

flack commented 9 years ago

Alright, I've reproduced this now, cherry-picked Oleg's commit and verified that the issue is fixed. Thanks for the report!