topic Re: about LOOP statement in PAI when working on a table control in Application Development and Automation Discussions

about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 03:32:48 GMT

I am working on a table control.

Here's the part of the logic.

LOOP AT gt_xxx_scr. "internal table related to the table control

MODULE modify_data.

ENDLOOP.

MODULE modify_data is used to modify or append when the user makes some changes on TC.

The TC displayed correctly at the first place.

But after gt_xxx_scr was changed but make NO change on the TC,

the logic did go into the LOOP that makes the TC displayed incorrectly.

I think that it should not go into the loop when no change is made on the TC.

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 03:59:29 GMT

Hi Ming Yu,

For a table control, if you change or not it will always go into the loop when there is some user action that occurs and that is how module pool program works. So you need to do the appending and modifying data in a proper way.

Thanks,

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 04:28:46 GMT

in module which you hae writen for midiying the TC write proper logic.

MODIFY t_tc FROM w_tc INDEX tbc-current_line [TRANSPORTING <f1> <f2> .. ].

IF sy-subrc <> 0.

APPEND w_tc TO t_tc.

ENDIF.

regards

laxman

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 05:43:35 GMT

For the first time that internal table gets the content, the system doesn't go into the LOOP logic.

And I have tried before(as far as I remembered), if you don't make any change on TC(like enter some values onto the TC), the logic will skip the LOOP.

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 05:46:55 GMT

Yes, I write this in the MODULE modify_data.

But the problem is that when internal table's content is changed, the value of 't_tc' is last internal table's value.

That makes modify and append work incorrectly.

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 05:54:37 GMT

Perhaps you need to refresh the screen. Is it for some ALV module pool program or generic one. If its ALV module pool program then we do have system field to refresh the screen.

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 05:59:50 GMT

It's just a common Module Pool program.

Which confuses me is, after 'LOOP AT gt_xxx_scr'

the work area is not the first record of current gt_xxx_scr, but a certain record of last internal table's.

Re: about LOOP statement in PAI when working on a table control

former_member202818 — Wed, 05 Nov 2014 06:11:38 GMT

Hi Ming,

The Loop at PBO for putting data into TC from internal table.

The Loop at PAI for getting data from TC into internal table.

For more ...

Regards

Re: about LOOP statement in PAI when working on a table control

former_member202818 — Wed, 05 Nov 2014 06:20:44 GMT

The loop at PAI is only loop through the records that you are currently seeing in table control.

So this is mandatory that you must specify index while modifying.

MODIFY it FROM wa INDEX tc-current_line.

IF sy-subrc <> 0.

APPEND wa TO it.

ENDIF.

Re: about LOOP statement in PAI when working on a table control

Former Member — Wed, 05 Nov 2014 06:27:06 GMT

Yes, I do specify the index.

But the point is, the internal table has already been changed and it doesn't need to go into the LOOP.

If it goes into the LOOP, and execute：

MODIFY it FROM wa INDEX tc-current_line.

IF sy-subrc <> 0.

APPEND wa TO it.

ENDIF.

This will change the content of the internal table and make display wrong after PBO.

Re: about LOOP statement in PAI when working on a table control

former_member202818 — Wed, 05 Nov 2014 06:53:03 GMT

So rather than modifying through table control you have another code for it?

Keep those piece of code after Loop... endloop.

And if you don't want this..

MODIFY it FROM wa INDEX tc-current_line.

IF sy-subrc <> 0.

APPEND wa TO it.

ENDIF.

comment them..

Re: about LOOP statement in PAI when working on a table control

Former Member — Thu, 06 Nov 2014 06:58:33 GMT

Hi, I have worked this out.

Actually, the change of the internal table should be run after LOOP.

Just like:

LOOP AT gt_xxx_scr. "internal table related to the table control

MODULE modify_data.

ENDLOOP.

MODULE get_data_to_internal_table.

The LOOP here is different from common ones, it reads the line content from the table control rather than reads from the internal table.

Anyway, thank you for your help.

Re: about LOOP statement in PAI when working on a table control

Former Member — Thu, 06 Nov 2014 07:00:06 GMT

By the way, you can either write ' get_data_to_internal_table.' in PAI or in PBO, in this case, it is triggered by a button, so I wrote this in PAI.

Re: about LOOP statement in PAI when working on a table control

former_member202818 — Thu, 06 Nov 2014 08:49:19 GMT

May i ask, why do you want a additional module for modifying internal table? What is the purpose of table control?

Normally the records seeing in table control can be manipulated from loop at PBO(before display)

or loop at PAI(after display, to catch the changes made by user).

Re: about LOOP statement in PAI when working on a table control

Former Member — Thu, 06 Nov 2014 09:17:06 GMT

Hi,

Because there is a search criteria, every time the search criteria changes, the internal table should be changed also no matter if table control is changed or not.

It's like a RESET function, not just Modify or Add rows in the table control.

Re: about LOOP statement in PAI when working on a table control

former_member202818 — Thu, 06 Nov 2014 10:32:53 GMT

replace the loop inside MODULE get_data_to_internal_table with ....

do.

read it_tab into wa_tab index sy-index.

if sy-subrc = 0.

*****code to reset

else.

exit.

endif.

enddo.

Re: about LOOP statement in PAI when working on a table control

Former Member — Fri, 07 Nov 2014 01:22:34 GMT

But editing table control is also needed, if sy-subrc = 4 in your case, you don't know the new record is from the new internal table or it is just a new row from table control entered by a user.

Re: about LOOP statement in PAI when working on a table control

Pavithra_madhu — Fri, 07 Nov 2014 03:55:35 GMT

Hi,

in screen use workarea.. then if only changes are there go inside MODULE IT_CARDLIST_MODIFY ON CHAIN-REQUEST

LOOP AT IT_CARD_LIST.

CHAIN.

FIELD WA_CARD_LIST-CB.

FIELD WA_CARD_LIST-YMTCARD.

FIELD WA_CARD_LIST-YMTNO.

FIELD WA_CARD_LIST-MATNR.

FIELD WA_CARD_LIST-MAKTX.

FIELD WA_CARD_LIST-CHARG.

MODULE IT_CARDLIST_MODIFY ON CHAIN-REQUEST.

endchain.

ENDLOOP.

In PAI

MODULE IT_CARDLIST_MODIFY INPUT.

*** if need condition add here before modify

MODIFY IT_CARD_LIST

FROM WA_CARD_LIST

INDEX IT_CARDLIST-CURRENT_LINE.

ENDMODULE.

Is this u want.

Re: about LOOP statement in PAI when working on a table control

Former Member — Fri, 07 Nov 2014 06:44:25 GMT

Hi, does it mean only if table control has been changed, then go inside the LOOP?

But it seems that it still has some problems, if you change the internal table and table control both, yet you don't want LOOP is executed.

Re: about LOOP statement in PAI when working on a table control

Pavithra_madhu — Fri, 07 Nov 2014 08:30:54 GMT

Hi,

if table control has been changed only go inside the MODULE IT_CARDLIST_MODIFY ON CHAIN-REQUEST. otherwise just go through the loop and nothing will be updated.

can you explain ur problem with coding or explain more ??