search

EcsRx / ecsrx.unity

A simple framework for unity using the ECS paradigm but with unirx for fully reactive systems.
MIT License
415 stars 65 forks source link

IEnumerableExtensions.GetApplicableSystems does not return all applicable systems #45

Closed ceinicke closed 7 years ago

ceinicke commented 7 years ago

Line 42 should be "return systems.Where(x => x.TargetGroup.TargettedComponents.All(y => componentTypes.Contains(y)));" This way it checks that all targeted components are in the given list of components.

Bezarius commented 7 years ago

that's right

grofit commented 7 years ago

Been out of the country, just raise a PR for it.

grofit commented 7 years ago

Have added a fix for this and a test to verify behaviour for future versions, this will be released in a minor release shortly.