Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added utility methods for synchronizing defined relationsips #2264

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Added utility methods necessary to help in properly synchronizing rel…
…ationships between tables.
ecruz165 committed May 2, 2023
commit e4eaf27c4185a1f8de427703505df1071d171ed9
Original file line number Diff line number Diff line change
@@ -191,4 +191,25 @@ public Set<Tag> getTags() {
public void setTags(Set<Tag> tags) {
this.tags = tags;
}

public void addCcParamEntity(CcParamEntity ccParamEntity) {
this.parameters.add(ccParamEntity);
ccParamEntity.setCohortCharacterization(this);
}

public void removeCcParamEntity(CcParamEntity ccParamEntity) {
this.parameters.remove(ccParamEntity);
ccParamEntity.setCohortCharacterization(null);
}

public void addCcStrataEntity(CcStrataEntity ccStrataEntity) {
this.stratas.add(ccStrataEntity);
ccStrataEntity.setCohortCharacterization(this);
}

public void removeCcStrataEntity(CcStrataEntity ccStrataEntity) {
this.stratas.remove(ccStrataEntity);
ccStrataEntity.setCohortCharacterization(null);
}

}
Original file line number Diff line number Diff line change
@@ -210,4 +210,25 @@ public Set<Tag> getTags() {
public void setTags(Set<Tag> tags) {
this.tags = tags;
}

public void addCohortAnalysisGenerationInfo(CohortAnalysisGenerationInfo cohortAnalysisGenerationInfo) {
this.cohortAnalysisGenerationInfoList.add(cohortAnalysisGenerationInfo);
cohortAnalysisGenerationInfo.setCohortDefinition(this);
}

public void removeCohortAnalysisGenerationInfo(CohortAnalysisGenerationInfo cohortAnalysisGenerationInfo) {
this.cohortAnalysisGenerationInfoList.remove(cohortAnalysisGenerationInfo);
cohortAnalysisGenerationInfo.setCohortDefinition(null);
}

public void addCohortGenerationInfo(CohortGenerationInfo cohortGenerationInfo) {
this.generationInfoList.add(cohortGenerationInfo);
cohortGenerationInfo.setCohortDefinition(this);
}

public void removeCohortGenerationInfo(CohortGenerationInfo cohortGenerationInfo) {
this.generationInfoList.remove(cohortGenerationInfo);
cohortGenerationInfo.setCohortDefinition(null);
}

}
Original file line number Diff line number Diff line change
@@ -187,4 +187,13 @@ public void setCreatedBy(UserEntity createdBy) {
public UserEntity getCreatedBy() {
return createdBy;
}

public void setCohortDefinition(CohortDefinition cohortDefinition) {
this.cohortDefinition = cohortDefinition;
cohortDefinition.addCohortGenerationInfo(this);
}
public void removeCohortDefinition(CohortDefinition cohortDefinition) {
cohortDefinition.removeCohortGenerationInfo(this);
this.cohortDefinition = null;
}
}
Original file line number Diff line number Diff line change
@@ -68,4 +68,6 @@ public int hashCode() {
return cohortDefinitionId + sourceId;
}



}
Original file line number Diff line number Diff line change
@@ -76,4 +76,14 @@ public void setResultFiles(List<AnalysisResultFile> resultFiles) {
this.resultFiles = resultFiles;
}

public void addAnalysisResultFile(AnalysisResultFile analysisResultFile) {
this.resultFiles.add(analysisResultFile);
analysisResultFile.setExecution(this);
}

public void removeAnalysisResultFile(AnalysisResultFile analysisResultFile) {
this.resultFiles.remove(analysisResultFile);
analysisResultFile.setExecution(null);
}

}
Original file line number Diff line number Diff line change
@@ -184,12 +184,10 @@ public void setLocked(final Boolean locked) {
}

public Set<CohortCharacterizationEntity> getCohortCharacterizations() {

return cohortCharacterizations;
}

public void setCohortCharacterizations(final Set<CohortCharacterizationEntity> cohortCharacterizations) {

this.cohortCharacterizations = cohortCharacterizations;
}

@@ -207,5 +205,16 @@ public void setStatType(final CcResultType statType) {

this.statType = statType;
}

public void addCohortCharacterizationEntity(CohortCharacterizationEntity cohortCharacterizationEntity) {
this.cohortCharacterizations.add(cohortCharacterizationEntity);
cohortCharacterizationEntity.getFeatureAnalyses().add(this);
}

public void removeCohortCharacterizationEntity(CohortCharacterizationEntity cohortCharacterizationEntity) {
this.cohortCharacterizations.remove(cohortCharacterizationEntity);
cohortCharacterizationEntity.getFeatureAnalyses().remove(this);
}

}

Original file line number Diff line number Diff line change
@@ -48,4 +48,15 @@ public List<ConceptSet> getConceptSets() {

return Objects.nonNull(this.conceptSetEntity) ? this.conceptSetEntity.getConceptSets() : Collections.emptyList();
}

public void addFeAnalysisCriteriaEntity(T feAnalysisCriteriaEntity) {
this.design.add(feAnalysisCriteriaEntity);
feAnalysisCriteriaEntity.setFeatureAnalysis(this);
}

public void removeFeAnalysisCriteriaEntity(T feAnalysisCriteriaEntity) {
this.design.remove(feAnalysisCriteriaEntity);
feAnalysisCriteriaEntity.setFeatureAnalysis(null);
}

}
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
package org.ohdsi.webapi.feanalysis.domain;


import javax.persistence.DiscriminatorValue;
import javax.persistence.Entity;

@@ -13,4 +14,5 @@ public FeAnalysisWithDistributionCriteriaEntity() {
public FeAnalysisWithDistributionCriteriaEntity(FeAnalysisWithCriteriaEntity analysis) {
super(analysis);
}

}
Original file line number Diff line number Diff line change
@@ -5,6 +5,7 @@
import javax.persistence.Lob;
import org.hibernate.annotations.Type;
import org.ohdsi.analysis.cohortcharacterization.design.FeatureAnalysis;
import org.ohdsi.webapi.cohortcharacterization.domain.CohortCharacterizationEntity;

@Entity
@DiscriminatorValue("not null")
@@ -31,4 +32,5 @@ public void setDesign(final String design) {

this.design = design;
}

}
Original file line number Diff line number Diff line change
@@ -216,4 +216,13 @@ public FeasibilityStudy setInclusionRules(List<InclusionRule> inclusionRules) {
this.inclusionRules = inclusionRules;
return this;
}

public void addStudyGenerationInfo(StudyGenerationInfo studyGenerationInfo) {
studyGenerationInfoList.add(studyGenerationInfo);
studyGenerationInfo.setFeasibilityStudy(this);
}
public void removeStudyGenerationInfo(StudyGenerationInfo studyGenerationInfo) {
studyGenerationInfoList.remove(studyGenerationInfo);
studyGenerationInfo.setFeasibilityStudy(null);
}
}
Original file line number Diff line number Diff line change
@@ -139,4 +139,8 @@ public FeasibilityStudy getStudy() {
public Source getSource() {
return source;
}

public void setFeasibilityStudy(FeasibilityStudy feasibilityStudy) {
this.study = feasibilityStudy;
}
}
6 changes: 6 additions & 0 deletions src/main/java/org/ohdsi/webapi/ircalc/ExecutionInfo.java
Original file line number Diff line number Diff line change
@@ -152,7 +152,13 @@ public IncidenceRateAnalysis getAnalysis() {
return analysis;
}

public void setAnalysis(IncidenceRateAnalysis incidenceRateAnalysis) {
this.analysis = incidenceRateAnalysis;
}

public Source getSource() {
return source;
}


}
Original file line number Diff line number Diff line change
@@ -130,4 +130,13 @@ public Set<Tag> getTags() {
public void setTags(Set<Tag> tags) {
this.tags = tags;
}

public void addExecutionInfo(ExecutionInfo executionInfo){
executionInfoList.add(executionInfo);
executionInfo.setAnalysis(this);
}
public void removeExecutionInfo(ExecutionInfo executionInfo){
executionInfoList.remove(executionInfo);
}

}
Original file line number Diff line number Diff line change
@@ -185,4 +185,25 @@ public Set<Tag> getTags() {
public void setTags(Set<Tag> tags) {
this.tags = tags;
}

public void addPathwayEventCohort(PathwayEventCohort pathwayEventCohort) {
this.eventCohorts.add(pathwayEventCohort);
pathwayEventCohort.setPathwayAnalysis(this);
}

public void removePathwayEventCohort(PathwayEventCohort pathwayEventCohort) {
this.eventCohorts.remove(pathwayEventCohort);
pathwayEventCohort.setPathwayAnalysis(null);
}

public void addPathwayTargetCohort(PathwayTargetCohort pathwayTargetCohort) {
this.targetCohorts.add(pathwayTargetCohort);
pathwayTargetCohort.setPathwayAnalysis(this);
}

public void removePathwayTargetCohort(PathwayTargetCohort pathwayTargetCohort) {
this.targetCohorts.remove(pathwayTargetCohort);
pathwayTargetCohort.setPathwayAnalysis(null);
}

}
Original file line number Diff line number Diff line change
@@ -78,4 +78,15 @@ public Set<RolePermissionEntity> getRolePermissions() {
public void setRolePermissions(Set<RolePermissionEntity> rolePermissions) {
this.rolePermissions = rolePermissions;
}

public void addRolePermissionEntity(RolePermissionEntity rolePermissionEntity) {
this.rolePermissions.add(rolePermissionEntity);
rolePermissionEntity.setPermission(this);
}

public void removeRolePermissionEntity(RolePermissionEntity rolePermissionEntity) {
this.rolePermissions.remove(rolePermissionEntity);
rolePermissionEntity.setPermission(null);
}

}
21 changes: 21 additions & 0 deletions src/main/java/org/ohdsi/webapi/shiro/Entities/RoleEntity.java
Original file line number Diff line number Diff line change
@@ -89,4 +89,25 @@ public Boolean isSystemRole() {
public void setSystemRole(Boolean system) {
systemRole = system;
}

public void addRolePermissionEntity(RolePermissionEntity rolePermissionEntity) {
this.rolePermissions.add(rolePermissionEntity);
rolePermissionEntity.setRole(this);
}

public void removeRolePermissionEntity(RolePermissionEntity rolePermissionEntity) {
this.rolePermissions.remove(rolePermissionEntity);
rolePermissionEntity.setRole(null);
}

public void addUserRoleEntity(UserRoleEntity userRoleEntity) {
this.userRoles.add(userRoleEntity);
userRoleEntity.setRole(this);
}

public void removeUserRoleEntity(UserRoleEntity userRoleEntity) {
this.userRoles.remove(userRoleEntity);
userRoleEntity.setRole(null);
}

}
11 changes: 11 additions & 0 deletions src/main/java/org/ohdsi/webapi/shiro/Entities/UserEntity.java
Original file line number Diff line number Diff line change
@@ -93,4 +93,15 @@ public UserOrigin getOrigin() {
public void setOrigin(UserOrigin origin) {
this.origin = origin;
}

public void addUserRoleEntity(UserRoleEntity userRoleEntity) {
this.userRoles.add(userRoleEntity);
userRoleEntity.setUser(this);
}

public void removeUserRoleEntity(UserRoleEntity userRoleEntity) {
this.userRoles.remove(userRoleEntity);
userRoleEntity.setUser(null);
}

}
20 changes: 16 additions & 4 deletions src/main/java/org/ohdsi/webapi/source/Source.java
Original file line number Diff line number Diff line change
@@ -17,10 +17,7 @@

import com.odysseusinc.arachne.execution_engine_common.api.v1.dto.KerberosAuthMechanism;
import java.io.Serializable;
import java.util.Collection;
import java.util.Date;
import java.util.List;
import java.util.Objects;
import java.util.*;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets leave this alone for now, since it's a little inconsistent now between the other explicit imports vs this one being wildcard.

import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
@@ -287,4 +284,19 @@ public Integer getId() {

return sourceId;
}
public void addSourceDaimon(SourceDaimon sourceDaimon) {
if (this.daimons == null) {
this.daimons = new ArrayList<>();
}
this.daimons.add(sourceDaimon);
sourceDaimon.setSource(this);
}

public void removeSourceDaimon(SourceDaimon sourceDaimon) {
if (this.daimons!= null) {
this.daimons.remove(sourceDaimon);
sourceDaimon.setSource(null);
}
}

}
Original file line number Diff line number Diff line change
@@ -94,4 +94,15 @@ public String getUserRoles() {
public void setUserRoles(String userRoles) {
this.userRoles = userRoles;
}

public void addRoleGroupEntity(RoleGroupEntity roleGroupEntity) {
this.roleGroupMapping.add(roleGroupEntity);
roleGroupEntity.setUserImportJob(this);
}

public void removeRoleGroupEntity(RoleGroupEntity roleGroupEntity) {
this.roleGroupMapping.remove(roleGroupEntity);
roleGroupEntity.setUserImportJob(null);
}

}