Not closing all open positions with magic number error

input double profitToClosePositions = 5;
input int magic_number = 1;
//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
int OnInit()
  {
//---
   
//---
   return(INIT_SUCCEEDED);
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
//---
   
  }
//+------------------------------------------------------------------+
//| Expert tick function                                             |
//+------------------------------------------------------------------+
void OnTick()
  {
//---
   
if(Profit()>= profitToClosePositions)
{
CloseOrders();
}
  }
//+------------------------------------------------------------------+

double Profit(){
double Prof=0;
for (int i=0; i<OrdersTotal(); i++) {
if (OrderSelect(i, SELECT_BY_POS, MODE_TRADES)) {
if(OrderMagicNumber() == magic_number){

Prof = Prof + OrderProfit()+ OrderCommission() + OrderSwap();

}
} 
}
return(Prof);
}


void CloseOrders(void)
{for( int i = 0 ; i < OrdersTotal() ; i++ ) {
      //We select the order of index i selecting by position and from the pool of market/pending trades
      RefreshRates();
      int value =OrderSelect( i, SELECT_BY_POS, MODE_TRADES );
      //If the pair of the order (OrderSymbol() is equal to the pair where the EA is running (Symbol()) then return true
      
         if(OrderType()==OP_BUY && OrderMagicNumber()==magic_number){
            OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_BID),3,clrWhite);
            return;
         }
         if(OrderType()==OP_SELL && OrderMagicNumber()==magic_number){
            OrderClose(OrderTicket(),OrderLots(),MarketInfo(OrderSymbol(),MODE_ASK),3,clrWhite);
            return;
         }
      
      
   }
   //If the loop finishes it mean there were no open orders for that pair
return;
}