getPathNodes should not return the whole tree for an empty path

[plequang]

Description

Currently, the implementation of getPathNodes in DefaultTree returns the whole tree when the pathLocator argument is null, undefined or empty.

cosmoz-tree/cosmoz-default-tree.html

Lines 168 to 170 in 5768674

	if (!pathLocator) {
	return tree;
	}

I think it should rather return an empty array, as an empty pathLocator does not represent any node of the tree. Also, other function in DefaultTree expect getPathNodes to return an array.

[JaySunSyn]

I think you are right if we only consider cosmoz-tree. However an empty path locator represents the root path in the cosmoz-treenode-navigator.

=> What def should be changed is returning roots instead of treeData so the function always returns an array.

I guess we have those possibilities on an empty path:
a) we return an empty array & implement the respective changes in the navigator
b) we return the roots array
c) we return nodeObj as array

What do you think?

[JaySunSyn]

see also #6 (comment)

[JaySunSyn]

I have another one @plequang
d) if (pathLocator === '') return roots & else if (!pathLocator) return []