Skip to content
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

Bugfix/ls24003783/array in ds inz #597

Merged
merged 6 commits into from
Aug 26, 2024

Conversation

dom-apuliasoft
Copy link
Collaborator

Description

Given a DataStructure $A containing an array field $B, if $B was also marked with empty INZ keyword the system tried to initialize $B with a real number, throwing a runtime error.

To avoid this issue I added a check on the definition being an array during the creation process.

Related to:

  • LS24003783

Checklist:

  • If this feature involves RPGLE fixes or improvements, they are well-described in the summary.
  • There are tests for this feature.
  • RPGLE code used for tests is easily understandable and includes comments that clarify the purpose of this feature.
  • The code follows Kotlin conventions (run ./gradlew ktlintCheck).
  • The code passes all tests (run ./gradlew check).
  • Relevant documentation is included in the docs directory.

@lanarimarco lanarimarco merged commit edd4de4 into develop Aug 26, 2024
1 check passed
@lanarimarco lanarimarco deleted the bugfix/LS24003783/array-in-ds-inz branch August 26, 2024 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants