From do-not-reply at jboss.org Mon Aug 15 18:06:37 2011
Content-Type: multipart/mixed; boundary="===============7612457323868024741=="
MIME-Version: 1.0
From: do-not-reply at jboss.org
To: gatein-commits at lists.jboss.org
Subject: [gatein-commits] gatein SVN: r7140 - in
 portal/branches/api/component/api-impl/src:
 test/java/org/gatein/portal/api/impl/portal and 1 other directory.
Date: Mon, 15 Aug 2011 18:06:37 -0400
Message-ID: <201108152206.p7FM6bKm026896@svn01.web.mwc.hst.phx2.redhat.com>

--===============7612457323868024741==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

Author: julien_viet
Date: 2011-08-15 18:06:37 -0400 (Mon, 15 Aug 2011)
New Revision: 7140

Modified:
   portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/a=
pi/impl/portal/NavigationImpl.java
   portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/a=
pi/impl/portal/SiteImpl.java
   portal/branches/api/component/api-impl/src/test/java/org/gatein/portal/a=
pi/impl/portal/NavigationTestCase.java
Log:
more work on API


Modified: portal/branches/api/component/api-impl/src/main/java/org/gatein/p=
ortal/api/impl/portal/NavigationImpl.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/=
api/impl/portal/NavigationImpl.java	2011-08-15 21:29:37 UTC (rev 7139)
+++ portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/=
api/impl/portal/NavigationImpl.java	2011-08-15 22:06:37 UTC (rev 7140)
@@ -157,7 +157,8 @@
       String pageRef =3D context.getState().getPageRef();
       if (pageRef !=3D null)
       {
-         return gateIn.get(gateIn.pageId(site.getId(), pageRef));
+         // return gateIn.get(gateIn.pageId(site.getId(), pageRef));
+         throw new UnsupportedOperationException();
       }
       else
       {
@@ -172,7 +173,16 @@
 =

    public void setTargetPage(Page.Id targetId)
    {
-      context.setState(new NodeState.Builder(context.getState()).pageRef(t=
argetId.toString()).build());
+      if (targetId !=3D null)
+      {
+         String ref =3D targetId.getSite().getType().getName() + "::" + ta=
rgetId.getSite().getName() + "::" + targetId.getName();
+         context.setState(context.getState().builder().pageRef(ref).build(=
));
+      }
+      else
+      {
+
+         context.setState(context.getState().builder().pageRef(null).build=
());
+      }
    }
 =

    public Site getSite()
@@ -250,7 +260,7 @@
 =

    public Navigation get(String key)
    {
-      return get(getIdForChild(key));
+      return context.getNode(key);
    }
 =

    public Navigation get(org.gatein.api.id.Id<Navigation> navigationId)

Modified: portal/branches/api/component/api-impl/src/main/java/org/gatein/p=
ortal/api/impl/portal/SiteImpl.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/=
api/impl/portal/SiteImpl.java	2011-08-15 21:29:37 UTC (rev 7139)
+++ portal/branches/api/component/api-impl/src/main/java/org/gatein/portal/=
api/impl/portal/SiteImpl.java	2011-08-15 22:06:37 UTC (rev 7140)
@@ -44,6 +44,7 @@
 import org.gatein.portal.api.impl.util.AdaptedIterableIdentifiableCollecti=
on;
 import org.gatein.portal.api.impl.util.GateInTypesResolver;
 =

+import java.lang.reflect.UndeclaredThrowableException;
 import java.util.List;
 =

 /** @author <a href=3D"mailto:chris.laprun(a)jboss.com">Chris Laprun</a> */
@@ -129,7 +130,7 @@
    static class PageContainer implements HierarchicalContainer<String, Pag=
e>
    {
       private final GateInImpl gateIn;
-      private final Site site;
+      private final SiteImpl site;
       private final Query<PageData> pageDataQuery;
 =

       PageContainer(GateInImpl gateIn, SiteImpl site)
@@ -149,11 +150,13 @@
          try
          {
             gateIn.begin();
-            return gateIn.getDataStorage().getPage(PageKey.create(key));
+            SiteKey sk =3D site.getSiteKey();
+            PageKey pk =3D new PageKey(sk.getTypeName(), sk.getName(), key=
);
+            return gateIn.getDataStorage().getPage(pk);
          }
          catch (Exception e)
          {
-            return null;
+            throw new UndeclaredThrowableException(e);
          }
          finally
          {

Modified: portal/branches/api/component/api-impl/src/test/java/org/gatein/p=
ortal/api/impl/portal/NavigationTestCase.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- portal/branches/api/component/api-impl/src/test/java/org/gatein/portal/=
api/impl/portal/NavigationTestCase.java	2011-08-15 21:29:37 UTC (rev 7139)
+++ portal/branches/api/component/api-impl/src/test/java/org/gatein/portal/=
api/impl/portal/NavigationTestCase.java	2011-08-15 22:06:37 UTC (rev 7140)
@@ -17,7 +17,9 @@
 import org.exoplatform.portal.pom.data.ModelDataStorage;
 import org.gatein.api.GateIn;
 import org.gatein.api.portal.Navigation;
+import org.gatein.api.portal.Page;
 import org.gatein.api.portal.Site;
+import org.gatein.api.util.HierarchicalContainer;
 import org.gatein.portal.api.impl.GateInImpl;
 =

 import java.util.Iterator;
@@ -74,6 +76,9 @@
       NavigationContext nav =3D new NavigationContext(SiteKey.portal("clas=
sic"), new NavigationState(0));
       navService.saveNavigation(nav);
       root =3D navService.loadNode(NodeModel.SELF_MODEL, nav, Scope.ALL, n=
ull);
+
+      //
+      storage.create(new org.exoplatform.portal.config.model.Page("portal"=
, "classic", "homepage").build());
    }
 =

    @Override
@@ -111,6 +116,25 @@
       Navigation home =3D i.next();
       assertSame(site, home.getSite());
       assertEquals("home", home.getName());
+      assertSame(home, navigation.get("home"));
       assertFalse(i.hasNext());
    }
+
+   public void testPage()
+   {
+      root.add(null, "home");
+      navService.saveNode(root, null);
+
+      //
+      Site site =3D gatein.getPortal(new Site.Id(Site.PORTAL, "classic"));
+      HierarchicalContainer<String,Page> pages =3D site.getPageRegistry();
+      Page homePage =3D pages.get("homepage");
+      assertNotNull(homePage);
+
+      //
+      Navigation navigation =3D site.getNavigation();
+      Navigation home =3D navigation.get("home");
+      assertNull(home.getTargetPage());
+      home.setTargetPage(homePage);
+   }
 }


--===============7612457323868024741==--