Skip to content

Add nullSafeConverter method to Converter interface #23946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
peholmst wants to merge 1 commit into
base: main
Choose a base branch
from
+110 −0

Add nullSafeConverter method to Converter interface

940caa8
Select commit
Loading
Failed to load commit list.
Draft

Add nullSafeConverter method to Converter interface #23946

Add nullSafeConverter method to Converter interface
940caa8
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 19, 2026 in 1m 53s

Quality Gate passed

Issues
2977 New issues
1 Accepted issue

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 316 in flow-polymer-template/src/test/java/com/vaadin/flow/templatemodel/BeanModelTypeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this lambda with method reference '"intObject"::equals'. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDA_3xQIG-OxPgjP6&open=AZ0FDA_3xQIG-OxPgjP6&pullRequest=23946

Check warning on line 739 in flow-server/src/test/java/com/vaadin/flow/component/ComponentEventBusTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDEVSxQIG-OxPgjs8&open=AZ0FDEVSxQIG-OxPgjs8&pullRequest=23946

Check warning on line 397 in flow-server/src/test/java/com/vaadin/flow/server/communication/UploadHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the parentheses around the "event" parameter 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDCIIxQIG-OxPgjhJ&open=AZ0FDCIIxQIG-OxPgjhJ&pullRequest=23946

Check warning on line 158 in flow-server/src/test/java/com/vaadin/flow/server/communication/WebComponentBootstrapHandlerViteTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDCK8xQIG-OxPgjhl&open=AZ0FDCK8xQIG-OxPgjhl&pullRequest=23946

Check warning on line 464 in flow-server/src/test/java/com/vaadin/flow/signals/shared/SharedListSignalTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this lambda with method reference 'stream::toList'. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDDZVxQIG-OxPgjn9&open=AZ0FDDZVxQIG-OxPgjn9&pullRequest=23946

Check warning on line 234 in flow-server/src/test/java/com/vaadin/flow/signals/local/ValueSignalTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this lambda with method reference 'signal::get'. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDDkKxQIG-OxPgjow&open=AZ0FDDkKxQIG-OxPgjow&pullRequest=23946

Check warning on line 517 in flow-polymer-template/src/test/java/com/vaadin/flow/templatemodel/TemplateModelTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "TemplateModel"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBCKxQIG-OxPgjUL&open=AZ0FDBCKxQIG-OxPgjUL&pullRequest=23946

Check warning on line 283 in flow-server/src/test/java/com/vaadin/flow/server/communication/ServerRpcHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDCGYxQIG-OxPgjgx&open=AZ0FDCGYxQIG-OxPgjgx&pullRequest=23946

Check warning on line 62 in flow-server/src/test/java/com/vaadin/flow/internal/StringUtilTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this String concatenation with Text block. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDFYaxQIG-OxPgjz1&open=AZ0FDFYaxQIG-OxPgjz1&pullRequest=23946

Check warning on line 358 in flow-polymer-template/src/test/java/com/vaadin/flow/component/polymertemplate/NpmTemplateParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "TemplateModel"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBGcxQIG-OxPgjXp&open=AZ0FDBGcxQIG-OxPgjXp&pullRequest=23946

Check warning on line 341 in flow-polymer-template/src/test/java/com/vaadin/flow/component/polymertemplate/PolymerTemplateTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "TemplateParser"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBI6xQIG-OxPgjYv&open=AZ0FDBI6xQIG-OxPgjYv&pullRequest=23946

Check warning on line 105 in flow-polymer-template/src/test/java/com/vaadin/flow/templatemodel/BeanModelTypeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "TemplateModel"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDA_3xQIG-OxPgjPl&open=AZ0FDA_3xQIG-OxPgjPl&pullRequest=23946

Check warning on line 544 in flow-polymer-template/src/test/java/com/vaadin/flow/templatemodel/TemplateModelTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "TemplateModel"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBCKxQIG-OxPgjUT&open=AZ0FDBCKxQIG-OxPgjUT&pullRequest=23946

Check warning on line 1265 in flow-server/src/test/java/com/vaadin/flow/server/startup/RouteRegistryInitializerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDCjFxQIG-OxPgjj_&open=AZ0FDCjFxQIG-OxPgjj_&pullRequest=23946

Check warning on line 301 in flow-polymer-template/src/test/java/com/vaadin/flow/templatemodel/TemplateModelTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Include"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBCKxQIG-OxPgjTR&open=AZ0FDBCKxQIG-OxPgjTR&pullRequest=23946

Check warning on line 179 in flow-server/src/test/java/com/vaadin/flow/component/ShortcutRegistrationTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDD_AxQIG-OxPgjqT&open=AZ0FDD_AxQIG-OxPgjqT&pullRequest=23946

Check warning on line 142 in flow-server/src/test/java/com/vaadin/flow/component/ComponentMetaDataTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDEMqxQIG-OxPgjsW&open=AZ0FDEMqxQIG-OxPgjsW&pullRequest=23946

Check warning on line 87 in flow-polymer-template/src/test/java/com/vaadin/flow/templatemodel/TemplateModelProxyHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "PropertyFilter"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBBHxQIG-OxPgjRB&open=AZ0FDBBHxQIG-OxPgjRB&pullRequest=23946

Check warning on line 1891 in flow-server/src/test/java/com/vaadin/flow/dom/ElementJacksonTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this call to a deprecated class, it has been marked for removal. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDEvTxQIG-OxPgjwE&open=AZ0FDEvTxQIG-OxPgjwE&pullRequest=23946

Check failure on line 127 in flow-server/src/test/java/com/vaadin/flow/server/communication/rpc/PublishedServerEventHandlerRpcHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDB8yxQIG-OxPgjfH&open=AZ0FDB8yxQIG-OxPgjfH&pullRequest=23946

Check warning on line 331 in flow-server/src/test/java/com/vaadin/flow/server/communication/WebComponentProviderTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless assignment to local variable "set". 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDB-PxQIG-OxPgjfi&open=AZ0FDB-PxQIG-OxPgjfi&pullRequest=23946

Check warning on line 50 in flow-server/src/test/java/com/vaadin/flow/server/DefaultErrorHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDDMjxQIG-OxPgjm0&open=AZ0FDDMjxQIG-OxPgjm0&pullRequest=23946

Check warning on line 231 in flow-server/src/test/java/com/vaadin/flow/shared/BrowserDetailsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "BrowserDetails"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDGEExQIG-OxPgj2o&open=AZ0FDGEExQIG-OxPgj2o&pullRequest=23946

Check warning on line 70 in flow-polymer-template/src/test/java/com/vaadin/flow/component/polymertemplate/BundleParserTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "BundleParser"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDBH7xQIG-OxPgjYD&open=AZ0FDBH7xQIG-OxPgjYD&pullRequest=23946

Check warning on line 2036 in flow-server/src/test/java/com/vaadin/flow/dom/ElementJacksonTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Swap these 2 arguments so they are in the correct order: expected value, actual value. 

See more on https://sonarcloud.io/project/issues?id=vaadin_flow&issues=AZ0FDEvTxQIG-OxPgjwb&open=AZ0FDEvTxQIG-OxPgjwb&pullRequest=23946
View more details on SonarQubeCloud