Hi Harsha, Please find the comments inline.
On Tue, Mar 14, 2017 at 4:32 PM, Harsha Thirimanna <hars...@wso2.com> wrote: > Hi Lanka, > > Shall we implement these two methods also in claim service side by merging > dialect and profile ? > > public Set<Claim> transformToNativeDialect(Set<Claim> otherDialectClaims, > String claimDialect, Optional<String> > profile) { > > Assume the usage is given a set of external claim URIs, the dialect URI and a profile, get a set of claims which are mapped to default claim dialect and filtered by the profile. So that only the claims defined in profile will be returned. Please correct me if this understanding is incorrect. If that is the case aren't we merging two functionalities here? Mapping from one dialect to another is a task the claim mapping service, while filtering according to the profile should be done by a service for profile. This same applies to below method too. > public Set<Claim> transformToOtherDialect(Set<Claim> nativeDialectClaims, > String dialect, Optional<String> > profile) { > > > thanks > > *Harsha Thirimanna* > *Associate Tech Lead | WSO2* > > Email: hars...@wso2.com > Mob: +94715186770 <071%20518%206770> > Blog: http://harshathirimanna.blogspot.com/ > Twitter: http://twitter.com/harshathirimann > Linked-In: linked-in: http://www.linkedin.com/pub/ > harsha-thirimanna/10/ab8/122 > <http://wso2.com/signature> > -- Pushpalanka. -- Pushpalanka Jayawardhana, B.Sc.Eng.(Hons). Senior Software Engineer, WSO2 Lanka (pvt) Ltd; wso2.com/ Mobile: +94779716248 Blog: pushpalankajaya.blogspot.com/ | LinkedIn: lk.linkedin.com/in/pushpalanka/ | Twitter: @pushpalanka
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev