[jboss-cvs] jboss-portal/theme/src/main/org/jboss/portal/theme/navigation ...

[Martin H. <mho...@no...>]

  User: mholzner
  Date: 06/03/17 18:52:35

  Modified:    theme/src/main/org/jboss/portal/theme/navigation 
                        PageNavigationalState.java
  Log:
  start of the layout API (compiles and works, but is far from complete :)
  - remodeled the RenderPageCommand (introduced base command class -> MarkupCommand)
  - added new commands for RenderRegionFragment and RenderWindowFragment (need to be hooked up to a URL and Command Factory; this is the start of getting to AJAX in the portal)
  - made the page invocation obsolete (still there though)
  
  Revision  Changes    Path
  1.3       +4 -1      jboss-portal/theme/src/main/org/jboss/portal/theme/navigation/PageNavigationalState.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: PageNavigationalState.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-portal/theme/src/main/org/jboss/portal/theme/navigation/PageNavigationalState.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -b -r1.2 -r1.3
  --- PageNavigationalState.java	22 Feb 2006 15:57:27 -0000	1.2
  +++ PageNavigationalState.java	17 Mar 2006 23:52:35 -0000	1.3
  @@ -13,7 +13,7 @@
    * A state provider for all the navigational state of the windows of one page.
    *
    * @author <a href="mailto:mho...@no...">Martin Holzner</a>
  - * @version $Revision: 1.2 $
  + * @version $Revision: 1.3 $
    */
   public final class PageNavigationalState implements NavigationalState
   {
  @@ -90,6 +90,9 @@
      public boolean isWindowVisible(String windowId)
      {
         WindowContext windowCtx = (WindowContext)windowContexts.get(windowId);
  +      if(windowCtx == null){
  +         return false;
  +      }
         return windowCtx.isVisible();
      }