[MAGNOLIA-514] Form example Created: 03/Aug/05 Updated: 10/Aug/05 Resolved: 10/Aug/05 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | samples |
| Affects Version/s: | 2.1 Final |
| Fix Version/s: | 2.1 Final |
| Type: | Bug | Priority: | Major |
| Reporter: | Nicolas Modrzyk | Assignee: | Fabrizio Giustina |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Any |
||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
| Date of First Response: |
| Description |
|
I am using 2.1RC3 and I am highly suspicious that the example form does not work. (It does not work either on the online demo from the Magnolia site)
Also, I would be for moving the class MailHandler to a proper class in the Magnolia code, and have it log the exception when mailing failed along with successfully logging sent emails ... |
| Comments |
| Comment by Nicolas Modrzyk [ 03/Aug/05 ] |
|
following the above: MailHandler mail=new MailHandler(server,toObj.length,ccLength); throws which was hidden because of the |
| Comment by Nicolas Modrzyk [ 03/Aug/05 ] |
|
Sorry the above is wrong: The exception is not coming from the MailHandler call but: java.lang.UnsupportedOperationException: Method not yet implemented |
| Comment by Nicolas Modrzyk [ 03/Aug/05 ] |
|
Patch for the main.jsp related to the form. (redirect, classcast as explained in previous comments) And of course, the NotImplementedException needs the mail.jar and activation.jar libraries that cannot be redistributed within the LGPL license, silly me. |
| Comment by Fabrizio Giustina [ 03/Aug/05 ] |
|
committed to svn, thanks |