[Table] Zero 0 is not sorted correctly

[CaselIT]

Bug

When sorting over a column of numbers the number zero is sorted incorreclty
Example:

What is the expected behavior?

0 should be larger than negative numbers

What is the current behavior?

0 is smaller than negative numbers

Is there anything else we should know?

The problem is probably here
https://github.com/angular/material2/blob/c05005a688a1ff929af966835ae33ca85c106fc9/src/lib/table/table-data-source.ts#L128-L139

0 evaluates to false, while it shouldn't in this case.
I think something along the lines of

if ((valueA || valueA === 0) && (valueB || valueB === 0)) {
  // Check if one value is greater than the other; if equal, comparatorResult should remain 0.
  if (valueA > valueB) {
    comparatorResult = 1;
  } else if (valueA < valueB) {
    comparatorResult = -1;
  }
} else if (valueA || valueA === 0) {
  comparatorResult = 1;
} else if (valueB || valueB === 0) {
  comparatorResult = -1;
}

should fix this issue

[CaselIT]

In the meantime a workaround is to define a custom sortingDataAccessor with

if (value === 0) {
    return Number.EPSILON;
}

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}