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)