From 6a1c88bbdee921bd6373f713cb16e69676f2cda0 Mon Sep 17 00:00:00 2001
From: Kurtas <Kurtas@users.noreply.github.com>
Date: Tue, 10 May 2016 21:01:38 +0200
Subject: [PATCH] Fix sorting for null values

Columns with some null values are not sorted correctly.
---
 src/store/TableDataStore.js | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/store/TableDataStore.js b/src/store/TableDataStore.js
index a4775fcc5..979583a4b 100644
--- a/src/store/TableDataStore.js
+++ b/src/store/TableDataStore.js
@@ -11,21 +11,19 @@ function _sort(arr, sortField, order, sortFunc, sortFuncExtraData) {
     if (sortFunc) {
       return sortFunc(a, b, order, sortField, sortFuncExtraData);
     } else {
+      const valueA = a[sortField] === null ? '' : a[sortField];
+      const valueB = b[sortField] === null ? '' : b[sortField];
       if (isDesc) {
-        if (b[sortField] === null) return false;
-        if (a[sortField] === null) return true;
-        if (typeof b[sortField] === 'string') {
-          return b[sortField].localeCompare(a[sortField]);
+        if (typeof valueB === 'string') {
+          return valueB.localeCompare(valueA);
         } else {
-          return a[sortField] > b[sortField] ? -1 : ((a[sortField] < b[sortField]) ? 1 : 0);
+          return valueA > valueB ? -1 : ((valueA < valueB) ? 1 : 0);
         }
       } else {
-        if (b[sortField] === null) return true;
-        if (a[sortField] === null) return false;
-        if (typeof a[sortField] === 'string') {
-          return a[sortField].localeCompare(b[sortField]);
+        if (typeof valueA === 'string') {
+          return valueA.localeCompare(valueB);
         } else {
-          return a[sortField] < b[sortField] ? -1 : ((a[sortField] > b[sortField]) ? 1 : 0);
+          return valueA < valueB ? -1 : ((valueA > valueB) ? 1 : 0);
         }
       }
     }