mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-07-31 17:39:05 +00:00
Fixed major issues ("Simplify Boolean Expression") reported in Sonar
git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/modules/recordsmanagement/HEAD@71962 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
@@ -130,14 +130,14 @@ public class RMv22GhostOnDestroyDispositionActionPatch extends AbstractModulePat
|
||||
*/
|
||||
private void getDispositionSchedules(NodeRef nodeRef, Set<DispositionSchedule> dispositionSchedules)
|
||||
{
|
||||
if (filePlanService.isRecordCategory(nodeRef) == true)
|
||||
if (filePlanService.isRecordCategory(nodeRef))
|
||||
{
|
||||
DispositionSchedule dispositionSchedule = this.dispositionService.getDispositionSchedule(nodeRef);
|
||||
if (dispositionSchedule != null)
|
||||
{
|
||||
dispositionSchedules.add(dispositionSchedule);
|
||||
}
|
||||
|
||||
|
||||
List<ChildAssociationRef> children = nodeService.getChildAssocs(nodeRef, ContentModel.ASSOC_CONTAINS, RegexQNamePattern.MATCH_ALL);
|
||||
for (ChildAssociationRef childAssoc : children)
|
||||
{
|
||||
|
@@ -123,7 +123,7 @@ public class RMMethodSecurityInterceptor extends MethodSecurityInterceptor
|
||||
if (logger.isDebugEnabled())
|
||||
{
|
||||
CapabilityReport capability = getCapabilityReport(name);
|
||||
if (expected == false)
|
||||
if (!expected)
|
||||
{
|
||||
conditionName = "!" + conditionName;
|
||||
}
|
||||
@@ -155,7 +155,7 @@ public class RMMethodSecurityInterceptor extends MethodSecurityInterceptor
|
||||
for (Map.Entry<String, Boolean> entry : capability.conditions.entrySet())
|
||||
{
|
||||
buffer.append(" - ").append(entry.getKey()).append(" (");
|
||||
if (entry.getValue() == true)
|
||||
if (entry.getValue())
|
||||
{
|
||||
buffer.append("passed");
|
||||
}
|
||||
|
@@ -53,7 +53,7 @@ import org.springframework.extensions.surf.util.ParameterCheck;
|
||||
* @author Tuna Aksoy
|
||||
* @since 2.2
|
||||
*/
|
||||
public class TransferServiceImpl extends ServiceBaseImpl
|
||||
public class TransferServiceImpl extends ServiceBaseImpl
|
||||
implements TransferService, RecordsManagementModel
|
||||
{
|
||||
/** Transfer node reference key */
|
||||
@@ -176,14 +176,14 @@ public class TransferServiceImpl extends ServiceBaseImpl
|
||||
setPDFIndicationFlag(transferNodeRef, nodeRef);
|
||||
|
||||
// Set the transferring indicator aspect
|
||||
nodeService.addAspect(nodeRef, ASPECT_TRANSFERRING, null);
|
||||
if (isRecordFolder(nodeRef) == true)
|
||||
nodeService.addAspect(nodeRef, ASPECT_TRANSFERRING, null);
|
||||
if (isRecordFolder(nodeRef))
|
||||
{
|
||||
// add the transferring indicator aspect to all the child records
|
||||
for (NodeRef record : recordService.getRecords(nodeRef))
|
||||
{
|
||||
nodeService.addAspect(record, ASPECT_TRANSFERRING, null);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return transferNodeRef;
|
||||
|
Reference in New Issue
Block a user