3 Replies Latest reply on Feb 4, 2008 5:02 AM by tom.baeyens

    Identity Service API

    pierre.vigneras

      Hi!

      Some stuff are missing in some classes of the org.jbpm.id package:

      User.getMembership()
      User.setMembership()

      and in the Session,
      method for User creation/deletion, Group creation/deletion.

      Finally, setting the membership may be centralized in the Session also :
      setMembership(User user, Group group). So, this method will take care of the two directions links management.

      Thanks for adding the required stuff.
      Regards.


        • 1. Re: Identity Service API
          tom.baeyens

          i'm ok if you add User.getMembership() and User.setMembership()

          for setMembership(User user, Group group), i would prefer something like setMembership(String userId, String groupId); Otherwise, the user will have to lookup the User and Group objects, resulting in potentially 3 transactions depending on the environment.

          • 2. Re: Identity Service API

            I agree with Tom in this one :-) I also prefer the use of Ids than the objects as parameters (this also applies to the extensions Facade, i.e the XPDL facade).

            Pierre, could you proceed with those additions in the identity module please ?

            regards,
            Miguel Valdes

            • 3. Re: Identity Service API
              tom.baeyens

              "I also prefer the use of Ids than the objects as parameters"

              thanks.

              just for the record. return values are different. they can/should be the real objects.