QualityCentral

Public Report

Report From: Delphi-BCB/VCL/Action Classes [ Add a report in this area ]

Report #: 68719

Status: Closed

TSendMail is useless and bugs

Project: Delphi

Build #: 17555

Version: 12.0

Submitted By: Alexandru Ciobanu

Report Type: Basic functionality failure

Date Reported: 11/8/2008 4:49:37 AM

Severity: Serious / Highly visible problem

Last Updated: 3/20/2012 2:24:39 AM

Platform: All platforms

Internal Tracking #: 266473

Resolution: Fixed (Resolution Comments)

Resolved in Build: : 14.0.3567.25216

Duplicate of: None

Voting and Rating

Overall Rating: No Ratings Yet

0.00 out of 5

Total Votes: None

Description

1. TSendMail action in ExtActns is missing "Recepients" and "Subject" properties. The ExecuteAction method will fill those to nil before calling Mapi* functions. This make this class useless.

2. The message text to be send is still ANSI:
lpszNoteText := PAnsiChar(AnsiString(Text.Text));

3. If the user cancels the mail sending window he will receive, a MessageBox will be risen - this is very bad design. A new event like: OnMailSendStatus(Boolean) should be called by this code to allow the developer to take the decision what to do with erros/success code.

Steps to Reproduce:

1. Create a new VCL Forms application
2. Put in a TActionManager.
3. A a new standard action called TSendMail

Workarounds

None

Attachment

None

Comments

None

Developer Tools

Database Tools