abap-file-formats icon indicating copy to clipboard operation
abap-file-formats copied to clipboard

DOMA feedback

Open larshp opened this issue 2 years ago • 4 comments

https://github.com/SAP/abap-file-formats/blob/main/file-formats/doma/type/zif_aff_doma_v1.intf.abap

TYPES ty_data_type TYPE c LENGTH 4. this is the internal data type(after conversion exit), ie. not what developers see in SE11?

there is also fixed value appends, which as far as I remember are related to domains?

larshp avatar May 11 '23 12:05 larshp

The data type represents the predefined type of the domain.

This is the first value the user has to enter in the editor (see screen shot of ADT and SE11): image

image

schneidermic0 avatar May 16 '23 09:05 schneidermic0

I guess you are right. The aspect of value appends seems not to be covered, yet

schneidermic0 avatar May 16 '23 09:05 schneidermic0

yea, so as a developer I enter "DF16_DEC", what goes into the field of length 4,

TYPES ty_data_type TYPE c LENGTH 4.

? image

larshp avatar May 16 '23 10:05 larshp

OK. Now, I get your point. Yes, I agree. The length of 4 characters does not make sense in the file format. Even if the actual field value on DB is only 4 characters, the ABAP file format should use either (a) values used by the enduser in the tools (ADT/SE11) or have (b) proper enum values. I think I would prefer option a (these can be still also the enum/constant values)

schneidermic0 avatar May 16 '23 11:05 schneidermic0