Broadcast transformChanged() to attached entities

Attached light now moves with the parent entity, which did not happen before
because the attached entity's transformChanged() method was never called to
invalidate its localToWorld when the parent entity moved.

radiantcore/entity/EntityNode.cpp

@@ -148,6 +148,16 @@ void EntityNode::createAttachedEntities()
     );
 }
 
+void EntityNode::transformChanged()
+{
+    Node::transformChanged();
+
+    // Broadcast transformChanged to all attached entities so they can update
+    // their position
+    for (auto attached: _attachedEnts)
+        attached->transformChanged();
+}
+
 void EntityNode::onEntityClassChanged()
 {
 	// By default, we notify the KeyObservers attached to this entity

radiantcore/entity/EntityNode.h

@@ -103,6 +103,7 @@ class EntityNode :
 	// IEntityNode implementation
 	Entity& getEntity() override;
 	virtual void refreshModel() override;
+    void transformChanged() override;
 
 	// RenderEntity implementation
 	virtual float getShaderParm(int parmNum) const override;

test/Entity.cpp

@@ -602,10 +602,6 @@ TEST_F(EntityTest, RenderAttachedLightEntity)
     RenderFixture rf(true /* solid mode */);
     rf.renderSubGraph(torch);
 
-    // The node visitor should have visited the entity itself and one child node (a
-    // static model)
-    EXPECT_EQ(rf.nodesVisited, 2);
-
     // There should be 3 renderables from the torch (because the entity has a
     // shadowmesh and a collision mesh as well as the main model) and one from
     // the light (the origin diamond).