source code hygiene, better method naming and variable naming
Signed-off-by: Michael Hirsch <michael.hirsch@bosch-si.com>
This commit is contained in:
@@ -190,7 +190,7 @@ public class ControllerManagement {
|
|||||||
if (target == null) {
|
if (target == null) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
return actionRepository.count(ActionSpecifications.hasTargetArtifactAssigned(target, localArtifact)) > 0;
|
return actionRepository.count(ActionSpecifications.hasTargetAssignedArtifact(target, localArtifact)) > 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
@@ -47,14 +47,14 @@ public class ActionSpecifications {
|
|||||||
* assigned
|
* assigned
|
||||||
* @return a specification to use with spring JPA
|
* @return a specification to use with spring JPA
|
||||||
*/
|
*/
|
||||||
public static Specification<Action> hasTargetArtifactAssigned(final Target target,
|
public static Specification<Action> hasTargetAssignedArtifact(final Target target,
|
||||||
final LocalArtifact localArtifact) {
|
final LocalArtifact localArtifact) {
|
||||||
return (actionRoot, query, cb) -> {
|
return (actionRoot, query, criteriaBuilder) -> {
|
||||||
final Join<Action, DistributionSet> dsJoin = actionRoot.join(Action_.distributionSet);
|
final Join<Action, DistributionSet> dsJoin = actionRoot.join(Action_.distributionSet);
|
||||||
final SetJoin<DistributionSet, SoftwareModule> modulesJoin = dsJoin.join(DistributionSet_.modules);
|
final SetJoin<DistributionSet, SoftwareModule> modulesJoin = dsJoin.join(DistributionSet_.modules);
|
||||||
final ListJoin<SoftwareModule, LocalArtifact> artifactsJoin = modulesJoin.join(SoftwareModule_.artifacts);
|
final ListJoin<SoftwareModule, LocalArtifact> artifactsJoin = modulesJoin.join(SoftwareModule_.artifacts);
|
||||||
return cb.and(cb.equal(artifactsJoin.get(LocalArtifact_.id), localArtifact.getId()),
|
return criteriaBuilder.and(criteriaBuilder.equal(artifactsJoin.get(LocalArtifact_.id), localArtifact.getId()),
|
||||||
cb.equal(actionRoot.get(Action_.target), target));
|
criteriaBuilder.equal(actionRoot.get(Action_.target), target));
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user