Skip to content

Rename Pipe emptyInputReplacement to defaultValue and improve Frank!Doc#10221

Merged
nielsm5 merged 4 commits intomasterfrom
issue/10166_DocImprovements
Jan 6, 2026
Merged

Rename Pipe emptyInputReplacement to defaultValue and improve Frank!Doc#10221
nielsm5 merged 4 commits intomasterfrom
issue/10166_DocImprovements

Conversation

@tnleeuw
Copy link
Contributor

@tnleeuw tnleeuw commented Jan 5, 2026

Changes

Improve Frank!Doc / JavaDoc and rename attribute emptyInputReplacement to defaultValue.

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)

Copy link
Contributor

@evandongen evandongen left a comment

Choose a reason for hiding this comment

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

Duidelijk met het commentaar erbij 👌

Comment on lines +380 to +381
public void setDefaultValue(String string) {
defaultValue = string;
Copy link
Contributor

Choose a reason for hiding this comment

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

	public void setDefaultValue(String defaultValue) {
		this.defaultValue = defaultValue;

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 6, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
61.9% Coverage on New Code (required ≥ 65%)

See analysis details on SonarQube Cloud

@nielsm5 nielsm5 merged commit 39755f1 into master Jan 6, 2026
26 of 44 checks passed
@nielsm5 nielsm5 deleted the issue/10166_DocImprovements branch January 6, 2026 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IfPipe should consider non-existing sessionKey in 'getInputFromSessionKey' as empty instead of throwing an error

3 participants