You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by "Shawn McKinney (JIRA)" <ji...@apache.org> on 2016/10/06 21:41:21 UTC

[jira] [Resolved] (FC-187) pass the tenantid

     [ https://issues.apache.org/jira/browse/FC-187?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Shawn McKinney resolved FC-187.
-------------------------------
    Resolution: Fixed

> pass the tenantid
> -----------------
>
>                 Key: FC-187
>                 URL: https://issues.apache.org/jira/browse/FC-187
>             Project: FORTRESS
>          Issue Type: Sub-task
>    Affects Versions: 1.0.1
>            Reporter: Shawn McKinney
>            Assignee: Shawn McKinney
>             Fix For: 2.0.0-RC1
>
>
> There were a couple of places where the tenantid wasn't being passed down to the DAO.  
> Mostly centered around this method below.  Notice how the tenantid isn't being passed into the api call.  
> And there is another problem in this new method creates a cycle with the reviewmgrimpl as it is dependent on vutil and with this new method vice versa.
> While I'm not crazy about the solution, I have broken the code from this vutil and moved into the validate methods of both the permp and userp classes.
> public final class VUtil implements ConstraintValidator
> ...
>     public static void permAttrSetName( String value ) throws ValidationException
>     {    	    	
>     	try{
>     		ReviewMgr reviewMgr = ReviewMgrFactory.createInstance();      		
>     		PermissionAttributeSet paSet = reviewMgr.readPermAttributeSet(new PermissionAttributeSet(value));
>     	}
>         catch(Exception e){
>             String error = "permissionAttributeSet - not found with name [" + value + "]";
>             throw new ValidationException( GlobalErrIds.PERM_ATTRIBUTE_SET_NOT_FOUND, error );
>         }
>         RegExUtil.getInstance().safeText( value );
>     }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)