> Steve, why do say that the callback req’s the getEntry on entity? Why > couldn’t it go on the interface? > Entry transform(T fortressEntity, LdapEntry entry) throws > FortressOperationAbortedException;
The use case I originally described was that when I performed a findUsers() call via the ReviewMgr, I had no way to retrieve the additional data that came from the underlying Entry. I was therefore repeating the query using the LDAP API, then merging together the requisite fields to output a SCIM user via the REST interface. So my original request was to be able to get the underlying LDAP entry for each FortEntity I was processing. I think the most straightforward way to do this is with the method I described: Entry getEntry(); Again, this method would only need to go on the entry types that are not Fortress specific (User and Group). With respect to the transform method, I originally had: Entry transform(Entry entry) throws FortressOperationAbortedException; This forces every callback to treat the supplied data as an LDAP Entry and it occurred to me that some people might actually just want to do post-processing on the FortEntity. And since the FortEntity in question already had a getEntry() method (per my first use case), it seemed redundant to pass both to the callbacks. One question - in your method signature above, is the fortressEntity simply a "refined" view of the (mutable) entry? If not, you could end up with some real problems producing the returned LDAP Entry. Steve "And so I pretend not to hear her. And go out to get an envelope because I'm going to have a hell of a good time in the process of buying one envelope. I meet a lot of people. And, see some great looking babes. And a fire engine goes by. And I give them the thumbs up. And, and ask a woman what kind of dog that is. And, and I don't know. The moral of the story is, is we're here on Earth to fart around. And, of course, the computers will do us out of that. And, what the computer people don't realize, or they don't care, is we're dancing animals. You know, we love to move around. And, we're not supposed to dance at all anymore." - Kurt Vonnegut
