topic Re: Selection screen checkbox - clarification needed in Application Development and Automation Discussions

Selection screen checkbox - clarification needed

former_member699182 — Tue, 04 Nov 2014 07:58:24 GMT

I have a checkbox group with 5 checkboxes in selection screen.

I want to see which of these checkbox is checked and assign it to a field I will be passing to a function module.

I know that the active checkbox value will be set to X.

But what if two checkboxes are checked at the same time?

Should I do a case-when or if-else?

Sorry this is quite a basic logic but am lil confused.

Please help.

Thanks.

Re: Selection screen checkbox - clarification needed

former_member201275 — Tue, 04 Nov 2014 08:11:53 GMT

Please post your code here as that makes not much sense.

Re: Selection screen checkbox - clarification needed

Former Member — Tue, 04 Nov 2014 08:16:35 GMT

Hi,

Use separate if endif block for each checkbox and in it assign the relevant field of function module.

Regards,

Ashish

Re: Selection screen checkbox - clarification needed

former_member699182 — Tue, 04 Nov 2014 08:20:39 GMT

Thanks. Right now i am doing it like the one below.

But I would like to know if there is any simpler way than this.

IF p_pln EQ 'X'.

ls_btcselect-prelim = 'X'.

ENDIF.

IF p_rel EQ 'X'.

ls_btcselect-schedul = 'X'.

ENDIF.

IF p_rdy EQ 'X'.

ls_btcselect-ready = 'X'.

ENDIF.

IF p_act EQ 'X'.

ls_btcselect-running = 'X'.

ENDIF.

IF p_fin EQ 'X'.

ls_btcselect-finished = 'X'.

ENDIF.

IF p_can EQ 'X'.

ls_btcselect-aborted = 'X'.

ENDIF.

CALL FUNCTION 'BP_JOB_SELECT'

EXPORTING

jobselect_dialog = 'N'

jobsel_param_in = ls_btcselect

TABLES

jobselect_joblist = lt_btcjob

EXCEPTIONS

no_jobs_found = 1

OTHERS = 2.

Thanks.

Re: Selection screen checkbox - clarification needed

former_member699182 — Tue, 04 Nov 2014 08:23:39 GMT

Thank you Ashish. Does that mean my code above is the only way we can go about it?

Re: Selection screen checkbox - clarification needed

former_member201275 — Tue, 04 Nov 2014 08:27:26 GMT

This is better/neater:

   CASE 'X'.
WHEN p_pln.
    ls_btcselect-prelim = 'X'.
WHEN p_rel.
    ls_btcselect-schedul = 'X'.
WHEN p_rdy.
    ls_btcselect-ready = 'X'.
WHEN p_act.
    ls_btcselect-running = 'X'.
WHEN p_fin.
    ls_btcselect-finished = 'X'.
WHEN p_can.
    ls_btcselect-aborted = 'X'.
ENDCASE.

Re: Selection screen checkbox - clarification needed

Mohamed_Mukhtar — Tue, 04 Nov 2014 08:27:56 GMT

Hi Gita,

You may use FM RS_SELECTIONSCREEN_READ to read selection screen .

Pass report name and screen number.

Thanks

Re: Selection screen checkbox - clarification needed

Former Member — Tue, 04 Nov 2014 08:42:01 GMT

The you have written now is ok.

When two check boxes selected then two fields wil be filled with 'X'.

for ex: p_pln and p_rel are selected then

ls_btcselect-prelim = 'X'.

ls_btcselect-schedul = 'X'.

will be passed to FM.

regards.

laxman.

Re: Selection screen checkbox - clarification needed

Former Member — Tue, 04 Nov 2014 08:42:18 GMT

Hi Gita,

Glen anthnony's code is short and best one, it reduces multiple if statments as in your code.

Thanks & Regards

Prakash.

Re: Selection screen checkbox - clarification needed

Former Member — Tue, 04 Nov 2014 08:45:12 GMT

Hi,

You can attach a USER-COMMAND to all the checkboxes in order to receive the values of the particular checked box and capture those values AT SELECTION SCREEN OUTPUT.

for e.g.

TABLES SSCRFIELDS .

DATA : SCOMM TYPE SSCRFIELDS-UCOMM.

PARAMETERS: P_PLN AS CHECKBOX USER-COMMAND UC1.

PARAMETERS: P_REL AS CHECKBOX USER-COMMAND UC2.

PARAMETERS: P_RDY AS CHECKBOX USER-COMMAND UC3.

PARAMETERS: P_ACT AS CHECKBOX USER-COMMAND UC4.

PARAMETERS: P_FIN AS CHECKBOX USER-COMMAND UC5.

PARAMETERS: P_CAN AS CHECKBOX USER-COMMAND UC6.

AT SELECTION-SCREEN OUTPUT.

CASE SCOMM.

WHEN 'UC1'.

ls_btcselect-prelim = 'X'.

WHEN 'UC2'.

ls_btcselect-schedul = 'X'.

WHEN 'UC3'.

ls_btcselect-ready = 'X'.

WHEN 'UC4'

ls_btcselect-running = 'X'.

WHEN 'UC5'.

ls_btcselect-finished = 'X'.

WHEN 'UC6'.

ls_btcselect-aborted = 'X'.

ENDCASE.

Re: Selection screen checkbox - clarification needed

Former Member — Tue, 04 Nov 2014 08:45:59 GMT

Hi,

You don't even have to put if endif block. Simply equate the checkboxes to the fields of the structure, those are checked will be assigned as 'X' in the structure and which are not ticked will be left blank.

ls_btcselect-prelim = p_pln.

ls_btcselect-schedule = p_rel.

Similarly for all fields.

Thus which are ticked will only be assigned as 'X' in the structure.

Re: Selection screen checkbox - clarification needed

former_member699182 — Wed, 05 Nov 2014 01:48:06 GMT

Thanks Glen. I tried this case-when method already and it just sets any one field to X and comes out of endcase. Is that how case-when works? Not sure. If endif worked fine for me though i would prefer case-when.

Re: Selection screen checkbox - clarification needed

former_member699182 — Wed, 05 Nov 2014 01:49:51 GMT

Thanks all for the helpful answers.

Re: Selection screen checkbox - clarification needed

Former Member — Wed, 05 Nov 2014 01:54:00 GMT

Hi Gita,

I would say Ashish Rawat's suggestion is the best way of doing it. i.e. just assign the checkboxes directly to the structure fields.

Thanks,

Naveen

Re: Selection screen checkbox - clarification needed

former_member699182 — Wed, 05 Nov 2014 01:56:29 GMT

Logical.

Thanks a lot.