Get row node by id

[dephiros]

Get row node by id

[nkoterba]

@ceolter Although this issue is closed, I was trying to determine the best way to handle my use case:

I have updated row data for a specific row from the backend server. I'd like to just update the row's data and call api.refreshRows([rowToRefresh]).

However, the only way to locate the row is to call api.forEachNode and find the matching node with id. However, once I find the correct row I can't event short circuit the forEachNode with a break statement like a normal javascript array forEach.

Do you still recommend just maintaining our own map to find rows based on Ids or is there another more clever way to just find a row based on it's id?

Thanks!

[ceolter]

i assume you are talking about cellRenderer? if yes, then node is passed as a param, and the id is in the node.

another way, which i don't recommend, but you might want, is to also use api.getBusinessKeyForNode(), that will put the id you give it into the DOM as an attribute to the row. you don't have to have it the same as the node id, but you can do if you decide.

[dephiros]

@ceolter I have the node id and want to get the node outside of the cellRenderer context. I would like a more efficient way to get the node object(maybe getNodeFromId in the gridApi) without having to use forEachNode and go through all the nodes. Is it possible or could it be implemented?

[ceolter]

nope - the only collection the grid has of the nodes is in the rowModel, which just keeps the nodes in a list. if you want to index by id, then i suggest you keep a map yourself? the grid doens't need it, so it's not a grid responsibility. given you are providing the nodeId's, then you have the perfect callback getRowNodeId for populating your map.

[ceolter]

because the grid doesn't need it, it would be a wast of memory for the grid to keep such a map, given very few people (only one so far) has asked for it :)

[dephiros]

got it. Thank you for the response :). I will keep a node reference with my custom checkbox for now

[a497988]

+1

[shijusn]

Is there a way to early exit/break from forEachNode loop once we find a satisfying node?
return statement doesn't work inside forEachNode()