incubator-clerezza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Reto Bachmann-Gmuer <reto.bachm...@trialox.org>
Subject Re: svn commit: r1021280 - /incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager/src/main/java/org/apache/clerezza/platform/usermanager/UserManagerImpl.java
Date Mon, 11 Oct 2010 17:21:12 GMT
HI Manuel

This patch effectively causes the user to require write access where
they previously (and logical) only needed read-access.

Maybe getUnsecuredMGraph should be redefined (and possibly renamed
accordingly) to return a read-only-mgraph if write right cannot be
granted.

Cheers,
reto

On Mon, Oct 11, 2010 at 10:27 AM,  <mir@apache.org> wrote:
> Author: mir
> Date: Mon Oct 11 08:27:07 2010
> New Revision: 1021280
>
> URL: http://svn.apache.org/viewvc?rev=1021280&view=rev
> Log:
> CLEREZZA-316: service depends on existence of system graph. Therefore getting the system
graph over @Reference again
>
> Modified:
>    incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager/src/main/java/org/apache/clerezza/platform/usermanager/UserManagerImpl.java
>
> Modified: incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager/src/main/java/org/apache/clerezza/platform/usermanager/UserManagerImpl.java
> URL: http://svn.apache.org/viewvc/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager/src/main/java/org/apache/clerezza/platform/usermanager/UserManagerImpl.java?rev=1021280&r1=1021279&r2=1021280&view=diff
> ==============================================================================
> --- incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager/src/main/java/org/apache/clerezza/platform/usermanager/UserManagerImpl.java
(original)
> +++ incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.usermanager/src/main/java/org/apache/clerezza/platform/usermanager/UserManagerImpl.java
Mon Oct 11 08:27:07 2010
> @@ -30,7 +30,7 @@ import java.util.Iterator;
>  import java.util.List;
>  import java.util.Set;
>  import java.util.concurrent.locks.Lock;
> -import org.apache.clerezza.platform.Constants;
> +import org.apache.clerezza.platform.config.SystemConfig;
>  import org.apache.felix.scr.annotations.Component;
>  import org.apache.felix.scr.annotations.Reference;
>  import org.apache.felix.scr.annotations.Service;
> @@ -45,6 +45,7 @@ import org.apache.clerezza.rdf.core.Reso
>  import org.apache.clerezza.rdf.core.Triple;
>  import org.apache.clerezza.rdf.core.UriRef;
>  import org.apache.clerezza.rdf.core.access.LockableMGraph;
> +import org.apache.clerezza.rdf.core.access.SecuredMGraph;
>  import org.apache.clerezza.rdf.core.access.TcManager;
>  import org.apache.clerezza.rdf.core.impl.PlainLiteralImpl;
>  import org.apache.clerezza.rdf.core.impl.SimpleMGraph;
> @@ -73,8 +74,11 @@ public class UserManagerImpl implements
>
>        private final Logger logger = LoggerFactory.getLogger(getClass());
>
> +       @Reference(target = SystemConfig.SYSTEM_GRAPH_FILTER)
> +       private MGraph securedSystemGraph;
> +
>        private LockableMGraph getSystemGraph() {
> -               return tcManager.getMGraph(Constants.SYSTEM_GRAPH_URI);
> +               return ((SecuredMGraph) securedSystemGraph).getUnsecuredMGraph();
>        }
>
>        @Override
>
>
>

Mime
View raw message