Getting ORA-01422 Error in stored procedure

Question

ORA-01422: exact fetch returns more than requested number of rows
This page explains the error:
http://www.techonthenet.com/oracle/errors/ora01422.php
This is my procedure:
The select into clause is what's causing the error. I understand the nature of the error, but the strange thing is that when i execute the sql, removing the 'into'
part with my test-parameters I just receive a single row. Still when I execute my procedure with the same parameters I get the error. What could be wrong?
create or replace
PROCEDURE COUNTER_GETACTIVE
METERPOINTID IN MeterPointCounters.MeterPointId%Type,
METERPOINTCOUNTERNR IN METERPOINTCOUNTERS.METERPOINTCOUNTERNR%type,
p_cursor out REFCURSOR_PKG.counter_cursor
IS
l_counterNr CounterBridgeTable.CounterNr%type;
l_meterId CounterBridgeTable.MeterId%type;
BEGIN
select counternr, meterid into l_counterNr, l_meterId from CounterBridgeTable where meterpointid = METERPOINTID and meterpointcounternr = METERPOINTCOUNTERNR and datetom is null;
open p_cursor for select * from counters where counternr = l_counterNr and meterId = l_meterId;
END COUNTER_GETACTIVE;

Answer 1

You are actually running this query
select counternr, meterid from CounterBridgeTable where meterpointid = METERPOINTID and meterpointcounternr = METERPOINTCOUNTERNR and datetom is null;the param name is the same as the column, it is ignoring your param and pretty much doing a self join
try this:
create or replace
PROCEDURE COUNTER_GETACTIVE
p_METERPOINTID IN MeterPointCounters.MeterPointId%Type,
p_METERPOINTCOUNTERNR IN METERPOINTCOUNTERS.METERPOINTCOUNTERNR%type,
p_cursor out REFCURSOR_PKG.counter_cursor
IS
l_counterNr CounterBridgeTable.CounterNr%type;
l_meterId CounterBridgeTable.MeterId%type;
BEGIN
select counternr, meterid into l_counterNr, l_meterId from CounterBridgeTable where meterpointid = p_METERPOINTID and meterpointcounternr = p_METERPOINTCOUNTERNR and datetom is null;
open p_cursor for select * from counters where counternr = l_counterNr and meterId = l_meterId;
END COUNTER_GETACTIVE;Edited by: tanging on Jun 8, 2010 10:37 AM