<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On a somewhat related note, we should
      probably fix the constructor for SimpleGroup also, which currently
      looks like this:<br>
      <br>
      public SimpleGroup(String id, String name, Group parentGroup)<br>
      <br>
      Having both id and parentGroup parameters is redundant, so I
      suggest removing the id parameter (and removing the id field
      altogether) and instead have the getId() method return a
      calculated id.<br>
      <br>
      <br>
      On 11/08/2012 06:08 AM, Shane Bryzak wrote:<br>
    </div>
    <blockquote cite="mid:509ABFAE.9030309@redhat.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      <div class="moz-cite-prefix">They're actually a fundamental part
        of the identity model (see [1]).&nbsp; I have no real problem with
        the principle of removing the String versions of createGroup()
        (we would also have to do the same to createRole() for
        consistency) and in fact it would provide some additional
        advantages.&nbsp; For example, being able to set a Group to being
        disabled at creation time, setting attribute values, etc.&nbsp; My
        only concern is from a coding "correctness" point of view, and I
        guess is centered around the creation date being automatically
        set (or potentially overridden) on the Group instance that's
        passed to createGroup().&nbsp; It's probably not an important concern
        though, and I'm happy to concede on this one which would mean we
        end up with the following methods (replacing all existing
        createGroup() and createRole() methods):<br>
        <br>
        void createGroup(Group group);<br>
        <br>
        void createRole(Role role);<br>
        <br>
        <br>
        [1]
        <meta http-equiv="content-type" content="text/html;
          charset=ISO-8859-1">
        <a moz-do-not-send="true"
href="https://github.com/picketlink/picketlink/blob/master/idm/api/src/main/java/org/picketlink/idm/model/IdentityType.java">https://github.com/picketlink/picketlink/blob/master/idm/api/src/main/java/org/picketlink/idm/model/IdentityType.java</a><br>
        <br>
        On 11/08/2012 05:37 AM, Jason Porter wrote:<br>
      </div>
      <blockquote
cite="mid:CAF9TksPYNLjB2TJ2KhUvOBi=_awbbZ7Akt13HLgErHWK2Y95Hg@mail.gmail.com"
        type="cite">Aren't those implementation details though?</blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>