<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <div class="moz-cite-prefix">On 07/19/2012 12:13 AM, Nguyen Anh Kien
      wrote:<br>
    </div>
    <blockquote
cite="mid:CAJwzOwBrKkiba67r7909fdi-q26TjE1-UwT+EGRP=r3O4RxykA@mail.gmail.com"
      type="cite">The Navigation and Node API:
      <div>To get a node, I must use: <b>navigation.getNode("foo",
          "bar", "foobar")</b></div>
      <div>IMO, we should support get node by URI string param like <b>navigation.getNode("foo/bar/foobar")</b>.
        Because we often has an URI of node by node.getURI();. <br>
      </div>
    </blockquote>
    <br>
    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.<br>
    <br>
    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.<br>
    <br>
    <blockquote
cite="mid:CAJwzOwBrKkiba67r7909fdi-q26TjE1-UwT+EGRP=r3O4RxykA@mail.gmail.com"
      type="cite">
      <div>Also, <b>removeNode</b> and <b>addNode</b> methods should be
        the same behaviour.</div>
    </blockquote>
    <br>
    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.<br>
    <br>
    <blockquote
cite="mid:CAJwzOwBrKkiba67r7909fdi-q26TjE1-UwT+EGRP=r3O4RxykA@mail.gmail.com"
      type="cite">
      <div><br>
      </div>
      <div>For a Node, We often has cloneable tasks for node, so it
        should support clone() method in Node api<br clear="all">
        <div><br>
        </div>
         <br>
        <div>regards,</div>
        =============<br>
        Kien Nguyen<br>
        M&amp;E: (+84) 933.975.960 - <a moz-do-not-send="true"
          href="mailto:kien.nguyen@exoplatform.com" target="_blank">kienna@exoplatform.com</a><br>
        Skype &amp; yahoo: nakien2a<br>
        Portal Team - eXo Platform South East Asia (<a
          moz-do-not-send="true" href="http://www.exoplatform.com/"
          target="_blank">http://www.exoplatform.com</a>)<br>
        <br>
      </div>
    </blockquote>
    <br>
    <br>
  </body>
</html>