-
Bug
-
Resolution: Unresolved
-
Minor
-
None
-
NEW
-
NEW
In:
org.optaplanner.core.config.heuristic.selector.move.generic.ChangeMoveSelectorConfig#buildUnfoldedMoveSelectorConfig(org.optaplanner.core.config.heuristic.policy.HeuristicConfigPolicy)
This code is potentially bad:
@Override
protected MoveSelectorConfig buildUnfoldedMoveSelectorConfig(HeuristicConfigPolicy configPolicy) {
...
entityDescriptors = configPolicy.getSolutionDescriptor().getGenuineEntityDescriptors();
...
for (EntityDescriptor entityDescriptor : entityDescriptors) {
...
} else {
variableDescriptorList.addAll(entityDescriptor.getGenuineVariableDescriptors()); // Should be declared only? Now its effective.
}...
}
if Cat extends Animal and Animal has a genuine planning variable "foo" and both Cat and Animal are a genuine planning entity, then "foo" is added twice I think.