X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=ecomp-portal-BE-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fonap%2Fportalapp%2Fportal%2Fdomain%2FEPUserApp.java;h=ebdf85429d9308875f07a3b7500f673d6675a45d;hb=abf55f3469a7942bab06961d55a5107777f5391f;hp=d644c9980ea6e927ba48e36228b67f37a0009b3c;hpb=b436ad77af84bab4af0dd93a0190b36065c9950f;p=portal.git diff --git a/ecomp-portal-BE-common/src/main/java/org/onap/portalapp/portal/domain/EPUserApp.java b/ecomp-portal-BE-common/src/main/java/org/onap/portalapp/portal/domain/EPUserApp.java index d644c998..ebdf8542 100644 --- a/ecomp-portal-BE-common/src/main/java/org/onap/portalapp/portal/domain/EPUserApp.java +++ b/ecomp-portal-BE-common/src/main/java/org/onap/portalapp/portal/domain/EPUserApp.java @@ -97,7 +97,7 @@ public class EPUserApp extends DomainVo implements java.io.Serializable, Compara } public Integer getPriority() { - return (this.priority == null) ? 1 : priority; + return priority; } public void setPriority(Integer priority) { @@ -116,8 +116,7 @@ public class EPUserApp extends DomainVo implements java.io.Serializable, Compara return (otherUserIdIsSameAsThisUserId(castOther)) && (otherAppIdIsSameAsThis(castOther)) - && (otherRoleIsSameAsThis(castOther)) - && (otherPriorityIsSameAsThis(castOther)); + && (otherRoleIsSameAsThis(castOther)); } public int hashCode() { @@ -126,22 +125,18 @@ public class EPUserApp extends DomainVo implements java.io.Serializable, Compara result = 37 * result + (int) (this.getUserId()==null ? 0 : this.getUserId().intValue()); result = 37 * result + (int) (this.getApp().getId()==null ? 0 : this.getApp().getId().intValue()); result = 37 * result + (int) (this.getRole().getId()==null ? 0 : this.getRole().getId().intValue()); - result = 37 * result + (int) (this.priority==null ? 0 : this.priority); return result; } public int compareTo(Object other){ EPUserApp castOther = (EPUserApp) other; - Long c1 = (this.getUserId()==null ? 0 : this.getUserId()) + (this.getApp()==null||this.getApp().getId()==null ? 0 : this.getApp().getId()) + (this.getRole()==null||this.getRole().getId()==null ? 0 : this.getRole().getId()) + (this.priority==null ? 0 : this.priority); - Long c2 = (castOther.getUserId()==null ? 0 : castOther.getUserId()) + (castOther.getApp()==null||castOther.getApp().getId()==null ? 0 : castOther.getApp().getId()) + (castOther.getRole()==null||castOther.getRole().getId()==null ? 0 : castOther.getRole().getId()) + (castOther.priority==null ? 0 : castOther.priority); + Long c1 = (this.getUserId()==null ? 0 : this.getUserId()) + (this.getApp()==null||this.getApp().getId()==null ? 0 : this.getApp().getId()) + (this.getRole()==null||this.getRole().getId()==null ? 0 : this.getRole().getId()); + Long c2 = (castOther.getUserId()==null ? 0 : castOther.getUserId()) + (castOther.getApp()==null||castOther.getApp().getId()==null ? 0 : castOther.getApp().getId()) + (castOther.getRole()==null||castOther.getRole().getId()==null ? 0 : castOther.getRole().getId()); return c1.compareTo(c2); } - private boolean otherPriorityIsSameAsThis(EPUserApp other){ - return (this.priority==null && other.getPriority()==null) || this.getPriority().equals(other.getPriority()); - } - + private boolean otherRoleIsSameAsThis(EPUserApp other){ return this.getRole().getId().equals(other.getRole().getId()); }