-
Notifications
You must be signed in to change notification settings - Fork 72
[EDOT] Add new object EDOT #682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 30 commits
a8ec8b7
499d83a
e1e96b2
ecdefa0
3a2e9b9
a1ec907
6ed5d4c
4c96d92
a7025f3
bdd762a
8848e73
1c07f07
777533b
683a98d
6838a10
ec64a94
91e81a1
f69b218
9d3530c
7164a56
6c1c9d7
b742a7f
58795a5
bc54473
e90dd95
013ffe6
d644706
48946ba
6f7986d
33fafac
eef4b02
9922e72
83d2cc9
5021759
4c90f28
83ba32f
ac7f9f9
2219ab3
aaed091
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
wurzka marked this conversation as resolved.
Show resolved
Hide resolved
i506210 marked this conversation as resolved.
Show resolved
Hide resolved
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # EDOT File Format | ||
|
|
||
| File | Cardinality | Definition | Schema | Example | ||
| :--- | :--- | :--- | :--- | :--- | ||
| `<name>.edot.json` | 1 | [`zif_aff_edot_v1.intf.abap`](./type/zif_aff_edot_v1.intf.abap) | [`edot-v1.json`](./edot-v1.json) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,161 @@ | ||
| { | ||
| "$comment": "This file is autogenerated, do not edit manually, see https://github.com/SAP/abap-file-formats for more information.", | ||
| "$schema": "https://json-schema.org/draft/2020-12/schema", | ||
| "$id": "https://github.com/SAP/abap-file-formats/blob/main/file-formats/edot/edot-v1.json", | ||
| "title": "eDocument Type", | ||
| "description": "eDocument type", | ||
| "type": "object", | ||
| "properties": { | ||
| "formatVersion": { | ||
| "title": "Format Version", | ||
| "description": "Format version", | ||
| "type": "string", | ||
| "const": "1" | ||
| }, | ||
| "header": { | ||
| "title": "Header", | ||
| "description": "Header", | ||
| "type": "object", | ||
| "properties": { | ||
| "description": { | ||
| "title": "Description", | ||
| "description": "eDocument type description", | ||
| "type": "string", | ||
| "maxLength": 30 | ||
| }, | ||
| "originalLanguage": { | ||
| "title": "Original Language", | ||
| "description": "Original language of the eDocument type", | ||
| "type": "string", | ||
| "minLength": 2 | ||
| }, | ||
| "abapLanguageVersion": { | ||
| "title": "ABAP Language Version", | ||
| "description": "ABAP language version", | ||
| "type": "string", | ||
| "enum": [ | ||
| "standard", | ||
| "cloudDevelopment" | ||
| ], | ||
| "enumTitles": [ | ||
| "Standard", | ||
| "ABAP Cloud Development" | ||
| ], | ||
| "enumDescriptions": [ | ||
| "Standard", | ||
| "ABAP cloud development" | ||
| ], | ||
| "default": "standard" | ||
| } | ||
| }, | ||
| "additionalProperties": false, | ||
| "required": [ | ||
| "description", | ||
| "originalLanguage" | ||
| ] | ||
| }, | ||
| "generalInformation": { | ||
| "title": "General Information", | ||
| "description": "General information", | ||
| "type": "object", | ||
| "properties": { | ||
| "tableName": { | ||
| "title": "eDocument Table Name", | ||
| "description": "eDocument table name", | ||
| "type": "string", | ||
| "maxLength": 30 | ||
| }, | ||
| "isArchivePreprocessNeeded": { | ||
| "title": "Preprocess Before Archiving", | ||
| "description": "Preprocess before archiving", | ||
| "type": "boolean" | ||
| }, | ||
| "isCreatedInContingencyMode": { | ||
| "title": "Created in Contingency Mode", | ||
| "description": "Created in contingency mode", | ||
| "type": "boolean" | ||
| }, | ||
| "contingencyEdocType": { | ||
| "title": "Contingency eDocument Type", | ||
| "description": "Contingency eDocument type", | ||
| "type": "string", | ||
| "maxLength": 10 | ||
| } | ||
| }, | ||
| "additionalProperties": false | ||
| }, | ||
| "readAccessLoggingSettings": { | ||
| "title": "Read Access Logging Settings", | ||
| "description": "Read access logging settings", | ||
| "type": "array", | ||
| "uniqueItems": true, | ||
| "items": { | ||
| "title": "Read Access Loggging Settings", | ||
| "description": "Read access logging settings", | ||
| "type": "object", | ||
| "properties": { | ||
| "fileType": { | ||
| "title": "File Type", | ||
| "description": "File type", | ||
| "type": "string", | ||
| "maxLength": 10 | ||
| }, | ||
| "fileStructure": { | ||
| "title": "File Structure Type", | ||
| "description": "File structure type", | ||
| "type": "string", | ||
| "maxLength": 30 | ||
| }, | ||
| "fileDescription": { | ||
| "title": "File Description", | ||
| "description": "File description", | ||
| "type": "string", | ||
| "maxLength": 60 | ||
| } | ||
| }, | ||
| "additionalProperties": false, | ||
| "required": [ | ||
| "fileType", | ||
| "fileStructure" | ||
| ] | ||
| } | ||
| }, | ||
| "edocSpecAdditionalTables": { | ||
| "title": "eDocument Type Specific Additional Tables", | ||
| "description": "eDocument type specific additional tables", | ||
| "type": "array", | ||
| "uniqueItems": true, | ||
| "items": { | ||
| "title": "eDocument Type Specific Additional Tables", | ||
| "description": "eDocument type specific additional tables", | ||
| "type": "object", | ||
| "properties": { | ||
| "sequenceNumber": { | ||
| "title": "Sequence Number", | ||
| "description": "Sequence number", | ||
| "type": "integer", | ||
| "minimum": 1, | ||
| "maximum": 9999 | ||
| }, | ||
| "tableName": { | ||
| "title": "Table Name", | ||
| "description": "Table name", | ||
| "type": "string", | ||
| "maxLength": 30 | ||
| } | ||
| }, | ||
| "additionalProperties": false, | ||
| "required": [ | ||
| "sequenceNumber", | ||
| "tableName" | ||
| ] | ||
| } | ||
| } | ||
| }, | ||
| "additionalProperties": false, | ||
| "required": [ | ||
| "formatVersion", | ||
| "header", | ||
| "generalInformation" | ||
| ] | ||
| } |
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,26 @@ | ||||||||||
| { | ||||||||||
| "formatVersion": "1", | ||||||||||
| "header": { | ||||||||||
| "description": "Example EDOT for file formats", | ||||||||||
| "originalLanguage": "en" | ||||||||||
| }, | ||||||||||
| "generalInformation": { | ||||||||||
| "tableName": "EDOFRINV", | ||||||||||
| "isArchivePreprocessNeeded": false, | ||||||||||
| "isCreatedInContingencyMode": false | ||||||||||
|
||||||||||
| "tableName": "EDOFRINV", | |
| "isArchivePreprocessNeeded": false, | |
| "isCreatedInContingencyMode": false | |
| "tableName": "EDOFRINV" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't understand the comments, do I need to make any change here
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just an idea. I am not sure whether this makes sense:
Almost all fields start with "file". I am not sure, but I was wondering whether it make sense to add "files" to the name of the array.
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boolean values with value false aren't serialized by default.
| "fileDescription": "eDocument France: Invoice Request", | |
| "isCloudRelevant": false | |
| "fileDescription": "eDocument France: Invoice Request" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same - I didn't understand the comments, do I need to make any change here
i506210 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,101 @@ | ||
| "! <p class="shorttext synchronized">AFF for eDocument Type (EDOT)</p> | ||
| "! eDocument types that can be used in eInvoice process | ||
| INTERFACE zif_aff_edot_v1 | ||
| PUBLIC. | ||
|
|
||
| TYPES: | ||
| "! <p class="shorttext">Read Access Loggging Settings</p> | ||
| "! Read access logging settings | ||
| BEGIN OF ty_read_access_logging_setting, | ||
| "! <p class="shorttext">File Type</p> | ||
| "! File type | ||
| "! $required | ||
| file_type TYPE c LENGTH 10, | ||
|
Comment on lines
+10
to
+13
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are file types defined by a hard set of values? If so, we could use an enumeration here. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Its not hard set of values but coming from check table, we need to have value help instead fixed domain, I think it will be covered during development |
||
| "! <p class="shorttext">File Structure Type</p> | ||
| "! File structure type | ||
| "! $required | ||
| file_structure TYPE c LENGTH 30, | ||
|
||
| "! <p class="shorttext">File Description</p> | ||
| "! File description | ||
| file_description TYPE c LENGTH 60, | ||
| END OF ty_read_access_logging_setting. | ||
| "! <p class="shorttext">Read Access Logging Settings</p> | ||
| "! Read access logging settings | ||
| TYPES ty_read_acc_logging_settings TYPE SORTED TABLE OF ty_read_access_logging_setting WITH UNIQUE KEY file_type. | ||
| TYPES: | ||
| "! <p class="shorttext">eDocument Type Specific Additional Tables</p> | ||
| "! eDocument type specific additional tables | ||
| BEGIN OF ty_additional_table, | ||
GuilhermeSaraiva96 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| "! <p class="shorttext">Sequence Number</p> | ||
| "! Sequence number | ||
| "! $required | ||
| "! $minimum 1 | ||
| "! $maximum 9999 | ||
| sequence_number TYPE i, | ||
| "! <p class="shorttext">Table Name</p> | ||
| "! Table name | ||
| "! $required | ||
| table_name TYPE zif_aff_types_v1=>ty_object_name_30, | ||
| END OF ty_additional_table. | ||
| "! <p class="shorttext">Read Access Logging Settings</p> | ||
| "! Read access logging settings | ||
| TYPES ty_additional_tables TYPE SORTED TABLE OF ty_additional_table WITH UNIQUE KEY sequence_number. | ||
| TYPES: | ||
| "! <p class="shorttext">General Information</p> | ||
| "! General information | ||
| BEGIN OF ty_general_information, | ||
| "! <p class="shorttext">eDocument Table Name</p> | ||
| "! eDocument table name | ||
| table_name TYPE zif_aff_types_v1=>ty_object_name_30, | ||
| "! <p class="shorttext">Preprocess Before Archiving</p> | ||
| "! Preprocess before archiving | ||
| is_archive_preprocess_needed TYPE abap_bool, | ||
| "! <p class="shorttext">Created in Contingency Mode</p> | ||
| "! Created in contingency mode | ||
| is_created_in_contingency_mode TYPE abap_bool, | ||
|
||
| "! <p class="shorttext">Contingency eDocument Type</p> | ||
| "! Contingency eDocument type | ||
| contingency_edoc_type TYPE c LENGTH 10, | ||
i506210 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| END OF ty_general_information. | ||
| TYPES: | ||
| "! <p class="shorttext">Header for EDOT object</p> | ||
| "! The header for an ABAP main object (without source code) with a description of 30 characters (no key user) | ||
| BEGIN OF ty_header_30_cloud, | ||
| "! <p class="shorttext">Description</p> | ||
| "! eDocument type description | ||
| "! $required | ||
| description TYPE c LENGTH 30, | ||
| "! <p class="shorttext">Original Language</p> | ||
| "! Original language of the eDocument type | ||
| "! $required | ||
| original_language TYPE sy-langu, | ||
| "! <p class="shorttext">ABAP Language Version</p> | ||
| "! ABAP language version | ||
| "! $values {@link zif_aff_types_v1.data:co_abap_language_version_cloud} | ||
| "! $default {@link zif_aff_types_v1.data:co_abap_language_version_cloud.standard} | ||
| abap_language_version TYPE zif_aff_types_v1=>ty_abap_language_version_cloud, | ||
| END OF ty_header_30_cloud. | ||
| TYPES: | ||
| "! <p class="shorttext">eDocument Type</p> | ||
| "! eDocument type | ||
| BEGIN OF ty_main, | ||
| "! <p class="shorttext">Format Version</p> | ||
| "! Format version | ||
| "! $required | ||
| format_version TYPE zif_aff_types_v1=>ty_format_version, | ||
| "! <p class="shorttext">Header</p> | ||
| "! Header | ||
| "! $required | ||
| header TYPE ty_header_30_cloud, | ||
| "! <p class="shorttext">General Information</p> | ||
| "! General information | ||
| "! $required | ||
| general_information TYPE ty_general_information, | ||
| "! <p class="shorttext">Read Access Logging Settings</p> | ||
| "! Read access logging settings | ||
| read_access_logging_settings TYPE ty_read_acc_logging_settings, | ||
| "! <p class="shorttext">eDocument Type Specific Additional Tables</p> | ||
| "! eDocument type specific additional tables | ||
| edoc_spec_additional_tables TYPE ty_additional_tables, | ||
i506210 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| END OF ty_main. | ||
| ENDINTERFACE. | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| { | ||
| "formatVersion": "1", | ||
| "header": { | ||
| "description": "AFF for eDocument Type (EDOT)", | ||
| "originalLanguage": "en" | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Example is missing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think having an example would also help me for the review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Somehow I am unable to generate example, I created class and Transformation but its not clear to me in the report what object to be mentioned as input.