You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Dennis Balkir (JIRA)" <ji...@apache.org> on 2017/09/13 10:11:00 UTC
[jira] [Created] (OFBIZ-9711) [FB] Package
org.apache.ofbiz.entity.cache
Dennis Balkir created OFBIZ-9711:
------------------------------------
Summary: [FB] Package org.apache.ofbiz.entity.cache
Key: OFBIZ-9711
URL: https://issues.apache.org/jira/browse/OFBIZ-9711
Project: OFBiz
Issue Type: Sub-task
Components: framework
Affects Versions: Trunk
Reporter: Dennis Balkir
Priority: Minor
- AbstractEntityConditionCache.java:68, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
RCN: Redundant nullcheck of model, which is known to be non-null in org.apache.ofbiz.entity.cache.AbstractEntityConditionCache.remove(GenericEntity)
This method contains a redundant check of a known non-null value against the constant null.
- AbstractEntityConditionCache.java:169, UCF_USELESS_CONTROL_FLOW
UCF: Useless control flow in org.apache.ofbiz.entity.cache.AbstractEntityConditionCache.storeHook(boolean, GenericEntity, GenericEntity)
This method contains a useless control flow statement, where control flow continues onto the same place regardless of whether or not the branch is taken. For example, this is caused by having an empty statement block for an if statement:
if (argv.length == 0) {
// TODO: handle this case
}
- AbstractEntityConditionCache.java:183, UCF_USELESS_CONTROL_FLOW
UCF: Useless control flow in org.apache.ofbiz.entity.cache.AbstractEntityConditionCache.storeHook(String, boolean, List, List)
This method contains a useless control flow statement, where control flow continues onto the same place regardless of whether or not the branch is taken. For example, this is caused by having an empty statement block for an if statement:
if (argv.length == 0) {
// TODO: handle this case
}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)