tedwon opened a new pull request, #23173: URL: https://github.com/apache/camel/pull/23173
## What Fix a copy-paste error in `Dhis2ResourceTables.analytics()` where `skipEvents != null` was used instead of `skipAggregate != null` when deciding whether to set the `skipAggregate` query parameter. ## Why The `skipAggregate` parameter is only sent to the DHIS2 API when `skipEvents` is non-null, regardless of whether `skipAggregate` was actually set. This means: - If `skipEvents` is null but `skipAggregate` is set, the `skipAggregate` parameter is silently dropped - If `skipEvents` is non-null but `skipAggregate` is null, the parameter is sent with a null value ## Fix Change the condition on line 38 from `skipEvents != null` to `skipAggregate != null`. Found via static analysis (Coverity `COPY_PASTE_ERROR` checker). -- 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]
