Why is the second procedure in my package not working??

Question

Hi I have a package that I want to run every 15 minutes. When I try to run the procedures manually and each one seperately they work. But when I put together these procedures in a package, the second procedure does not work or not updating the table. Any idea why this is creating a problem??
PROCEDURE UPDATE_CS_ACTIVITY is the one in the below package that does not seems working .. but when i run the procedure manually it works fine.
Any help in this regard is greatly appreciated...
Thanx in advance
Here is the Package code:
CREATE OR REPLACE PACKAGE BODY UPDATE_ACTIVITY AS
CSREWORK NUMBER(4);
CSTRACKOUT NUMBER(4);
STARTDATE VARCHAR2(20);
ENDDATE VARCHAR2(20);
PROCEDURE INSERTVALUES IS
BEGIN
DELETE FROM 2DAY_ACTIVITY;
IF SUBSTR(TO_CHAR(SYSDATE,'MM/DD/YYYY HH24:MI:SS'),12,2) > 18
THEN
STARTDATE := '18:30:00';
INSERT INTO 2DAY_ACTIVITY(STAGESTEPPROD,RESOURCES,CSTRACKOUT)
SELECT "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES",COUNT(TRACKOUTTIME)
FROM "ACT_1DAY_VIEW" "ACT_VIEW"
GROUP BY "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES" WHERE "ACT_VIEW"."TRACKOUTTIME" > SUBSTR(TO_CHAR(SYSDATE,'MM/DD/YYYY HH24:MI:SS'),1,10)||' '||STARTDATE AND ISREWORKSTEP = 'F';
INSERT INTO WAFER_2DAY_ACTIVITY(STAGESTEPPROD,RESOURCES,CSREWORK)
SELECT "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES",COUNT(TRACKOUTTIME)
FROM "ACT_1DAY_VIEW" "ACT_VIEW"
GROUP BY "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES" WHERE "ACT_VIEW"."TRACKOUTTIME" > SUBSTR(TO_CHAR(SYSDATE,'MM/DD/YYYY HH24:MI:SS'),1,10)||' '||STARTDATE AND ISREWORKSTEP = 'T';
COMMIT;
ELSE
     STARTDATE := '06:30:00';
     INSERT INTO 2DAY_ACTIVITY(STAGESTEPPROD,RESOURCES,CSTRACKOUT)
SELECT "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES",COUNT(TRACKOUTTIME)
FROM "ACT_1DAY_VIEW" "ACT_VIEW"
GROUP BY "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES" WHERE "ACT_VIEW"."TRACKOUTTIME" > SUBSTR(TO_CHAR(SYSDATE,'MM/DD/YYYY HH24:MI:SS'),1,10)||' '||STARTDATE AND ISREWORKSTEP = 'F';
INSERT INTO 2DAY_ACTIVITY(STAGESTEPPROD,RESOURCES,CSREWORK)
SELECT "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES",COUNT(TRACKOUTTIME)
FROM "ACT_1DAY_VIEW" "ACT_VIEW"
GROUP BY "ACT_VIEW"."STAGESTEPPROD","ACT_VIEW"."RESOURCES" WHERE "ACT_VIEW"."TRACKOUTTIME" > SUBSTR(TO_CHAR(SYSDATE,'MM/DD/YYYY HH24:MI:SS'),1,10)||' '||STARTDATE AND ISREWORKSTEP = 'T';
     COMMIT;
END IF;
END INSERTVALUES;
PROCEDURE UPDATE_CS_ACTIVITY AS
BEGIN
INSERT INTO CS_ACTIVITY(STAGESTEPPROD, RESOURCES, CSTRACKOUT, CSREWORK) select MAX(stagestepprod),MAX(resources), max(cstrackout) ,max(csrework) from 2day_activity group by 2DAY_ACTIVITY.stagestepprod,2DAY_ACTIVITY.resources;
COMMIT;
END UPDATE_CS_ACTIVITY;
END UPDATE_ACTIVITY;

Answer 1

You have create global variables CSREWORK and CSTRACKOUT
in your package.
Your have then selected the
max(cstrackout) ,max(csrework) which would probably pick these variables up.
Rename your variables to v_CSREWORK and v_CSTRACKOUT or remove altogether as no sign they are being used which should fix that problem. The normal naming standard for oracle is to name variables with a v_ and parameters with a p_ to prevent problems with naming.