[ 
https://issues.apache.org/jira/browse/MYFACES-3147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Andraschko resolved MYFACES-3147.
----------------------------------------
    Resolution: Fixed

Improved the message but i think we must not translate it as it's just a 
internal exception which should never be read by a enduser.

> message of ViewExpiredException
> -------------------------------
>
>                 Key: MYFACES-3147
>                 URL: https://issues.apache.org/jira/browse/MYFACES-3147
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-314
>    Affects Versions: 2.0.5
>            Reporter: Gerhard Petracek
>            Assignee: Leonardo Uribe
>            Priority: Minor
>             Fix For: 2.3.0
>
>
> #1 the message isn't internationalized
> the message is hardcoded:
> throw new ViewExpiredException("No saved view state could be found for the 
> view identifier: " + viewId, viewId)
> #2 the message doesn't make a lot of sense
> #2.1 #getMessage concatenate two strings without a blank
> #2.2 the original message text gets concatenated with the view-id and 
> #getMessage adds the view-id again to the beginning of the message.
> this results in a message which isn't very useful - e.g.:
> /page.xhtmlNo saved view state could be found for the view identifier: 
> /page.xhtml
> -> since ViewExpiredException also provides #getViewId we can remove the 
> overridden #getMessage completely.
> workaround:
> just use the exception as a marker and use a custom message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to