Go Premium for a chance to win a PS4. Enter to Win

x
?
Solved

Why do I get the error invalid cursor?

Posted on 2011-09-28
1
Medium Priority
?
254 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 2000 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

Concerto Cloud for Software Providers & ISVs

Can Concerto Cloud Services help you focus on evolving your application offerings, while delivering the best cloud experience to your customers? From DevOps to revenue models and customer support, the answer is yes!

Learn how Concerto can help you.

Question has a verified solution.

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

How to Create User-Defined Aggregates in Oracle Before we begin creating these things, what are user-defined aggregates?  They are a feature introduced in Oracle 9i that allows a developer to create his or her own functions like "SUM", "AVG", and…
Background In several of the companies I have worked for, I noticed that corporate reporting is off loaded from the production database and done mainly on a clone database which needs to be kept up to date daily by various means, be it a logical…
This video explains at a high level about the four available data types in Oracle and how dates can be manipulated by the user to get data into and out of the database.
Via a live example, show how to take different types of Oracle backups using RMAN.

877 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