Fixed the code since it does not make any sense when node id was not passed as argument - it suppose to calculate the id by taking shared node - ID 0 is always reserved and used as "Empty/Invalid".
That method is currently used in two places in both cases we pass node Id so this change doesn't really improve anything but will prevent bugs if someone try to use it without explicitly passing the node id.
Fixed the code since it does not make any sense when node id was not passed as argument - it suppose to calculate the id by taking shared node - ID 0 is always reserved and used as "Empty/Invalid".
That method is currently used in two places in both cases we pass node Id so this change doesn't really improve anything but will prevent bugs if someone try to use it without explicitly passing the node id.