CVS: Tapestry/framework/src/net/sf/tapestry/pageload PageLoader.java,1.15.2.8,1.15.2.9 PageSource.java,1.13.2.6,1.13.2.7

Update of /cvsroot/tapestry/Tapestry/framework/src/net/sf/tapestry/pageload
In directory sc8-pr-cvs1:/tmp/cvs-serv9155/framework/src/net/sf/tapestry/pageload

Modified Files:
      Tag: hship-2-3
	PageLoader.java PageSource.java 
Log Message:
Automatically register components as page listeners if they implement necessary interfaces
Major rework on page property persistance
[ 653358 ] IPage.getName() == qualified name
[ 608768 ] Changes saved AFTER IPage.detach()

Index: PageLoader.java
===================================================================
RCS file: /cvsroot/tapestry/Tapestry/framework/src/net/sf/tapestry/pageload/PageLoader.java,v
retrieving revision 1.15.2.8
retrieving revision 1.15.2.9
diff -C2 -d -r1.15.2.8 -r1.15.2.9
*** PageLoader.java	17 Dec 2002 20:50:21 -0000	1.15.2.8
--- PageLoader.java	30 Dec 2002 03:04:57 -0000	1.15.2.9
***************
*** 504,513 ****
       *  Instantitates a page from its specification.
       *
       *
!      * We instantiate the page object, then set its specification,
!      * name and locale.
!      *
!      * @see IEngine
!      * @see ChangeObserver
       **/
  
--- 504,516 ----
       *  Instantitates a page from its specification.
       *
+      *  @param name the unqualified, simple, name for the page
+      *  @param namespace the namespace containing the page's specification
+      *  @param spec the page's specification
+      * 
+      *  We instantiate the page object, then set its specification,
+      *  names and locale.
       *
!      *  @see IEngine
!      *  @see ChangeObserver
       **/
  
***************
*** 527,530 ****
--- 530,534 ----
              result.setSpecification(spec);
              result.setName(name);
+             result.setPageName(namespace.constructQualifiedName(name));
              result.setLocale(_locale);
          }
***************
*** 547,551 ****
       *  The page is immediately attached to the {@link IEngine engine}.
       *
!      *  @param name the name of the page to load
       *  @param namespace from which the page is to be loaded (used
       *  when resolving components embedded by the page)
--- 551,555 ----
       *  The page is immediately attached to the {@link IEngine engine}.
       *
!      *  @param name the simple (unqualified) name of the page to load
       *  @param namespace from which the page is to be loaded (used
       *  when resolving components embedded by the page)

Index: PageSource.java
===================================================================
RCS file: /cvsroot/tapestry/Tapestry/framework/src/net/sf/tapestry/pageload/PageSource.java,v
retrieving revision 1.13.2.6
retrieving revision 1.13.2.7
diff -C2 -d -r1.13.2.6 -r1.13.2.7
*** PageSource.java	16 Dec 2002 12:56:35 -0000	1.13.2.6
--- PageSource.java	30 Dec 2002 03:04:57 -0000	1.13.2.7
***************
*** 139,143 ****
          Object[] keys;
  
!         keys = new Object[] { page.getName(), page.getLocale()};
  
          // Don't make a copy, this array is just for the MultiKey.
--- 139,143 ----
          Object[] keys;
  
!         keys = new Object[] { page.getPageName(), page.getLocale()};
  
          // Don't make a copy, this array is just for the MultiKey.
***************
*** 177,181 ****
                  result =
                      loader.loadPage(
!                         pageName,
                          _pageSpecificationResolver.getNamespace(),
                          cycle,
--- 177,181 ----
                  result =
                      loader.loadPage(
!                         _pageSpecificationResolver.getSimplePageName(),
                          _pageSpecificationResolver.getNamespace(),
                          cycle,



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf