mcgilman opened a new pull request, #11734:
URL: https://github.com/apache/nifi/pull/11734

   Skip nonexistent position sanitization for connections while preserving bend 
coordinate validation in the flow designer and connector canvas.
   
   # Summary
   
   [NIFI-16406](https://issues.apache.org/jira/browse/NIFI-16406)
   
   Canvas connections do not have a position. Their geometry is represented by 
endpoints and bend coordinates.
   
   This change:
   
   - Stops sanitizing nonexistent connection positions in the flow designer and 
connector canvas
   - Preserves validation and fallback behavior for invalid connection bends
   - Adds regression tests confirming positionless connections do not emit 
warnings or receive artificial positions
   
   # Tracking
   
   Please complete the following tracking steps prior to pull request creation.
   
   ### Issue Tracking
   
   - [Apache NiFi Jira](https://issues.apache.org/jira/browse/NIFI) issue 
created
   
   ### Pull Request Tracking
   
   - Pull Request title starts with Apache NiFi Jira issue number, such as 
`NIFI-00000`
   - Pull Request commit message starts with Apache NiFi Jira issue number, as 
such `NIFI-00000`
   - Pull request contains [commits 
signed](https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits)
 with a registered key indicating `Verified` status
   
   ### Pull Request Formatting
   
   - Pull Request based on current revision of the `main` branch
   - Pull Request refers to a feature branch with one commit containing changes
   
   # Verification
   
   - `npx nx test nifi` — 2,915 tests passed
   - `npx nx lint nifi`
   - `npx nx build nifi --configuration=production`
   - Prettier and `git diff --check`
   
   ### Build
   
   - [ ] Build completed using `./mvnw clean install -P contrib-check`
     - [ ] JDK 21
     - [ ] JDK 25
   
   ### Licensing
   
   - [ ] New dependencies are compatible with the [Apache License 
2.0](https://apache.org/licenses/LICENSE-2.0) according to the [License 
Policy](https://www.apache.org/legal/resolved.html)
   - [ ] New dependencies are documented in applicable `LICENSE` and `NOTICE` 
files
   
   No new dependencies were introduced.
   
   ### Documentation
   
   - [ ] Documentation formatting appears as expected in rendered files
   
   No documentation changes were required.
   


-- 
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]

Reply via email to