Send an email base on data in database

Question

Hi,
I have created an stored procedure,which will send an email on certain condition but it doesn't seen to work.
can anyone pls help.
sp :=
CREATE OR REPLACE procedure AMC.amc_renewal(amc_date date)
as
begin
for dates in (select serial_no,
ORION_SUPPLIER_CODE,
VENDOR,
AMC_COST,
START_DATE,
LAST_DATE,
EMAIL,
MAIL_DATE
from AMC_DETAIL
where TO_DATE(MAIL_DATE,'DD-MM-YYYY') = TO_DATE(SYSDATE,'DD-MM-YYYY')) loop
APEX_mail.send(p_to => DATES.EMAIL,
p_from => '[email protected]',
p_subj => 'You have an appt on '||DATES.START_DATE,
p_body => 'Dear '||DATES.SERIAL_NO ||
'you have an appt on '||DATES.LAST_DATE);
wwv_flow_mail.push_queue(
P_SMTP_HOSTNAME => '10.X.X.X',
P_SMTP_PORTNO => 'XX'
end loop;
end;
/

Answer 1

Shirish wrote:
Hi,
I have created an stored procedure,which will send an email on certain condition but it doesn't seen to work.When you have a problem always include the following information with your question:
<li>Full APEX version
<li>Full DB/version/edition/host OS
<li>Web server architecture (EPG, OHS or APEX listener/host OS)
<li>Browser(s) and version(s) used
Define "doesn't seen to work". As well as saying what doesn't happen, tell us what does:
<li>Error messages
<li>Mail queues/logs
<li>Debug traces
Always post code wrapped in <tt>\...\</tt> tags.
CREATE OR REPLACE procedure AMC.amc_renewal(amc_date date)
as
begin
for dates in (select serial_no,
ORION_SUPPLIER_CODE,
VENDOR,
AMC_COST,
START_DATE,
LAST_DATE,
EMAIL,
MAIL_DATE
from AMC_DETAIL
where TO_DATE(MAIL_DATE,'DD-MM-YYYY') = TO_DATE(SYSDATE,'DD-MM-YYYY')) loopWhat is the data type of the <tt>MAIL_DATE</tt> column? If it's DATE it doesn't need <tt>TO_DATE</tt>. <tt>SYSDATE</tt> is definitely a date and doesn't need <tt>TO_DATE</tt>.
Do not perform unnecessary type conversions. It's confusing and inefficient. What you probably mean here is:
where mail_date >= trunc(sysdate) and mail_date < trunc(sysdate) + 1 /* assuming mail_date contains time information */or
where mail_date = trunc(sysdate) /* if the time component of all mail_date values is 00:00:00 */
APEX_mail.send(p_to   => DATES.EMAIL,
p_from => '[email protected]',
p_subj  => 'You have an appt on '||DATES.START_DATE,
p_body => 'Dear '||DATES.SERIAL_NO ||
'you have an appt on '||DATES.LAST_DATE);
wwv_flow_mail.push_queue(
P_SMTP_HOSTNAME => '10.X.X.X',
P_SMTP_PORTNO => 'XX'
);Pushing the mail queue after every send is inefficient and difficult to manage. The best approach is not to include this in application code and let the scheduled background job handle it.
end loop;
end;