andrewz wrote
> Looking at the source code of "DomainPermission.setTargets", I see a very
> strange lines of code:
> 
>    protected void setTargets(Set
> <String>
>  targets) {
>         this.targets = targets;
>         if (this.targets != null && this.targets.equals(targets)) {
>             return;
>         }
>         this.targets = targets;
>         setParts(domain, actions, targets);
>     } 
> 
> 
> Note, that there're two similar lines:
> 
>         this.targets = targets;
> 
> The first one of those shouldn't be there because it makes "equals"
> condition in next line to be always "true", which in turn prevents
> "setParts" method from being invoked and thus updating "parts" collection.
> 
> Is this a known issue?

I was just bit by this today.  I notice there haven't been any replies.  Any
chance this will be fixed soon?



--
View this message in context: 
http://shiro-user.582556.n2.nabble.com/DomainPermission-setTargets-bug-tp7579366p7579509.html
Sent from the Shiro User mailing list archive at Nabble.com.

Reply via email to