[EuiInMemoryTable] Fixed data reset on selection change.

[ashikmeerankutty]

Summary

PR #3371 introduces a new issue when a filter is set and when we try to select a row the data is reset as shown in the screenshot. This issue can be found at https://eui.elastic.co/pr_3371/.

Before

After

Checklist

• Check against all themes for compatibility in both light and dark modes
• Checked in mobile
• Checked in IE11 and Firefox
  - [ ] Props have proper autodocs
  - [ ] Added documentation examples
  - [ ] Added or updated jest tests
• Checked for breaking changes and labeled appropriately
  - [ ] Checked for accessibility including keyboard-only and screenreader modes
• A changelog entry exists and is marked appropriately

[kibanamachine]

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

[chandlerprall]

Nice catch, and thanks for opening this to fix!

[anishagg17]

is there an issue with the dataStore function since some names are empty or is it intentional?

[ashikmeerankutty]

is there an issue with the dataStore function since some names are empty or is it intentional?

I think they are intentional as there are some null and undefined values in first name as you can see in

eui/src-docs/src/views/tables/data_store.js

Lines 34 to 45 in cc6040b

	const firstNames = [
	'Very long first name that will wrap or be truncated',
	'Another very long first name which will wrap or be truncated',
	'Clinton',
	'Igor',
	undefined,
	'Drew',
	null,
	'Rashid',
	undefined,
	'John',
	];

[ashikmeerankutty]

@chandlerprall Thanks for the review. Changes committed 👍

[chandlerprall]

jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_3419/

[chandlerprall]

Changes LGTM; Pulled and tried to break this locally, couldn't get it to not work.