Error Handling of errors RAISED on ERROR_SYS

Hi @msurasky-price

Here you have declared your own exception. Exception block only will hit if you raise your declared exception. Since you are calling standard IFS method, only system/IFS defined exceptions may occur here.

Use WHEN OTHERS since there can be different exceptions can occur during execution 

-- In this block we perform the actual change.
DECLARE
--  PARTTYPEERROR EXCEPTION; no need to declare exception

   [SOME_OTHER_VARIABLES]
BEGIN

   FOR sale_part_to_update_rec IN (    
      [SOME_SQL_TO_PERFORM_LOOP]
   ) LOOP      
     p0_ := NULL;
     p1_ := sale_part_to_update_rec.OBJID;
     p2_ := sale_part_to_update_rec.OBJVERSION;
     client_sys.add_to_attr(name_ => 'SOURCING_OPTION', value_ => sale_part_to_update_rec.SOURCING_OPTION, attr_ => p3_);
     client_sys.add_to_attr(name_ => 'RULE_ID', value_ => '', attr_ => p3_);
     p4_ := 'DO';

     BEGIN     
            IFSAPP.SALES_PART_API.MODIFY__( p0_ , p1_ , p2_ , p3_ , p4_ );     
            COMMIT;
     EXCEPTION
        WHEN OTHERS THEN 
               [MY_ERROR_HANDLING_CODE]
     END;          
   END LOOP;
END;

Hope it helps!

Damith

It definitely helps @dsj!

I managed to implement the code that I needed with the code error handling that was necessary.

The only thing is that I didn’t want to handle all the exceptions using the same logic, that is what I initially wanted to avoid using WHEN OTHERS THEN for all the exceptions and use a WHEN [EXCEPTION_TYPE] instead. I managed to selectively use a different approach but I ended up using the SQLCODE function like this

        WHEN OTHERS THEN
             err_code := SQLCODE;
             err_msg := SUBSTR(SQLERRM, 1, 200);
             IF err_code = -20110 THEN
                 [CODE_LOGIC THAT IS SPECIFIC TO -20110]
             ELSE
                -- Simple raise Other exceptions that are NOT -20110
                RAISE;
             END IF;

Would you say this is the best practice?

Thanks!