diff --git a/source/java/org/alfresco/repo/site/SiteServiceImpl.java b/source/java/org/alfresco/repo/site/SiteServiceImpl.java
index 26ede71de2..94951039c3 100644
--- a/source/java/org/alfresco/repo/site/SiteServiceImpl.java
+++ b/source/java/org/alfresco/repo/site/SiteServiceImpl.java
@@ -150,7 +150,7 @@ public class SiteServiceImpl extends AbstractLifecycleBean implements SiteServic
     // note: caches are tenant-aware (if using EhCacheAdapter shared cache)
     
     private SimpleCache<String, Object> singletonCache; // eg. for siteHomeNodeRef
-    private final String KEY_SITEHOME_NODEREF = "key.sitehome.noderef";
+    private static final String KEY_SITEHOME_NODEREF = "key.sitehome.noderef";
     
     private SimpleCache<String, NodeRef> siteNodeRefCache; // for site shortname to nodeRef lookup
     
diff --git a/source/test-java/org/alfresco/repo/site/SiteServiceImplTest.java b/source/test-java/org/alfresco/repo/site/SiteServiceImplTest.java
index e0fb9ba23f..2daa0b1a23 100644
--- a/source/test-java/org/alfresco/repo/site/SiteServiceImplTest.java
+++ b/source/test-java/org/alfresco/repo/site/SiteServiceImplTest.java
@@ -21,6 +21,7 @@ package org.alfresco.repo.site;
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Comparator;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
@@ -1279,7 +1280,15 @@ public class SiteServiceImplTest extends BaseAlfrescoSpringTest
             //exception.printStackTrace();
         }
     }
-    
+
+    public void testDefaults()
+    {
+        assertFalse(this.siteService.isSiteAdmin(null));
+        assertTrue(this.siteService.hasCreateSitePermissions());
+        Comparator<String> comparator = siteServiceImpl.getRoleComparator();
+        assertNotNull(comparator);
+    }
+
     public void testListSiteMemberships()
     {
         String siteName1 = "testMembership1";