Source:
<code>private boolean InitializeStorage() { boolean returnValue = false; setStoragePoolId(getVds().getstorage_pool_id()); if (getStoragePool() == null || StoragePoolStatus.Uninitialized == getStoragePool().getstatus() || StoragePoolStatus.Maintanance == getStoragePool().getstatus()) { returnValue = true; _connectStorageSucceeded = true; _connectPoolSucceeded = true; } else { Guid storagePoolGuid = getVds().getstorage_pool_id(); boolean suppressCheck = getAllRunningVdssInPool().size() == 0; StoragePoolParametersBase tempStorageBaseParams = new StoragePoolParametersBase( storagePoolGuid); tempStorageBaseParams.setVdsId(getVds().getId()); tempStorageBaseParams.setSuppressCheck(suppressCheck); tempStorageBaseParams .setTransactionScopeOption(TransactionScopeOption.Suppress); if (Backend .getInstance() .runInternalAction( VdcActionType.ConnectHostToStoragePoolServers, tempStorageBaseParams).getSucceeded() <START> || suppressCheck) { <END> _connectStorageSucceeded = true; reloadStoragePool(); returnValue = _connectPoolSucceeded = executeConnectStoragePoolCommand(storagePoolGuid); if (suppressCheck) { if (!returnValue) { returnValue = reconstructMasterDomain(storagePoolGuid); if (returnValue) { reloadStoragePool(); returnValue = _connectPoolSucceeded = executeConnectStoragePoolCommand(storagePoolGuid); } if (!returnValue) { AuditLogDirector .log(new AuditLogableBase(getVdsId()), AuditLogType.VDS_STORAGE_CONNECTION_FAILED_BUT_LAST_VDS); returnValue = true; } } } } } return returnValue; }</code><technical_language>Do want connect host storage servers if suppressCheck=true? If - do call of if, for readability. If - move ||, allow short circuiting</technical_language>

Target:
private boolean InitializeStorage() { boolean returnValue = false; setStoragePoolId(getVds().getstorage_pool_id()); if (getStoragePool() == null || StoragePoolStatus.Uninitialized == getStoragePool().getstatus() || StoragePoolStatus.Maintanance == getStoragePool().getstatus()) { returnValue = true; _connectStorageSucceeded = true; _connectPoolSucceeded = true; } else { Guid storagePoolGuid = getVds().getstorage_pool_id(); boolean suppressCheck = getAllRunningVdssInPool().size() == 0; StoragePoolParametersBase tempStorageBaseParams = new StoragePoolParametersBase( storagePoolGuid); tempStorageBaseParams.setVdsId(getVds().getId()); tempStorageBaseParams.setSuppressCheck(suppressCheck); tempStorageBaseParams .setTransactionScopeOption(TransactionScopeOption.Suppress); if (Backend .getInstance() .runInternalAction( VdcActionType.ConnectHostToStoragePoolServers, tempStorageBaseParams).getSucceeded() || suppressCheck) { _connectStorageSucceeded = true; reloadStoragePool(); returnValue = _connectPoolSucceeded = executeConnectStoragePoolCommand(storagePoolGuid); if (suppressCheck) { if (!returnValue) { returnValue = executeReconstructMasterDomainCommand(storagePoolGuid); if (returnValue) { reloadStoragePool(); returnValue = _connectPoolSucceeded = executeConnectStoragePoolCommand(storagePoolGuid); } if (!returnValue) { AuditLogDirector .log(new AuditLogableBase(getVdsId()), AuditLogType.VDS_STORAGE_CONNECTION_FAILED_BUT_LAST_VDS); returnValue = true; } } } } } return returnValue; }