[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_71428
]
Carlos Sanchez commented on CONTINUUM-771:
--
Applied patches but the last one that doesn't apply correctly. Also the
solution goes through using maven-user as s
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_70500
]
Carlos Sanchez commented on CONTINUUM-771:
--
Applied again for missing files.
Other comments:
Year of copyright must be {year created}-{year last modified}, so f
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_70386
]
Carlos Sanchez commented on CONTINUUM-771:
--
Applied the patch in the branch, please look to CONTINUUM-783 for improvements
on link construction
> Add user man
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69985
]
Jesse McConnell commented on CONTINUUM-771:
---
henry, check out CONTINUUM-781
based on some feedback from trygve I moved that base class to the
plexus-xwork-in
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69800
]
Emmanuel Venisse commented on CONTINUUM-771:
It isn't CONTINUUM-559 but CONTINUUM-759.
I'm happy with that but with some minor comments described in CONTINUUM-759
> Ad
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69791
]
Jesse McConnell commented on CONTINUUM-771:
---
fyi, the abstract class and test cases are a part of a patch I made on
CONTINUUM-559, if emmanuel is happy with that we can a
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69701
]
Emmanuel Venisse commented on CONTINUUM-771:
Please, don't use LogFactory but plexus logger instead. Plexus logger is our
standard for all maven app.
In action class,
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69695
]
Carlos Sanchez commented on CONTINUUM-771:
--
The handling of exceptions needs to be improved
+catch (ContinuumException ce )
+{
+addActionError(
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69689
]
Henry S. Isidro commented on CONTINUUM-771:
---
Yup, I just figured it out with your reply to Teodoro above; will submit a new
patch later.
> Add user management screens
>
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69683
]
Emmanuel Venisse commented on CONTINUUM-771:
Henry,
your patch on xwork.xml isn't correct. I don't want to see full classname in
class property. It must be a short str
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69682
]
Emmanuel Venisse commented on CONTINUUM-771:
1. you must add Continuum class as a requirements of your action in
plexus-request.xml. We don't use annotations for the mo
[ http://jira.codehaus.org/browse/CONTINUUM-771?page=comments#action_69678
]
Teodoro Cue Jr. commented on CONTINUUM-771:
---
I've studied this issue and a few questions pop up.
1. How to inject the class Continuum into the action? I've seen a lot of
12 matches
Mail list logo
