topic Re: Screen painter in Application Development and Automation Discussions

Screen painter

Former Member — Wed, 31 May 2006 09:07:14 GMT

Dear Friends,

I have designed a screen for inputting the data. The codes written in PAI is given below. The problem I am facing is that, the data entered in the screen is not getting inserted into the table as well as it is not getting exit, when the exit button is pressed.

I have created two buttons Save and Exit.

Please let me know what is the error.

--------------------------------

*& Module USER_COMMAND_1000 INPUT

--------------------------------

MODULE USER_COMMAND_1000 INPUT.

TABLES : ZTDR.

DATA OK_CODE LIKE SY-UCOMM.

CASE OK_CODE.

WHEN 'Save'.

INSERT ZTDR.

WHEN 'Exit'.

LEAVE PROGRAM.

ENDCASE.

ENDMODULE. " USER_COMMAND_1000 INPUT

--------------------------------

Thanks in advance.

MARK

Re: Screen painter

Former Member — Wed, 31 May 2006 09:11:49 GMT

Hi,

Instead of LEAVE PROGRAM, write LEAVE TO SCREEN 0. or CALL TRANSACTION SE38.

For saving the work, give the statement COMMIT WORK in the end. Also make sure that the function code of yuor buttons are given correctly in your code. Check if they are 'Save' and 'Exit' only.

Regards,

Aswin

Re: Screen painter

Former Member — Wed, 31 May 2006 09:14:37 GMT

check the name of button ..

WHEN 'Save'.

INSERT ZTDR.

WHEN 'Exit'.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:15:03 GMT

Hi,

small correction to your code..

MODULE USER_COMMAND_1000 INPUT.
TABLES : ZTDR.
DATA OK_CODE LIKE SY-UCOMM.
CASE OK_CODE.
<b>WHEN 'SAVE'.</b>
INSERT ZTDR.
<b>WHEN 'EXIT'.</b>
LEAVE PROGRAM.
ENDCASE.
ENDMODULE. " USER_COMMAND_1000 INPUT

okcode should be in CAPS.

Regards

vijay

Re: Screen painter

Former Member — Wed, 31 May 2006 09:17:35 GMT

Make the following change..

CASE OK_CODE.

WHEN 'SAVE'. "wrote SAVE in capitals

INSERT ZTDR.

WHEN 'EXIT'. "wrote EXIT in capitals

LEAVE PROGRAM.

ENDCASE.

Also make sure that OK_CODE is declared in the Element List of the Screen 1000

Re: Screen painter

Former Member — Wed, 31 May 2006 09:19:51 GMT

Function COdes should always be in CAPS.

Enjoy Coding...

Thomas.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:30:17 GMT

Hi Mark,

Have you activated the screen flow logic in the screen painter..? I think that may be the reason why the exit button is not working and for the data not getting updated...

Regards,

SP.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:34:57 GMT

Tried, but no effect.

Any other suggestion.

Regards,

Re: Screen painter

Former Member — Wed, 31 May 2006 09:35:26 GMT

Tried, but no effect.

Any other suggestion.

Regards,

Re: Screen painter

Former Member — Wed, 31 May 2006 09:35:58 GMT

Tried, but no effect.

Any other suggestion.

Regards,

Re: Screen painter

Former Member — Wed, 31 May 2006 09:36:42 GMT

It is in active mode.

Regards.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:37:42 GMT

are u mention ok code in screen -->element list...

declared ok code in the element List of screen Painter.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:39:06 GMT

hi mark,

u have forgot to assign ok_code u have just defined it.

give after

DATA OK_CODE LIKE SY-UCOMM.

ok_code = sy-ucomm.

and try.

hope this helps,

priya.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:42:42 GMT

Hi Mark,

First assign OK_CODE = SY-UCOMM.

Also I hope the field's name is also related to the table...

For eg: if you have an element for entering field1 of table ZABC, the screen field name should be ZABC-field1. Then only you can use the statement

INSERT ZABC.

Also if you are using a variable for SY-UCOMM (say ok_code) that should be declared in the element List of screen Painter.. Have you done that??

Anyway, give me the details of the table like the element , its type etc..

Let me also try it out...

Regards,

SP.

Message was edited by: Sylendra Prasad

Re: Screen painter

Former Member — Wed, 31 May 2006 09:45:03 GMT

Hi,

ok_code = sy-ucomm. <<<<<----"you missed it.

Regards

vijay

Re: Screen painter

Former Member — Wed, 31 May 2006 09:53:06 GMT

Added the line 'ok_code = sy-ucomm.' But no effect.

Regards.

Re: Screen painter

Former Member — Wed, 31 May 2006 09:54:49 GMT

Atlease for Exit (leave program) should work. But it is not working .

Regards,

Re: Screen painter

Former Member — Wed, 31 May 2006 10:02:10 GMT

Hi Mark

I don't know how you have called the screen, anyway your code should be:

TOP (Include): here you should declare your data:

TABLES : ZTDR.

DATA OK_CODE LIKE SY-UCOMM.

Screen:

Insert OK_CODE in the elements list of the screen

MODULE USER_COMMAND_1000 INPUT.

CASE OK_CODE.

WHEN 'SAVE'.

INSERT ZTDR.

COMMIT WORK

WHEN 'EXIT'.

LEAVE PROGRAM.

ENDCASE.

ENDMODULE. " USER_COMMAND_1000 INPUT

It should be better to not use the 1000 as number for a screen, it's usually used for selection-screen.

Max

Re: Screen painter

Former Member — Wed, 31 May 2006 10:02:16 GMT

Hi Mark,

Can you just give the details of the table...?

What about the screen field names??

Suppose ZABC is the table and f1,f2...fn are the fields in the table, then the screen field names should be ZABC-f1,ZABC-f2,ZABC-fn...

Also for the exit button , is the Function type E..? Then it should be come under the module AT EXIT-COMMAND....

Also, have you defined OK_CODE in the element list???

Regards,

SP.

Re: Screen painter

Former Member — Wed, 31 May 2006 10:02:21 GMT

ok_code = sy-ucomm.

This line is added now. but not effect.

Regards.