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

Reply via email to