From 7d62731277c7f8fe81398c3ff6dd02f73122f74b Mon Sep 17 00:00:00 2001
From: Dave Ward <left@alfresco.com>
Date: Mon, 6 Apr 2009 11:46:17 +0000
Subject: [PATCH] Fixed NullPointerException on navigation to company home -
 NavigationBean was trying to access the CIFSServer directly. Now goes through
 the ServerConfigurationAccessor interface exported by the fileServers
 subsystem.

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@13847 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../org/alfresco/web/bean/NavigationBean.java | 24 +++++++++----------
 source/web/WEB-INF/faces-config-beans.xml     |  4 ++--
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/source/java/org/alfresco/web/bean/NavigationBean.java b/source/java/org/alfresco/web/bean/NavigationBean.java
index a094b00959..585c3b64e7 100644
--- a/source/java/org/alfresco/web/bean/NavigationBean.java
+++ b/source/java/org/alfresco/web/bean/NavigationBean.java
@@ -36,9 +36,9 @@ import javax.faces.context.FacesContext;
 import javax.faces.event.ActionEvent;
 
 import org.alfresco.error.AlfrescoRuntimeException;
-import org.alfresco.filesys.CIFSServerBean;
 import org.alfresco.filesys.repo.ContentContext;
 import org.alfresco.filesys.repo.ContentDiskInterface;
+import org.alfresco.jlan.server.config.ServerConfigurationAccessor;
 import org.alfresco.jlan.server.core.SharedDevice;
 import org.alfresco.jlan.server.core.SharedDeviceList;
 import org.alfresco.jlan.server.filesys.DiskSharedDevice;
@@ -168,18 +168,18 @@ public class NavigationBean implements Serializable
    }
 
    /**
-    * @param cifsServer The cifsServer to set.
+    * @param serverConfiguration The serverConfiguration to set.
     */
-   public void setCifsServer(CIFSServerBean cifsServer)
+   public void setServerConfiguration(ServerConfigurationAccessor serverConfiguration)
    {
-      this.cifsServer = cifsServer;
+      this.serverConfiguration = serverConfiguration;
    }
    
-   protected CIFSServerBean getCifsServer()
+   protected ServerConfigurationAccessor getServerConfiguration()
    {
-      if (cifsServer == null)
-         this.cifsServer = (CIFSServerBean) FacesHelper.getManagedBean(FacesContext.getCurrentInstance(), "cifsServer");
-      return cifsServer;
+      if (serverConfiguration == null)
+         this.serverConfiguration = (ServerConfigurationAccessor) FacesHelper.getManagedBean(FacesContext.getCurrentInstance(), "fileServerConfiguration");
+      return serverConfiguration;
    }
 
    /**
@@ -699,7 +699,7 @@ public class NavigationBean implements Serializable
          Path path = node.getNodePath();
          
          // resolve CIFS network folder location for this node
-         FilesystemsConfigSection filesysConfig = (FilesystemsConfigSection)getCifsServer().getConfiguration().getConfigSection(FilesystemsConfigSection.SectionName); 
+         FilesystemsConfigSection filesysConfig = (FilesystemsConfigSection)getServerConfiguration().getConfigSection(FilesystemsConfigSection.SectionName); 
          DiskSharedDevice diskShare = null;
          
          SharedDeviceList shares = filesysConfig.getShares();
@@ -963,7 +963,7 @@ public class NavigationBean implements Serializable
       {
          StringBuilder buf = new StringBuilder(32);
          
-         String serverName = this.getCifsServer().getConfiguration().getServerName();
+         String serverName = this.getServerConfiguration().getServerName();
          if (serverName != null && serverName.length() != 0)
          {
             buf.append("\\\\");
@@ -1134,8 +1134,8 @@ public class NavigationBean implements Serializable
    /** RuleService bean reference*/
    transient private RuleService ruleService;
    
-   /** CIFSServer bean reference */
-   transient private CIFSServerBean cifsServer;
+   /** File server configuration reference */
+   transient private ServerConfigurationAccessor serverConfiguration;
    
    /** CIFS content disk driver bean reference */
    protected ContentDiskInterface contentDiskDriver;
diff --git a/source/web/WEB-INF/faces-config-beans.xml b/source/web/WEB-INF/faces-config-beans.xml
index bf1f9fce49..1dbdcc7368 100644
--- a/source/web/WEB-INF/faces-config-beans.xml
+++ b/source/web/WEB-INF/faces-config-beans.xml
@@ -280,8 +280,8 @@
          <value>#{NamespaceService}</value>
       </managed-property>
       <managed-property>
-         <property-name>cifsServer</property-name>
-         <value>#{cifsServer}</value>
+         <property-name>serverConfiguration</property-name>
+         <value>#{fileServerConfiguration}</value>
       </managed-property>
       <managed-property>
          <property-name>ruleService</property-name>