[gatein-dev] GateIn Public API Preview and call for feedback

Nick Scavelli nscavell at redhat.com
Tue Jul 24 11:39:53 EDT 2012


On 07/19/2012 12:13 AM, Nguyen Anh Kien wrote:
> The Navigation and Node API:
> To get a node, I must use: *navigation.getNode("foo", "bar", "foobar")*
> IMO, we should support get node by URI string param like 
> *navigation.getNode("foo/bar/foobar")*. Because we often has an URI of 
> node by node.getURI();.

The node.getURI() method is slightly different now and is used to 
construct the URL to the navigation node (relative). So it would render 
the site name as well as the "node uri". However I don't like the method 
name, nor am I sure it's exactly what we want.

However I do think it's good to remove any internal separators even tho 
it's very common for tree based structures to use the '/' character. You 
have the String[] value of the node by doing node.getId().getPath(). I 
might add getPath on the node object itself so it's more clear. I would 
like to remove the dependency of the "/foo/bar" notation but I'll think 
about it and if it's that important we can support it.

> Also, *removeNode* and *addNode* methods should be the same behaviour.

How do you mean ? Same return signature ? I'm not sure what the 
implications are if I return the node that was removed since the 
underlying impl is connected.

>
> For a Node, We often has cloneable tasks for node, so it should 
> support clone() method in Node api
>
>
> regards,
> =============
> Kien Nguyen
> M&E: (+84) 933.975.960 - kienna at exoplatform.com 
> <mailto:kien.nguyen at exoplatform.com>
> Skype & yahoo: nakien2a
> Portal Team - eXo Platform South East Asia (http://www.exoplatform.com 
> <http://www.exoplatform.com/>)
>


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/gatein-dev/attachments/20120724/5fc169ad/attachment.html 


More information about the gatein-dev mailing list