topic Re: Code in BADI method in Application Development and Automation Discussions

Code in BADI method

Former Member — Fri, 09 May 2008 14:12:05 GMT

Hi Experts,

I am trying to implemet a BADI and below is the code in the implementation method.

could you please let me know what is wrong in this code?

i am getting errors like Tables cannot be used in BADIs etc....could you correct me.

Appreciate your help

METHOD xyz~t.

tables : x,y.

types: begin of l_tab1,

whno type x-whno,

cnter type x-cnter,

end of l_tab1.

types: begin of l_tab2,

whno type y-whno,

trtype type y-trtype,

end of l_tab2.

DATA: ITAB1 type table of l_tab1.

DATA: ITAB2 TYPE TABLE OF l_tab2.

SELECT x-whno, x-cnter FROM x INTO CORRESPONDING FIELDS OF TABLE itab1

WHERE x-LGTYP = z-WSTYP and z-whno = z-whno.

IF sy-subrc EQ 0.

REFRESH itab2.

SELECT whno, TRTYPe FROM y

INTO CORRESPONDING FIELDS OF TABLE itab2

FOR ALL ENTRIES IN itab1

WHERE y-cnter = itab1-cnter and y-whno = itab1-whno.

IF sy-subrc EQ 0.

z-mv = 0.

ENDIF.

ENDMETHOD.

thanks

Re: Code in BADI method

Former Member — Fri, 09 May 2008 14:40:14 GMT

Hi,

You have defined

Tables : x, y.

You cannot define the tables in the BADi.

If you are using this for workare then you can create the workarea using statement,

data : wa_x type x.

Even you can't use TYPE.

regards,

mahantesh

Edited by: Mahantesh Patil on May 9, 2008 4:41 PM

Re: Code in BADI method

Former Member — Fri, 09 May 2008 14:44:08 GMT

Hi,

Instead of writing the code in the method write the code in the function moduel by passing the variable and do what ever you need and get the expected data in the tables or iimporting parameters.

Thanks,

NN.

Re: Code in BADI method

matt — Fri, 09 May 2008 14:47:32 GMT

The only error seems to be your TABLES statement. You can just remove it and your program should work.

The TABLES statement just defines a work area. It used to be necessary, if I remember correctly, in order to select from db tables, but this is no longer the case.

matt

Re: Code in BADI method

Former Member — Fri, 09 May 2008 14:56:18 GMT

i am new to badi i don't know how to accomplish the same. Even if i remove tables statement it doesn't work.

it gives me another error. comma without preceeding colon(after select?). help me with this method.

will appreciate ur help.

thanks.

Re: Code in BADI method

Former Member — Fri, 09 May 2008 15:02:52 GMT

Hi,

SELECT x-whno, x-cnter FROM x INTO CORRESPONDING FIELDS OF TABLE itab1

WHERE x-LGTYP = z-WSTYP and z-whno = z-whno.

Please remove the comma from the above statement.

regards,

mahantesh

Re: Code in BADI method

Former Member — Fri, 09 May 2008 15:14:30 GMT

doesn't work. says unknown column name x-whno not determined until runtime cannot specify a field list.

Re: Code in BADI method

Former Member — Fri, 09 May 2008 15:39:15 GMT

Use:

SELECT whno, cnter FROM x INTO CORRESPONDING FIELDS OF TABLE itab1

WHERE LGTYP = z-WSTYP and whno = z-whno.

Re: Code in BADI method

Former Member — Fri, 09 May 2008 15:40:16 GMT

SELECT whno cnter FROM x INTO CORRESPONDING FIELDS OF TABLE itab1

WHERE LGTYP = z-WSTYP and whno = z-whno.

pasted the comma in the last post...remove comma

Re: Code in BADI method

Former Member — Fri, 09 May 2008 15:41:50 GMT

The error i got was aftet removing the comma. doesn't help.

Re: Code in BADI method

Former Member — Fri, 09 May 2008 16:38:23 GMT

Hi,

what this variable z- is indicating.

regards,

mahantesh

Re: Code in BADI method

Former Member — Fri, 09 May 2008 16:45:29 GMT

for the method Z(internal structure) is the changing parameter

Re: Code in BADI method

Former Member — Fri, 09 May 2008 19:31:40 GMT

THE BELOW code does work except for when i try to loop through the internal table it says loop should be used with into or transporation etc.... can anyone tell me how to use this. will appreciate ur help.

*----

TYPES: BEGIN OF LT_TAB1,

LGNUM TYPE X-LGNUM,

WORKSTATION TYPE X-WORKSTATION,

WRKSTTYP TYPE X-WRKSTTYP,

END OF lT_tab1.

TYPES: BEGIN OF LT_TAB2,

LGNUM TYPE Y-LGNUM,

TRTYP TYPE Y-TRTYP,

END OF lT_tab2.

data: L_TAB1 type standard table oF LT_TAB1 initial size 0.

data: L_TAB2 type standard table of LT_TAB2 initial size 0.

SELECT F_{LGNUM F}WORKSTATION F~WRKSTTYP INTO CORRESPONDING FIELDS OF TABLE L_TAB1

FROM X AS F WHERE F_{LGNUM = Z-LGNUM AND F}LGTYP = Z-WSTYP.

IF SY-SUBRC <> 0.

WRITE: / 'NO MATCHING RECORDS FOUND'.

ELSE.

WRITE:/ 'workcenter'.

loop at L_TAB1

WRITE:/ L_TAB1-LGNUM,L_TAB1-WORKSTTYP,L_TAB1-WORKSTATION.

ENDLOOP.

ENDIF.

Re: Code in BADI method

Former Member — Fri, 09 May 2008 19:36:54 GMT

You are missing a period '.' after loop at L_TAB1

loop at L_TAB1.

Re: Code in BADI method

Former Member — Fri, 09 May 2008 23:42:33 GMT

Hi Bindu,

You got already a lot of smart information. What's about you TYPES statement? When your tables: x, y. are transparent tables, your TYPES declaration has to look like:

TYPES: begin of l_tab1,
               whno LIKE x-whno,
               cnter LIKE x-cnter,
             end of l_tab1.

The same for l_tab2.

Check it out,

Heinz

Re: Code in BADI method

Former Member — Sat, 10 May 2008 05:33:22 GMT

Hello Bindu,

When you declare a table like

DATA: itab type table type_structure.

you don't have a Header-line!

You have to do the following for a proper LOOP run:

DATA: itab type table type_structure,
          wa_itab type type_structure.
...
...
LOOP at itab into wa_itab.
* working with the wa_itab fields
...
...
ENDLOOP.

Have success,

Heinz

Re: Code in BADI method

Former Member — Sat, 10 May 2008 05:37:11 GMT

Hello Bindu,

When you declare a table like

DATA: itab type table type_structure.

you don't have a Header-line!

You have to do the following for a proper LOOP run:

DATA: itab type table type_structure,
          wa_itab type type_structure.
...
...
LOOP at itab into wa_itab.
* working with the wa_itab fields
...
...
ENDLOOP.

Have success,

Heinz