Hi,

Currently Sling uses the String "default" for the default workspace
name, in several places.

According to [1] however, "if workspaceName is null, a default
workspace is automatically selected by the repository implementation".

This might make Sling more interoperable with various JCR
repositories, where the default workspace is not necessarily named
"default".

My suggestion:

1) Use null for the workspace name in Repository.login() calls, if the
configured workspace name is an empty string.

2) Set the default workspace name to that empty string

3) Change the "workspace" configuration variable description to
indicate that an empty string means "use default workspace supplied by
the JCR Repository".

WDYT?

-Bertrand

[1] 
http://www.day.com/maven/jsr170/javadocs/jcr-1.0/javax/jcr/Repository.html#login(javax.jcr.Credentials,%20java.lang.String)

Reply via email to