fix(Table): reverts unnecessary fix

[atiratree]

#1520 already fixed #1371 with some other enhacements.

#1621 reverted these enhacements, so this PR reverts that PR

It also added

`console.warn(`Cell value at [ ${rowKey}, ${key} ] should not be set to ${curr}`);` 

which I do not think is necessary, because there are multiple use cases when the row is represented as structured data and we render that data according to that and do not render cells. In this use case, I think it is not necessary to show warnings (especially one per each cell).

[patternfly-build]

PatternFly-React preview: https://1623-pr-patternfly-react-patternfly.surge.sh

[codecov-io]

Codecov Report

Merging #1623 into master will increase coverage by 0.01%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #1623      +/-   ##
==========================================
+ Coverage   82.87%   82.89%   +0.01%     
==========================================
  Files         573      573              
  Lines        6255     6249       -6     
  Branches       75       75              
==========================================
- Hits         5184     5180       -4     
+ Misses       1041     1039       -2     
  Partials       30       30

Flag	Coverage Δ
#patternfly3	84.75% <ø> (ø)	⬆️
#patternfly4	79.58% <83.33%> (+0.03%)	⬆️
#patternflymisc	95.68% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
...ternfly-4/react-table/src/components/Table/Body.js	95.65% <83.33%> (+5.99%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d40f11...2835d07. Read the comment docs.

[codecov-io]

Codecov Report

Merging #1623 into master will increase coverage by 0.01%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #1623      +/-   ##
==========================================
+ Coverage   82.87%   82.89%   +0.01%     
==========================================
  Files         573      573              
  Lines        6255     6249       -6     
  Branches       75       75              
==========================================
- Hits         5184     5180       -4     
+ Misses       1041     1039       -2     
  Partials       30       30

Flag	Coverage Δ
#patternfly3	84.75% <ø> (ø)	⬆️
#patternfly4	79.58% <83.33%> (+0.03%)	⬆️
#patternflymisc	95.68% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
...ternfly-4/react-table/src/components/Table/Body.js	95.65% <83.33%> (+5.99%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d40f11...2835d07. Read the comment docs.

[dlabaj]

Please add back the handling for where null and undefined are the values for curr.

[atiratree]

Please add back the handling for where null and undefined are the values for curr.

This should be already handled (I just tested it for null/undefined)

[dlabaj]

Okay I see what you did.

[tlabaj]

LGTM