[jira] [Created] (OFBIZ-9731) [FB] Package org.apache.ofbiz.order.shoppinglist

["Julian Leichert (JIRA)" <ji...@apache.org>]

Julian Leichert created OFBIZ-9731:
--------------------------------------

             Summary: [FB] Package org.apache.ofbiz.order.shoppinglist
                 Key: OFBIZ-9731
                 URL: https://issues.apache.org/jira/browse/OFBIZ-9731
             Project: OFBiz
          Issue Type: Sub-task
          Components: order
    Affects Versions: Trunk
            Reporter: Julian Leichert
            Priority: Minor


ShoppingListEvents.java:194, NP_LOAD_OF_KNOWN_NULL_VALUE´
- NP: Load of known null value in org.apache.ofbiz.order.shoppinglist.ShoppingListEvents.addListToCart(HttpServletRequest, HttpServletResponse)

The variable referenced at this point is known to be null due to an earlier check against null. Although this is valid, it might be a mistake (perhaps you intended to refer to a different variable, or perhaps the earlier check to see if the variable is null should have been a check to see if it was non-null).

ShoppingListEvents.java:328, DE_MIGHT_IGNORE
- DE: org.apache.ofbiz.order.shoppinglist.ShoppingListEvents.replaceShoppingListItem(HttpServletRequest, HttpServletResponse) might ignore java.lang.Exception

This method might ignore an exception.  In general, exceptions should be handled or reported in some way, or they should be thrown out of the method.

ShoppingListEvents.java:413, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
- RCN: Redundant nullcheck of itemsArray, which is known to be non-null in org.apache.ofbiz.order.shoppinglist.ShoppingListEvents.fillAutoSaveList(ShoppingCart, LocalDispatcher)

This method contains a redundant check of a known non-null value against the constant null.

ShoppingListServices.java:147, RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE
- RCN: Nullcheck of recurrenceInfo at line 147 of value previously dereferenced in org.apache.ofbiz.order.shoppinglist.ShoppingListServices.createListReorders(DispatchContext, Map)

A value is checked here to see whether it is null, but this value can't be null because it was previously dereferenced and if it were null a null pointer exception would have occurred at the earlier dereference. Essentially, this code and the previous dereference disagree as to whether this value is allowed to be null. Either the check is redundant or the previous dereference is erroneous.

ShoppingListServices.java:178, NP_NULL_ON_SOME_PATH
- NP: Possible null pointer dereference of createResp in org.apache.ofbiz.order.shoppinglist.ShoppingListServices.createListReorders(DispatchContext, Map)

There is a branch of statement that, if executed, guarantees that a null value will be dereferenced, which would generate a NullPointerException when the code is executed. Of course, the problem might be that the branch or statement is infeasible and that the null pointer exception can't ever be executed; deciding that is beyond the ability of FindBugs. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)