Wow, good catch, and I agree. Always use the factory manager in managers to avoid circular dependencies. That's exactly what we use it for.