getNodeByKey() does not find nodes if a numeric key is passed as number instead of string

[i-terezie]

The documentation for the getNodeByKey function does not indicate that it requires the installation of options to work:

generateIds: true,
idPrefix: "ft"-,

It would be nice to add it there.

[mar10]

This should not be the case: getNodeByKey() tries to lookup by html ID (which only succeeds when generateIds is true, but works independently from the idPrefix).
However it falls back to searching all nodes for a matching node.key property.

If you think you found a bug, please give more details, and I will re-open this issue.

[i-terezie]

getNodeByKey () works fine. The only problem is the lack of direct indication in the documentation, the need to enable generateIds: true.

[i-terezie]

I realized what I had a problem. As a key, I used the ID from the database. And in one place he ended up in a string in another number. And the getNodeByKey function uses a comparison if (node.key === key) {.
When converting to a search by ID, everything is attracted to the string and works. And without it, no.
In general, not quite expected behavior. Perhaps it is worth comparing without type.

[mar10]

node.key must be a string (and is already converted to a string mostly).
I will add a cast to string to getNodeByKey(key) to make the method more forgiving.