<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]). 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. For example, being able to set a Group to being
disabled at creation time, setting attribute values, etc. 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(). 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>