Sorting with nulls

[dustinsgoodman]

closes #939

[c0bra]

Dustin,

Pull requests need to include changes to the source files, not to the
compiled build files. Also unit tests are always welcome!

Thanks for the contribution!
On Jan 22, 2014 7:16 PM, "Dustin Goodman" notifications@github.com wrote:

closes #939 #939

You can merge this Pull Request by running

git pull https://github.com/sharocko/ng-grid sorting-with-nulls

Or view, comment on, or merge it at:

#940
Commit Summary

• prevents nulls from creeping back to the top of sort result
• allow fully custom sorts

File Changes

• M build/ng-grid.jshttps://github.com/Sorting with nulls #940/files#diff-0(35)

Patch Links:

• https://github.com/angular-ui/ng-grid/pull/940.patch
• https://github.com/angular-ui/ng-grid/pull/940.diff

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/940
.

[dustinsgoodman]

Oops! Not fully accustomed to the project hierarchy. I'll fix this in the morning and add tests. :)

[dustinsgoodman]

@c0bra Okay, I fixed my commits to actually only affect the source instead of the builds. I went in to write the tests and found that there weren't any predefined tests for the sortService's sortData function. I started to write them but kept hitting different roadblocks because of the set up required to get to the sorting. I don't have time this week to write the tests but I have manually tested this fix and it is now running on my production system. I'll leave it up to you guys what you do with it, but I definitely think this a fix worth implementing in 2.0.8 or 3.

[c0bra]

Here's a place to discuss sorting in 3.0: #1072

[dustinsgoodman]

@c0bra I'm going to put a comment in the new ticket regarding this issue. I think if the issue is addressed in 3.0 then this ticket can be closed.

[c0bra]

@dustinsgoodman I went ahead and merged this into 2.0.8; looked good to me! 6a6143c

[dustinsgoodman]

@c0bra Thanks man! Hopefully the fix gets carried into 3.0! :)