juicewcode opened a new pull request, #7038:
URL: https://github.com/apache/shenyu/pull/7038
## Summary
- Replace `updateByPrimaryKey` with `updateByPrimaryKeySelective` for
alert receiver updates.
- Add the selective mapper method and dynamic MyBatis SQL.
- Convert `enable` and `matchAll` from primitive `boolean` to nullable
`Boolean`.
- Omitted nullable fields are no longer written back to the database.
- Preserve the default value `true` when creating a new alert receiver.
- Add null-safe checks in alert dispatch logic.
## Test
- Updated `AlertReceiverServiceTest` to verify selective updates and
prevent calls to the full-column update method.
- Updated `AlertTransferTest` for nullable Boolean mapping.
- Added coverage for null `enable` and `matchAll` values.
- Existing alert dispatch test paths continue to set explicit Boolean
values.
- Performed static checks for remaining primitive getter usage, XML
validity, and whitespace errors.
<!-- Describe your PR here; e.g. Fixes #issueNo -->
<!--
Thank you for proposing a pull request. This template will guide you through
the essential steps necessary for a pull request.
-->
Make sure that:
- [x] You have read the [contribution
guidelines](https://shenyu.apache.org/community/contributor-guide).
- [x] You submit test cases (unit or integration tests) that back your
changes.
- [x] Your local test passed `./mvnw clean install
-Dmaven.javadoc.skip=true`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
