Re: [data] dto and isNew

2014-06-13 Thread Karl Kildén
I wrote a response to the users list but not sure it came through. It kinda belongs in this thread so here it goes. So I ran into issues with the DTO mapper api and voiced my concerns in irc. I saw this discussion and now I am trying the solution present in the current SNAPSHOT. However I have

Re: Data Module - Criteria Support

2014-06-13 Thread Radu Creanga
Hi Thomas, You're right and I see it is fixed already - great! Best, Radu On Thu, Jun 12, 2014 at 7:31 PM, Thomas Hug thomas@gmail.com wrote: Hi Radu, Thanks for paying attention! You're right, doesn't really make sense. Additionally, the result type can also be narrowed down to a

Re: [data] dto and isNew

2014-06-13 Thread Thomas Hug
Hey Karl Sorry missed your mail indeed - it's probably time I subscribe to the user mailing list too :-) You can still chain those mappers together, but I agree the casting and wiring is clunky. As options I see: - we add a public E toEntity(Dto dto) back which basically does the same as

Re: [data] dto and isNew

2014-06-13 Thread Karl Kildén
Hi and hrmmm, What if user group was changed? groupMapper.toEntity(user.getGroup(), userDto.getGroup()); This would then operate on stale data unless you fetch and send in correct group. And managing new groups is easy for me I think, I would call the method using groupMapper.toEntity(new

DeltaSpike docs help

2014-06-13 Thread Rafael Benevides
Hi Gerhard, Due to some issues opened recently ( DELTASPIKE-634, DELTASPIKE-634, DELTASPIKE-640, DELTASPIKE-641, DELTASPIKE-642 ) I'd like to know if this effort is also aligned to Red Hat effort on DeltaSpike docs help. Michelle and Sneha (copied in this email) already started to help on

Re: [data] dto and isNew

2014-06-13 Thread Thomas Hug
Hi Karl Sorry not sure if I get you right. Why would user.getGroup() be stale? As in the update case user has just been fetched by the PK, user.getGroup() should lazyload the group - right? On Fri, Jun 13, 2014 at 2:51 PM, Karl Kildén karl.kil...@gmail.com wrote: Hi and hrmmm, What if user

Re: [data] dto and isNew

2014-06-13 Thread Karl Kildén
Not sure I get myself ;) Lets walk through how I see it: 1. User foo is created and is assigned to the preexisting group Admin. It goes like this in the flow: user = new UserDTO() - *user*.setGroupDTO(selectedGroup) - save Some logic must make sure that we don't get EntityExistsException

[jira] [Commented] (DELTASPIKE-638) re-visit inconsistent handling of @BeforeHandles

2014-06-13 Thread Rafael Benevides (JIRA)
[ https://issues.apache.org/jira/browse/DELTASPIKE-638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14031078#comment-14031078 ] Rafael Benevides commented on DELTASPIKE-638: - [~lightguard] That's my

Re: DeltaSpike docs help

2014-06-13 Thread Gerhard Petracek
hi rafael, i guess thomas just opened those ticket, because there is still a lot to do. +1 for your suggestion. regards, gerhard 2014-06-13 16:42 GMT+02:00 Rafael Benevides benevi...@redhat.com: Hi Gerhard, Due to some issues opened recently ( DELTASPIKE-634, DELTASPIKE-634,

[jira] [Created] (DELTASPIKE-643) Ambiguous dependencies for type BoundRequestContext when weld context controller under WELD, Jersey, and Tomcat

2014-06-13 Thread jason zhang (JIRA)
jason zhang created DELTASPIKE-643: -- Summary: Ambiguous dependencies for type BoundRequestContext when weld context controller under WELD, Jersey, and Tomcat Key: DELTASPIKE-643 URL: