Skip to content

fix: Fix DML ON clause for imperative statements #2601

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

Open
wants to merge 6 commits into
base: development
Choose a base branch
from

Conversation

Nurkambay
Copy link
Contributor

Checklist:

  • Each of my commits contains one meaningful change
  • I have performed rebase of my branch on top of the development
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have checked my code and corrected any misspellings

@Nurkambay
Copy link
Contributor Author

According to the IDMS team information, all ON causes must generate IF statement with a PERFORM IDMS-STATUS statement.

  • IDMS tests updated according to this changes
  • New US to support COPY IDMS IDMS-STATUS. created.

@Nurkambay Nurkambay force-pushed the fix-idms branch 2 times, most recently from 9a1581b to 777c44e Compare November 25, 2024 17:17
@Nurkambay Nurkambay marked this pull request as draft November 25, 2024 19:29
@VitGottwald
Copy link
Contributor

Adding a link to the IDMS documentation for future reference:

As documented in AUTOSTATUS Protocols. the ON clause can be used on any DML statement except an IF DML statements (IF MEMBER , IF EMPTY). The COBOL IF for the ON clause will be generated the same exact way for every DML .

@Nurkambay Nurkambay changed the title fix: Fix IDMS OBTAIN statement that generates IF statement fix: Fix DML ON clause for imperative statements Nov 26, 2024
@Nurkambay Nurkambay marked this pull request as ready for review November 26, 2024 11:27
@Nurkambay
Copy link
Contributor Author

@Nurkambay Nurkambay marked this pull request as draft November 26, 2024 12:30
@Nurkambay
Copy link
Contributor Author

I have excluded DELETE statement from the generating IF statement process

@Nurkambay Nurkambay marked this pull request as ready for review November 26, 2024 15:16
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.

3 participants