This is an automated email from the ASF dual-hosted git repository. jleroux pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git
commit 164d39a7fc9729d54ae060897d3e31a0ba265117 Author: Jacques Le Roux <[email protected]> AuthorDate: Fri Dec 3 12:25:25 2021 +0100 Improved: Fix some bugs Spotbugs reports (OFBIZ-12386) Removes wrong comment, it's a test class --- .../org/apache/ofbiz/widget/renderer/macro/MacroFormRendererTest.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/framework/widget/src/test/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRendererTest.java b/framework/widget/src/test/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRendererTest.java index 1abb193..1473b9c 100644 --- a/framework/widget/src/test/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRendererTest.java +++ b/framework/widget/src/test/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRendererTest.java @@ -845,7 +845,6 @@ public class MacroFormRendererTest { modelForm.getPaginateTarget(withNotNull()); result = targetService; // FIXME SpotBugs reports: Argument 1 is definitely null but must not be null - // Also this method is not used at all OOTB requestHandler.makeLink(null, null, withSubstring(qbeString)); result = linkFromQbeString; } @@ -873,7 +872,6 @@ public class MacroFormRendererTest { modelForm.getPaginateTarget(withNotNull()); result = paginateTarget; // FIXME SpotBugs reports: Argument 1 is definitely null but must not be null - // Also this method is not used at all OOTB requestHandler.makeLink(null, null, withSubstring(qbeString)); result = linkFromQbeString;
