Skip to content

Make IfPipe#defaultMediaType more dynamic, add type TEXT#10258

Merged
nielsm5 merged 4 commits intomasterfrom
issue/10254_IfPipe_DefaultSupportedMediaType
Jan 13, 2026
Merged

Make IfPipe#defaultMediaType more dynamic, add type TEXT#10258
nielsm5 merged 4 commits intomasterfrom
issue/10254_IfPipe_DefaultSupportedMediaType

Conversation

@tnleeuw
Copy link
Contributor

@tnleeuw tnleeuw commented Jan 12, 2026

Changes

Pull Request Checklist

Title

  • Title expresses the business value (who benefits + what outcome)

Issues

Backports

  • Backport PRs created (if needed) and linked

Documentation

  • FF! Doc updated (user-facing behavior/config)
  • FF! Manual updated (if applicable)
  • Javadoc updated/generated (developer-facing APIs)

Tests

  • Unit tests added/updated
  • E2E/Integration tests added/updated (if applicable)

Breaking changes

  • Breaking change recorded in markdown file
  • Migration notes included (if needed)

@tnleeuw tnleeuw requested review from evandongen and nielsm5 January 12, 2026 17:16
@tnleeuw tnleeuw self-assigned this Jan 12, 2026
Comment on lines +164 to +165
TEXT(MediaType.TEXT_PLAIN)
;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
TEXT(MediaType.TEXT_PLAIN)
;
TEXT(MediaType.TEXT_PLAIN);

Comment on lines +406 to +407
* @param defaultMediaType The default media type to use when the media type of the message could not be determined. Should
* be one of {@literal XML}, {@literal JSON} or {@literal TEXT}.
Copy link
Member

Choose a reason for hiding this comment

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

De should be one of hoeft er niet in, dat komt automatisch omdat het een enum is.

@sonarqubecloud
Copy link

@nielsm5 nielsm5 merged commit 4bee51b into master Jan 13, 2026
28 of 29 checks passed
@nielsm5 nielsm5 deleted the issue/10254_IfPipe_DefaultSupportedMediaType branch January 13, 2026 12:07
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.

IfPipe default supportedMediaType (when no xpath nor jpath is still XML)

3 participants