Skip to content

Conversation

@OlegoO
Copy link
Contributor

@OlegoO OlegoO commented Jun 6, 2025

Description

feat: Introduced ShoppingCartEntityConfiguration for SQL table configuration, disabling the SQL output clause for resolving that SQL Server tables with triggers or certain computed columns now require special EF Core configuration https://learn.microsoft.com/en-us/ef/core/what-is-new/ef-core-7.0/breaking-changes?tabs=v7#mitigations-2

References

QA-test:

Jira-link:

https://virtocommerce.atlassian.net/browse/VCST-3232

Artifact URL:

https://vc3prerelease.blob.core.windows.net/packages/VirtoCommerce.Cart_3.831.0-pr-172-6712.zip

OlegoO added 2 commits June 6, 2025 15:16
… configuration, disabling the SQL output clause.

feat: Introduced `ShoppingCartEntityConfiguration` for SQL table configuration, disabling the SQL output clause for resolving that SQL Server tables with triggers or certain computed columns now require special EF Core configuration https://learn.microsoft.com/en-us/ef/core/what-is-new/ef-core-7.0/breaking-changes?tabs=v7#mitigations-2
Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.594
Timestamp: 06-06-2025T13:35:44

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.538
Timestamp: 06-06-2025T14:20:01

… any enabled triggers if the statement contains an OUTPUT clause without INTO clause.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 9, 2025

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.687
Timestamp: 09-06-2025T10:37:19

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