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 nextmarker in AFSFileClientImpl.Codeunit #1408

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

Pieterheinevdm
Copy link
Contributor

@Pieterheinevdm Pieterheinevdm commented Jun 26, 2024

if NodeList is empty no temprecords are beiing inserted and an error will occur because of the modify on an not inserted temprecord

Summary

Work Item(s)

Fixes #1409

Fixes AB#540616

if NodeList is empty no temprecords are beiing inserted and an error will occur because of the modify on an not inserted temprecord
@Pieterheinevdm Pieterheinevdm requested a review from a team as a code owner June 26, 2024 12:39
@github-actions github-actions bot added AL: System Application From Fork Pull request is coming from a fork labels Jun 26, 2024
@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Jul 1, 2024
@JesperSchulz JesperSchulz added the Linked Issue is linked to a Azure Boards work item label Jul 8, 2024
@JesperSchulz JesperSchulz self-assigned this Jul 8, 2024
@github-actions github-actions bot added this to the Version 25.0 milestone Jul 8, 2024
@JesperSchulz JesperSchulz enabled auto-merge (squash) July 8, 2024 12:33
@JesperSchulz JesperSchulz merged commit eef094b into microsoft:main Jul 8, 2024
30 of 33 checks passed
JesperSchulz pushed a commit that referenced this pull request Aug 27, 2024
if NodeList is empty no temprecords are beiing inserted and an error
will occur because of the modify on an not inserted temprecord

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes #1409

Fixes
[AB#540616](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/540616)
JesperSchulz added a commit that referenced this pull request Aug 27, 2024
<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Backport of #1408.

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#545984](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/545984)

Co-authored-by: Pieter-Heine van der Meulen <pieter-heine.van.der.meulen@9altitudes.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AL: System Application From Fork Pull request is coming from a fork Integration GitHub request for Integration area Linked Issue is linked to a Azure Boards work item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: setting nextmarker in AFSFileClientImpl.codeunit results in an error if nothing is found
3 participants