A415HzYeah, when I press “delete”, the modal popup does not go away even after the record is successfully deleted
I can also confirm that line
A415HzThank you very much for consideration and implementing the fix.
I will look forward to the next release.
A415HzA415Hz said:
Hello,
I don't have a test case that I can write here, but yep I was having issues.
However, I tested the latest commit from github and I can confirm that it all works, include ignoreCase issue that I was having.
This is great, and thanks for including the fix. I will look forward to the next release!
I made a new bug report post hee, as I went back and confirmed that ignoreCase still didn't work in 4.1.2:
A415HzOk here it is the test code I've prepared for 4.1.2.
If you could confirm it, I will appreciate it very much. I made it so that you can press “find” button from the search dialog window to see exactly what the search is trying to do!
The test code here is looking for data with last name either “smith” or “miller, AND
A415HzHello,
I don't have a test case that I can write here, but yep I was having issues.
However, I tested the latest commit from github and I can confirm that it all works, include ignoreCase issue that I was having.
This is great, and thanks for including the fix. I will look forward to the next release!
A415HzI can confirm that with version 4.1.1 the problem has not been addressed yet. I hope it will be addressed soon. Thank you for all the work.
A415HzHello,
I have an addition here:
The solution Jayque posted works, except a case where “ignoreCase” being set to true. I needed to add a line of code to include “ignoreCase: true”. Here is how.
“Around” line 1450 to 1475 or so, you will add the ignorCase thing.
if (group.rules !== undefined) {
if(s>0) {
var result = query.select();
query = $.jgrid.from(result);
if (ts.p.ignoreCase) { query = query.ignoreCase(); }
}
}
I hope that this minor adjustment will be included in the next release.
A415HzThank you very much, I was able to make multi group search work based on your fix.
I was having trouble doing multiple groups of “OR” that are eventually combined by “AND”.
For example “text” “contains” “whatever” AND (“inv_id” “eq” “101” OR “inv_id” “eq” “102” OR “inv_id “eq” “103”) would actually work ok in the current version, however things like:
“text” “contains” “whatever” AND (“inv_id” “eq” “101” OR “inv_id” “eq” “102” OR “inv_id “eq” “103”) AND (“user_id” “eq” “12” OR “user_id” “eq” “13”) didn't work.
So, jayque's patch code addresses this problem, and it's great.
I hope the fix will be applied in the next release.
A415HzSame here, when I try to sort the column with IgnoreCase: true and the column's sorttype is set to 'int', jqgrid hangs on “loading…”, but there is javascript error, as I think it's trying to call toLowerCase function on integers, while it's expected to have strings.
Copyright 2014 TriRand LtdAll Rights ReservedRSS
Back to Top