[
https://issues.apache.org/jira/browse/NIFI-15665?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18062660#comment-18062660
]
ASF subversion and git services commented on NIFI-15665:
--------------------------------------------------------
Commit 7abf5f759cd0e17358c852ae9ca43b912b1831f9 in nifi's branch
refs/heads/main from machov
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=7abf5f759cd ]
NIFI-15665: Remove duplicate description call in PROP_BROKER_URI
Fixes duplicate description() method calls in AbstractMQTTProcessor where
PROP_BROKER_URI had two description() calls. The first call was redundant
since the second call overrides it.
Removed the first description call to improve code clarity and ensure
users see the complete, detailed description with URI scheme information.
Fixes NIFI-15665
This closes #10960.
Signed-off-by: Pierre Villard <[email protected]>
> Remove duplicate description call in PROP_BROKER_URI (AbstractMQTTProcessor)
> ----------------------------------------------------------------------------
>
> Key: NIFI-15665
> URL: https://issues.apache.org/jira/browse/NIFI-15665
> Project: Apache NiFi
> Issue Type: Bug
> Components: Core Framework
> Reporter: Nikita Awasthi
> Priority: Trivial
> Time Spent: 20m
> Remaining Estimate: 0h
>
> The AbstractMQTTProcessor class contains duplicate description() method calls
> for the PROP_BROKER_URI property definition. The first description call is
> redundant since the second call overrides it completely.
> Current code:
> ```java
> PROP_BROKER_URI = new PropertyDescriptor.Builder()
> .name("Broker URI")
> .description("The URI to use to connect to the MQTT broker") // <- This
> line is redundant
> .required(true)
> .addValidator(StandardValidators.URI_VALIDATOR)
> .expressionLanguageSupported(ExpressionLanguageScope.ENVIRONMENT)
> .description("The URI to use to connect to the MQTT broker (e.g.
> tcp://localhost:1883). The URI scheme must be defined as tcp for plaintext
> MQTT or ssl for MQTT over SSL/TLS.")
> .build();
> ```
> Expected: Remove the first description() call to improve code clarity and
> ensure developers see the complete description with SSL/TLS information.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
