On Wed, Jun 3, 2009 at 12:39 PM, Thomas Mortagne
<thomas.mortagne(a)xwiki.com>wrote;wrote:
  On Wed, Jun 3, 2009 at 06:45, asiri
<platform-notifications(a)xwiki.org>
 wrote:
  Author: asiri
 Date: 2009-06-03 06:45:22 +0200 (Wed, 03 Jun 2009)
 New Revision: 20727
 Modified:
 
 platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/doc/XWikiDocument.java
  Log:
 [misc] Made setParent() method use a compact document name serializer 
 instead of
the toString() method in DocumentName class.
 Modified: 
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/doc/XWikiDocument.java
===================================================================
 --- 
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/doc/XWikiDocument.java
   2009-06-03 00:14:00 UTC (rev 20726)
  +++ 
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/doc/XWikiDocument.java
   2009-06-03 04:45:22 UTC (rev 20727)
  @@ -605,7 +605,9 @@
      */
     public void setParent(DocumentName parentName)
     {
 -        this.parent = parentName.toString();
 +        DocumentNameSerializer serializer =
 +            (DocumentNameSerializer) 
Utils.getComponent(DocumentNameSerializer.class, "compact");
 You don't need that, there is already a compactDocumentNameSerializer
 initialized in XWikiDocument constructor.
 
  +        this.parent =
serializer.serialize(parentName);
     }
     public String getFullName()
 @@ -3154,10 +3156,10 @@
     public List<String> getChildren(XWikiContext context) throws 
XWikiException
      {
 -        String[] whereParams = { this.getWikiName() + ":" + 
this.getFullName(), this.getFullName(), this.getName(),
  -            this.getSpace() };
 -
 -        String whereStatement = "doc.parent=? or doc.parent=? or 
(doc.parent=? and doc.space=?)";
  +        String[] whereParams =
 +            {this.getWikiName() + ":" + this.getFullName(), 
this.getFullName(), this.getName(), this.getSpace()};
  +
 +        String whereStatement = "doc.parent=? or doc.parent=? or 
(doc.parent=? and doc.space=?)";
          return 
context.getWiki().getStore().searchDocumentsNames(whereStatement,
 Arrays.asList(whereParams), context);
      }
 @@ -3953,10 +3955,7 @@
      * renaming algorithm takes into account the fact that there are 
 several ways
to write a link to a given page and
       * all those forms need to be renamed. For
example the following 
 links all point to the same page:
       * <ul>
 -     * <li>[Page]</li>
 -     * <li>[Page?param=1]</li>
 -     * <li>[currentwiki:Page]</li>
 -     * <li>[CurrentSpace.Page]</li>
 +     * <li>[Page]</li> <li>[Page?param=1]</li> 
<li>[currentwiki:Page]</li> <li>[CurrentSpace.Page]</li>
       * </ul>
      * <p>
      * Note: links without a space are renamed with the space added.
 _______________________________________________
 notifications mailing list
 notifications(a)xwiki.org
 
http://lists.xwiki.org/mailman/listinfo/notifications
 
 --
 Thomas Mortagne
 _______________________________________________
 notifications mailing list
 notifications(a)xwiki.org
 
http://lists.xwiki.org/mailman/listinfo/notifications