Solved

Why do I get the error invalid cursor?

Posted on 2011-09-28
1
248 Views
Last Modified: 2012-06-27
I am getting the error: invalid cursor when running the code below. Am I doing this wrong
for resupply_Cursor in (SELECT DISTINCT TO_CHAR(availability_date, 'mm/dd/yyyy') availability_date
                        FROM v_call_option_avail
                        WHERE availability_date >= to_date(v_Begin_Date, 'mm/dd/yyyy')
                        AND contract_id          = v_contract_id
                        AND call_option_type_id  = v_call_option_type_id)
  loop
  --getting the number of hours to save for each date in the cursor.
     if resupply_cursor.availability_date = v_DST_Date THEN
      hours := 23;
    elsif resupply_cursor.availability_date = v_ST_Date THEN
      hours := 25;
    else
      hours := 24;
    end if;
    
    INSERT INTO V_CALL_OPTION_AVAIL ( 
      CONTRACT_ID,
      AVAILABILITY_DATE, 
      HOUR, 
      CALL_OPTION_TYPE_ID, 
      MW, 
      TRADES_DEAL_NO, 
      PRICE, 
      ENTRY_DATE, 
      ENTRY_USER, 
      SUPERSEDED_DATE) (
    SELECT 
    NVL(a.contract_id, v_contract_id)                                                     contract_id,
    NVL(a.availability_date, to_date(resupply_cursor.availability_date,'mm/dd/yyyy'))     availability_date,
    h.hour,
    NVL(a.call_option_type_id, v_call_option_type_id)                                     call_option_type_id,
    NVL(a.mw, 0)                                                                          MW,
    NVL(a.trades_deal_no, v_Trades_Deal_No)                                               POR_ID,
    NVL(a.price, to_char(TO_NUMBER(0.00), '99D99'))                                       PRICE,
    NVL(a.entry_date, null),
    NVL(a.entry_user, v_Entry_User),
    NVL(a.superseded_date, null)
    FROM (SELECT level AS hour FROM dual CONNECT BY level <= 24
    ) h
    LEFT JOIN  v_call_option_avail a
    ON a.availability_date  >= (TRUNC(sysdate) + (h.hour - 1)/24)
    AND a.availability_date  <= (TRUNC(sysdate) + (h.hour)/24)
    AND a.call_option_type_id = v_call_option_type_id
    AND contract_id = v_contract_id);
  end loop;
  close resupply_Cursor;

Open in new window

0
Comment
Question by:bschave2
1 Comment
 
LVL 74

Accepted Solution

by:
sdstuber earned 500 total points
ID: 36718319
you don't close FOR loop cursors,  the FOR-construct itself opens and closes them


remove this line...
close resupply_Cursor;
0

Featured Post

Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

I remember the day when someone asked me to create a user for an application developement. The user should be able to create views and materialized views and, so, I used the following syntax: (CODE) This way, I guessed, I would ensure that use…
Checking the Alert Log in AWS RDS Oracle can be a pain through their user interface.  I made a script to download the Alert Log, look for errors, and email me the trace files.  In this article I'll describe what I did and share my script.
Video by: Steve
Using examples as well as descriptions, step through each of the common simple join types, explaining differences in syntax, differences in expected outputs and showing how the queries run along with the actual outputs based upon a simple set of dem…
Via a live example, show how to take different types of Oracle backups using RMAN.

713 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question