Error when creating SP

Question

trying to create this procedure, but keep receiving this error. can someone tell me why ?
Error(31,40): PLS-00103: Encountered the symbol "APPROVED" when expecting one of the following: * & = - + ; < / > at in is mod remainder not rem return returning <an exponent (**)> <> or != or ~= >= <= <> and or like LIKE2_ LIKE4_ LIKEC_ between into using || multiset bulk member SUBMULTISET_
create or replace
PROCEDURE BUILD_LL_MASTER
AS 
BEGIN
execute immediate 'delete from MASTER';
execute immediate '
    insert into MASTER (
    request_num,
    request_url,
    logged_date,
    form_status,
    requestor_b,
    requestor_s,
    requestor_el,
    open_closed
    SELECT d1.request_num,
       d1.request_url,
       d1.logged_date,
       d1.form_status,
       d1.requestor_b,
       d1.requestor_s,
       d1.requestor_e,
       (case when  UPPER(form_status)='APPROVED/CLOSED' OR UPPER(form_status)='REJECTED' THEN 'C' ELSE 'O' END) open_closed    
FROM DF1 d1';
execute immediate '
update MASTER
    set current_archived =
    case when logged_date > '16-AUG-11' THEN 'CURRENT' ELSE 'ARCHIVED' END';
commit;
EXCEPTION
when others then
rollback;
end;
--SELECT * FROM MASTER
--EXEC BUILD_MASTER

Answer 1

Yes. The reason is that you use execute immediate without any obvious need to do so.
this should work
create or replace
PROCEDURE BUILD_LL_MASTER
AS 
BEGIN
delete from MASTER;
    insert into MASTER (
    request_num,
    request_url,
    logged_date,
    form_status,
    requestor_b,
    requestor_s,
    requestor_el,
    open_closed
    SELECT d1.request_num,
       d1.request_url,
       d1.logged_date,
       d1.form_status,
       d1.requestor_b,
       d1.requestor_s,
       d1.requestor_e,
       (case when  UPPER(form_status)='APPROVED/CLOSED' OR UPPER(form_status)='REJECTED' THEN 'C' ELSE 'O' END) open_closed    
FROM DF1 d1;
update MASTER
    set current_archived =
    case when logged_date > trunc(sysdate)-2 THEN 'CURRENT' ELSE 'ARCHIVED' END;
/* commit really needed here? */
commit;
end;
/Edited by: Sven W. on Aug 19, 2011 2:44 PM
Btw. This last update part can be better done using a returning parameter with bulk.
Edited by: Sven W. on Aug 19, 2011 2:48 PM