This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a commit to branch nsx-reorder-acl in repository https://gitbox.apache.org/repos/asf/cloudstack.git
commit 04b7b280cf7894aef6c6953312384407f0ab394b Author: Pearl Dsilva <[email protected]> AuthorDate: Tue Feb 6 12:40:17 2024 -0500 tweak reorder logic --- .../java/com/cloud/network/vpc/NetworkACLServiceImpl.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/server/src/main/java/com/cloud/network/vpc/NetworkACLServiceImpl.java b/server/src/main/java/com/cloud/network/vpc/NetworkACLServiceImpl.java index b5b8507cf42..a5835a40312 100644 --- a/server/src/main/java/com/cloud/network/vpc/NetworkACLServiceImpl.java +++ b/server/src/main/java/com/cloud/network/vpc/NetworkACLServiceImpl.java @@ -979,14 +979,14 @@ public class NetworkACLServiceImpl extends ManagerBase implements NetworkACLServ NetworkACLVO lockedAcl = _networkACLDao.acquireInLockTable(ruleBeingMoved.getAclId()); List<NetworkACLItemVO> allAclRules = getAllAclRulesSortedByNumber(lockedAcl.getId()); validateAclConsistency(moveNetworkAclItemCmd, lockedAcl, allAclRules); - + NetworkACLItem networkACLItem = null; if (previousRule == null) { - return moveRuleToTheTop(ruleBeingMoved, allAclRules); - } - if (nextRule == null) { - return moveRuleToTheBottom(ruleBeingMoved, allAclRules); + networkACLItem = moveRuleToTheTop(ruleBeingMoved, allAclRules); + } else if (nextRule == null) { + networkACLItem = moveRuleToTheBottom(ruleBeingMoved, allAclRules); + } else { + networkACLItem = moveRuleBetweenAclRules(ruleBeingMoved, allAclRules, previousRule, nextRule); } - NetworkACLItem networkACLItem = moveRuleBetweenAclRules(ruleBeingMoved, allAclRules, previousRule, nextRule); VpcVO vpc = _vpcDao.findById(lockedAcl.getVpcId()); if (Objects.isNull(vpc)) { return networkACLItem;
