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
Show file tree
Hide file tree
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
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -68,4 +68,6 @@ public int hashCode() {
return cohortDefinitionId + sourceId;
}



}
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Up @@ -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;
}

Expand All @@ -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
Expand Up @@ -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;

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

}
Original file line number Diff line number Diff line change
Expand Up @@ -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")
Expand All @@ -31,4 +32,5 @@ public void setDesign(final String design) {

this.design = design;
}

}
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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
Expand Up @@ -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;
Expand Down Expand Up @@ -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
Expand Up @@ -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);
}

}