Hey Martin, I just want to apologize in advance for slowing down your attempt to single-handedly close out every JIRA issue. :-)
On 9/21/06, Mike Kienenberger <[EMAIL PROTECTED]> wrote:
Old value msg = MessageUtils.getMessage(FacesMessage.SEVERITY_ERROR, message, args); New Value: Locale locale = MessageUtils.getCurrentLocale(); String summaryText = MessageUtils.substituteParams(locale, getSummaryMessage(), args); String detailText = MessageUtils.substituteParams(locale, message, args); msg = new FacesMessage(FacesMessage.SEVERITY_ERROR, summaryText, detailText); I might be misreading either the original code or the new code, but it looks like Old Value != New value. Maybe I'm wrong, and in the case where getSummaryMessage() == null, it's the same thing, though. On 9/21/06, Martin Marinschek <[EMAIL PROTECTED]> wrote: > Yes, that will work, but only if we save the additional attribute :-/ > > You don't have a summaryMessage in there right now - I don't understand your > > "summaryMessage + detailMessage, not simply detailMessage." comment. > > regards, > > Martin > > > On 9/21/06, Mike Kienenberger <[EMAIL PROTECTED]> wrote: > > Also, message = summaryMessage + detailMessage, not simply detailMessage. > > > > At least, I'm pretty sure that's how it currently works. > > > > On 9/21/06, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > > Sorry, yes, I meant validator as well. Well, at least the property > > > setting - getting - restoreState and saveState parts are generated. So > > > where would you incorporate the check? > > > > > > Maybe we should just get rid of the detailMessage at all, and use > > > message instead. > > > > > > regards, > > > > > > Martin > > > > > > On 9/21/06, Mike Kienenberger <[EMAIL PROTECTED]> wrote: > > > > In case it's not clear, by "component" I really mean validator in this context. > > > > > > > > On 9/21/06, Mike Kienenberger <[EMAIL PROTECTED]> wrote: > > > > > On 9/21/06, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > > > > > Hmmm... Why not provide a custom Facelets-Tag for this? > > > > > > > > > > Because that's the wrong approach to fixing the problem. > > > > > > > > > > > The thing is that also the component will be generated - so we can't > > > > > > really have much custom code there, right? > > > > > > > > > > Why would the component be generated? That's where all of the > > > > > component-specific logic is. > > > > > > > > > > > > > > > > > > -- > > > > > > http://www.irian.at > > > > > > Your JSF powerhouse - > > > JSF Consulting, Development and > > > Courses in English and German > > > > > > Professional Support for Apache MyFaces > > > > > > > > -- > > http://www.irian.at > > Your JSF powerhouse - > JSF Consulting, Development and > Courses in English and German > > Professional Support for Apache MyFaces >
