I created a Jira issue https://issues.apache.org/jira/browse/FC-197
Would the new search method go on the ReviewMgr? I think the generic method is good for now, long term I really think we should really consider adding a query builder pattern so that all new search combinations don't require new API methods. ----- Original Message ----- From: "Shawn McKinney" <[email protected]> To: [email protected] Sent: Thursday, October 13, 2016 11:07:07 AM Subject: Re: Fortress Properties > On Oct 13, 2016, at 9:56 AM, Chris Pike <[email protected]> wrote: > > I think that will work, but I don't see a way to search by user properties, > so I still think we need to decide how to expose searching through the API. Yes to exposing a new search apis for prop names, values (see prev comments) As for how to search by prop name, something like this…. String searchVal = encodeSafeText( user.getUserId(), GlobalIds.USERID_LEN ); filterbuf.append( GlobalIds.FILTER_PREFIX ); filterbuf.append( Config.getInstance().getProperty( USER_OBJECT_CLASS ) ); filterbuf.append( ")(" ); filterbuf.append( GlobalIds.PROPS ); filterbuf.append( "=" ); filterbuf.append( searchVal ); filterbuf.append( “mypropname” + “*” + ))" );
