fix: improve to not treat as a valid option when the value is null. (#…

…564)

Co-authored-by: ​ <​>

src/TreeSelect.tsx

@@ -390,7 +390,7 @@ const TreeSelect = React.forwardRef<BaseSelectRef, TreeSelectProps>((props, ref)
 
  // =========================== Values ===========================
  const rawMixedLabeledValues = React.useMemo(
- () => toLabeledValues(internalValue),
+ () => toLabeledValues(internalValue === null ? [] : internalValue),
  [toLabeledValues, internalValue],
  );
 

tests/Select.multiple.spec.js

@@ -310,4 +310,42 @@ describe('TreeSelect.multiple', () => {
  ).map(ele => ele.textContent);
  expect(values).toEqual(['child1', 'child2', 'parent']);
  });
+
+ // https://github.com/ant-design/ant-design/issues/50578#issuecomment-2312130715
+ it('should not omit value when value is null', () => {
+ const { container } = render(
+ <TreeSelect
+ value={null}
+ multiple
+ placeholder="Fake placeholder"
+ treeData={[
+ {
+ label: 'parent',
+ value: 'parent',
+ children: [
+ {
+ label: 'child1',
+ value: 'child1',
+ },
+ {
+ label: 'child2',
+ value: 'child2',
+ },
+ ],
+ },
+ ]}
+ />,
+ );
+
+ const values = Array.from(
+ container.querySelectorAll('.rc-tree-select-selection-item-content'),
+ ); //.map(ele => ele.textContent);
+
+ expect(values).toHaveLength(0);
+
+ const placeholder = container.querySelector('[class$=placeholder]');
+ expect(placeholder).toBeTruthy();
+ expect(placeholder.textContent).toBe('Fake placeholder');
+ });
+
 });