topic Re: problem using Bapi_Salesorder_Change in Application Development and Automation Discussions

problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 06:23:14 GMT

Hi All,

I am updating the reason for rejection code for multiple sales orders using BAPI_SALESORDER_CHANGE.

Iam using the folowing code to update the sales orders.

but it is taking a long time to process.

Please can anybody look into this and let me know how to improve the performance.

Can anybody tell me can we call this bapi outside the loop? If we can how to pass the parameters?

LOOP AT I_SELECTED_SO.

I_ITEM-ITM_NUMBER = I_SELECTED_SO-POSNR.

I_ITEM-REASON_REJ = I_SELECTED_SO-ABGRU.

APPEND I_ITEM. CLEAR I_ITEM.

I_ITEMX-ITM_NUMBER = I_SELECTED_SO-POSNR.

I_ITEMX-UPDATEFLAG = 'U'.

I_ITEMX-REASON_REJ = 'X'.

APPEND I_ITEMX. CLEAR I_ITEMX.

Save rejection reason text

I_TEXT-DOC_NUMBER = I_SELECTED_SO-VBELN.

I_TEXT-ITM_NUMBER = I_SELECTED_SO-POSNR.

I_TEXT-LANGU = SY-LANGU.

APPEND I_TEXT.

I_BAPISDH1X-UPDATEFLAG = 'U'.

CALL FUNCTION 'BAPI_SALESORDER_CHANGE'

EXPORTING

SALESDOCUMENT = I_SELECTED_SO-VBELN

ORDER_HEADER_IN = I_BAPISDH1

ORDER_HEADER_INX = I_BAPISDH1X

TABLES

RETURN = I_RETURN

ORDER_ITEM_IN = I_ITEM

ORDER_ITEM_INX = I_ITEMX

ORDER_TEXT = I_TEXT.

READ TABLE I_RETURN WITH KEY TYPE = 'S'.

IF SY-SUBRC = 0.

CALL FUNCTION 'BAPI_TRANSACTION_COMMIT'

EXPORTING

WAIT = 'X'

IMPORTING

RETURN = I_BAPIRETURN.

IF NOT I_SELECTED_SO IS INITIAL.

MOVE I_SELECTED_SO-ABGRU TO I_REJECTED_SO-ABGRU.

MODIFY I_REJECTED_SO TRANSPORTING ABGRU

WHERE VBELN = I_SELECTED_SO-VBELN

AND POSNR = I_SELECTED_SO-POSNR.

I_REJECTED_SO-MESSAGE = 'REJECTED SUCESSFULLY'.

MODIFY I_REJECTED_SO TRANSPORTING MESSAGE

WHERE VBELN = I_SELECTED_SO-VBELN

AND POSNR = I_SELECTED_SO-POSNR.

ENDIF.

ELSE.

CALL FUNCTION 'FORMAT_MESSAGE'

EXPORTING

ID = I_RETURN-ID

LANG = 'EN'

NO = I_RETURN-NUMBER

V1 = I_RETURN-MESSAGE_V1

V2 = I_RETURN-MESSAGE_V2

V3 = I_RETURN-MESSAGE_V3

V4 = I_RETURN-MESSAGE_V4

IMPORTING

MSG = V_MSGTXT

EXCEPTIONS

NOT_FOUND = 1

OTHERS = 2.

I_REJECTED_SO-MESSAGE = V_MSGTXT.

MODIFY I_REJECTED_SO TRANSPORTING MESSAGE

WHERE VBELN = I_SELECTED_SO-VBELN

AND POSNR = I_SELECTED_SO-POSNR.

ENDIF.

ENDLOOP.

ENDFORM. " UPDATE_SO

Thanks & Regards

Sudheer Kumar

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 06:26:24 GMT

Sudheer

Remove WAIT = 'X' in the BAPI Commit Call... Just do "COMMIT WORK" as there is no dependency in updates to different sales orders in this case.

Thanks

Amol Lohade

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 06:40:02 GMT

Hi Sudhir,

BAPI doesn't commit. You need to commit explicitly

using FM BAPI_TRANSACTION_COMMIT.

Regards,

Hemant

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 06:45:38 GMT

Hi Amol,

I tried this but still it is taking long time.But it is not improved the performance.

Thanks & Regards,

Sudheer Kumar

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 06:48:28 GMT

Hi Hemanth,

Thanks for immediately reply.

I used BAPI_TRANSACTION_COMMIT already,

the actual problem is performance.

Please let me know how to improve the performance.

Thanks & Regards,

Sudheer Kumar

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 06:59:10 GMT

Sudheer

Try doing commit work only once after ENDLOOP and remove BAPI Commit work from the Loop... Endloop...

Thanks

Amol Lohade

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 09:52:02 GMT

Following are my suggestions

(1) You are not refreshing / clearing the internal table, else your internal table will get added up and you

may not get desired result

Pls modify your code

LOOP AT I_SELECTED_SO.

Referesh : I_ITEM,

I_ITEMX,

I_TEXT.

CLEAR : I_BAPISDH1,

I_BAPISDH1X.

(2) I am not sure, whether you are passing right value to I_BAPISDH1.

(3) Instead of modiying the I_REJECTED table twice, i think you can do the same with single

modify statement

MODIFY I_REJECTED_SO TRANSPORTING ABGRU

MESSAGE.

(4) I think you can do more optimizing by combining all the lines of same sales order and call BAPI at once, if your requirement is not to check for every line item.

Regards

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:04:02 GMT

Hi Madhan,

Thanks for your reply.

I have declared as shown below.

DATA: I_ITEM TYPE BAPISDITM OCCURS 0 WITH HEADER LINE,

I_ITEMX TYPE BAPISDITMX OCCURS 0 WITH HEADER LINE,

I_TEXT TYPE BAPISDTEXT OCCURS 0 WITH HEADER LINE.

DATA : ST_ITEM LIKE I_ITEM,

ST_ITEMX LIKE I_ITEMX,

ST_TEXT LIKE I_TEXT.

DATA: I_BAPISDH1 LIKE BAPISDH1 OCCURS 0 WITH HEADER LINE,

I_BAPISDH1X LIKE BAPISDH1X OCCURS 0 WITH HEADER LINE,

V_MSGTXT(100) TYPE C. "Variable for message text.

DATA: BEGIN OF I_RETURN OCCURS 0.

INCLUDE STRUCTURE BAPIRET2.

DATA: END OF I_RETURN.

DATA: BEGIN OF I_BAPIRETURN OCCURS 0.

INCLUDE STRUCTURE BAPIRET2.

DATA: END OF I_BAPIRETURN.

DATA : BEGIN OF I_SELECTED_SO occurs 0 ,

VBELN LIKE VBAP-VBELN, "Sales Doc No.

POSNR LIKE VBAP-POSNR, "Sales Doc Item No.

MATNR LIKE VBAP-MATNR, "Material No.

WERKS LIKE VBAP-WERKS, "Plant

KWMENG LIKE VBAP-KWMENG, "Ordered Quantity

AUART LIKE VBAK-AUART, "Sales Document Type

ERDAT LIKE VBAK-ERDAT, "Created On

ERNAM LIKE VBAK-ERNAM, "Created By

VKORG LIKE VBAK-VKORG, "Sales Organisation

VTWEG LIKE VBAK-VTWEG, "Distribution Channel

SPART LIKE VBAK-SPART, "Division

KUNNR LIKE VBAK-KUNNR, "Sold To Party

VSTEL LIKE VBAP-VSTEL, "Shipping Point

ABGRU LIKE VBAP-ABGRU, "Reason for Rejection Code

END OF I_SELECTED_SO.

DATA : BEGIN OF I_REJECTED_SO OCCURS 0,

VBELN LIKE VBAP-VBELN, "Sales Doc No.

POSNR LIKE VBAP-POSNR, "Sales Doc Item No.

MATNR LIKE VBAP-MATNR, "Material No.

WERKS LIKE VBAP-WERKS, "Plant

KWMENG LIKE VBAP-KWMENG, "Ordered Quantity

AUART LIKE VBAK-AUART, "Sales Document Type

ERDAT LIKE VBAK-ERDAT, "Created On

ERNAM LIKE VBAK-ERNAM, "Created By

VKORG LIKE VBAK-VKORG, "Sales Organisation

VTWEG LIKE VBAK-VTWEG, "Distribution Channel

SPART LIKE VBAK-SPART, "Division

KUNNR LIKE VBAK-KUNNR, "Sold To Party

VSTEL LIKE VBAP-VSTEL, "Shipping Point

ABGRU LIKE VBAP-ABGRU, "Reason for Rejection Code

MESSAGE(100) TYPE C, "SUCCESS OR ERROR MESSAGE

END OF I_REJECTED_SO.

and passing these to the BAPI.and no other code

Please let me know what we have to pass to I_BAPISDH1

and I_BAPISDH1X.

Thanks & Regards,

Sudheer Kumar.

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:08:18 GMT

Hi Madhan,

Please let me know can we call the BAPI_SALESORDER_CHANGE outside the loop?

or we have to call it only in a loop?

I tried it but only last record is updating when i called outside the loop.

Thanks & Regards,

Sudheer Kumar.

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:11:29 GMT

No you need to call with in loop only.

Try refereshing the internal tables and see, whether it works

Regards

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:12:55 GMT

Try to clear / referesh the values

LOOP AT I_SELECTED_SO.

Referesh : I_ITEM,

I_ITEMX,

I_TEXT.

CLEAR : I_BAPISDH1,

I_BAPISDH1X

Regards

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:25:55 GMT

Hi Madhan,

I tried this but still it is updating only one last record outside the loop.

Please can you tell me how to call this bapi outside the loop.

Thanks & Regards,

Sudheer Kumar.

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:37:14 GMT

Hi Madhan,

If we call the bapi within a loop it is working fine but

it is taking a long time.

Please let me know is there any other solution to improve the performance?

Thanks & Regards,

Sudheer Kumar.

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 10:38:12 GMT

You have call BAPI inside loop only.

The below method, can improve the performance, since you are grouping all theline items of same sales order and processing to-gether.

You declare one more internal table like I_SELECTED_SO, say I_SELECTED_SO_TEMP.

I_SELECTED_SO_TEMP [ ] = I_SELECTED_SO [ ].

SORT I_SELECTED_SO_TEMP BY VBELN.

DELETE ADJACENT DUPLICATES FROM I_SELECTED_SO_TEMP COMPARING VBELN.

LOOP AT I_SELECTED_SO_TEMP.

Referesh : I_ITEM,

I_ITEMX,

I_TEXT.

CLEAR : I_BAPISDH1,

I_BAPISDH1X.

LOOP AT I_SELECTED_SO WHERE VBELN = I_SELECTED_SO_TEMP-VBELN.

I_ITEM-ITM_NUMBER = I_SELECTED_SO-POSNR.

I_ITEM-REASON_REJ = I_SELECTED_SO-ABGRU.

APPEND I_ITEM. CLEAR I_ITEM.

I_ITEMX-ITM_NUMBER = I_SELECTED_SO-POSNR.

I_ITEMX-UPDATEFLAG = 'U'.

I_ITEMX-REASON_REJ = 'X'.

APPEND I_ITEMX. CLEAR I_ITEMX.

I_TEXT-DOC_NUMBER = I_SELECTED_SO-VBELN.

I_TEXT-ITM_NUMBER = I_SELECTED_SO-POSNR.

I_TEXT-LANGU = SY-LANGU.

APPEND I_TEXT.

ENDLOOP.

I_BAPISDH1X-UPDATEFLAG = 'U'.

*****cALL YOUR BAPI HERE

If it is success here.Than

LOOP AT I_REJECTED WHERE VBELN = I_SELECTED_SO_TEMP-VBELN..

MODIFY ALL THE LINE ITEM RECORDS.

ENDLOOP.

let the remaining code be like that

ENDLOOP.

Edited by: Madhan Doraikannan on Sep 5, 2008 12:39 PM

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 11:02:57 GMT

Hi Madhan,

I have to reject the sales orders at item level.

If I selected one particular line item in the sales order only that line item should be rejected.

Thanks & Regards,

Sudheer Kumar.

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 11:17:34 GMT

Do you want to reject all the line items in the intenal table I_SELECTED_SO

Regards

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 11:31:09 GMT

Hi Madhan,

Not all line items, whichever line item is selected only it should be updated in that particular sales order. remaining line items should not be updated.

Thanks & Regards,

Sudheer Kumar.

Edited by: sudheer kumar on Sep 5, 2008 1:31 PM

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 11:36:30 GMT

How are you selecting the line items to be set for rejection code.

Regards

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 11:39:50 GMT

Create a internal table which requires the line items to be rejected for all the sales order and do the coding as metioned before.

I hope it should work

Regards

Re: problem using Bapi_Salesorder_Change

Former Member — Fri, 05 Sep 2008 11:43:37 GMT

Hi Madhan,

I am displaying the sales orders along with the line items individually for selection using REUSE_ALV_POPUP_TO_SELECT.

SELECT VBAP~VBELN

VBAP~POSNR

VBAP~MATNR

VBAP~WERKS

VBAP~KWMENG

VBAK~AUART

VBAK~ERDAT

VBAK~ERNAM

VBAK~VKORG

VBAK~VTWEG

VBAK~SPART

VBAK~KUNNR

VBAP~VSTEL

VBAP~ABGRU

INTO TABLE I_SALESORDERS

FROM VBAK

INNER JOIN VBAP

ON VBAP_{VBELN = VBAK}VBELN

WHERE VBAP~VBELN IN S_VBELN

AND VBAP~POSNR IN S_POSNR.

and I am passing the selected orders to I_SELECTED_SO.

and after selecting the sales orders i am displaying a popup for reason for rejection codes using POPUP_GET_VALUES.

Once the reason for rejection code is selected i am calling the bapi to update the sales orders.

Thanks & Regards,

Sudheer Kumar.

Edited by: sudheer kumar on Sep 5, 2008 1:44 PM