Checking for truthy value of a always true token.

[sarojregmi200]

Hey there, thank you for making this library.

I was going through the source code and found out that in RenderTableBody function the data variable which is being checked using a if statement is always true.

The data variable defaults to [] which is truthy value in javascript.

Default value assignment:

rsuite-table/src/Table.tsx

Line 270 in 48ca2f0

data: dataProp = DATA_PLACEHOLDER,

Checking:

rsuite-table/src/Table.tsx

Line 971 in 48ca2f0

if (data) {

I would love to know the reason behind it.
I may be missing a whole lot of context, but I believe this can bring further problems in future updates.

[simonguo]

In actual business scenarios, it may be null to pass the data returned by the server directly

[sarojregmi200]

Got it, it's true that it may be null. While that makes complete sense.
Lets say for example if you set data to null it simply errors out and says data is null.

Image:

Example code to generate above response:

<Table 
    data={null}
    height={innerHeight}>
    <Column width={60} align="center" fixed>
        <HeaderCell>Id</HeaderCell>
        <Cell dataKey="id" />
    </Column>
</Table>

Or simply use the first example from the docs site and set data to null.