[jbossseam-issues] [JBoss JIRA] Reopened: (JBSEAM-3083) urlBase on <m:message> does not work

[Pete Muir (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBSEAM-3083?page=all ]

Pete Muir reopened JBSEAM-3083:
-------------------------------

             
Please revert the API change in the 2.0 branch.

Also, I don't think the API change is corrrect at all. If you are pointing at a different JSF app, then how do you expect it to process the request? The *reason* for this feature is that you need more than a relative path when you are rendering an html mail.

> urlBase on <m:message> does not work 
> -------------------------------------
>
>                 Key: JBSEAM-3083
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-3083
>             Project: Seam
>          Issue Type: Bug
>          Components: Mail
>    Affects Versions: 2.1.0.A1, 2.0.2.SP1
>            Reporter: Dan Allen
>         Assigned To: Dan Allen
>             Fix For: 2.1.0.BETA1, 2.0.3.CR1
>
>         Attachments: JBSEAM-3083-branch_20-v1.txt, JBSEAM-3083-trunk-v1.txt
>
>   Original Estimate: 15 minutes
>  Remaining Estimate: 15 minutes
>
> The attribute urlBase on <m:message> is broken in two fundamental ways. First, the logic as to whether to use the urlBase doesn't make any sense at all, resulting in the urlBase never being used. The second problem is that EL value expressions in the urlBase are not getting resolved.
> I am proposing a strategy as part of this fix. If the urlBase is used, it is taken verbatim, without appending the current context path. The reason is, if urlBase is overridden, it may be because it is pointing to a completely different application. If the developer want to reuse the context path, the expression #{facesContext.externalContext.requestContextPath} must be included in the urlBase attribute.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira