Close position within OnPositionClosed event

Hey Guys :)

I would like to close other positions inside OnPositionClosed event.
EXAMPLE:

protected override void OnPositionClosed(Position closedPosition)
{
       foreach (var p in Positions)
    {
       if(closedPosition.SymbolCode == p.SymbolCode)
       {
               CloseOldPosition(p.Id);
       }
    }
}

Would it be good practice?