For loop refusing to loop through ALL orders in the ledger. - page 2

[Paul Anscombe]

Judging by this code, the advisor always changes the text of only one button.

TheFlyingPrussian #:
That is correct

As you are only displaying one order per chart, as it loops through the orders it will always keep over-writing that info until it has displayed the last order it comes to for that chart....

you have confirmed this, so unclear what you are expecting otherwise?

[William Roeder]

Simplify your code

   for(int i = OrdersTotal(); - 1; i >=0; i--) if(
      OrderSelect(i, SELECT_BY_POS, MODE_TRADES)
   && OrderMagicNumber() == Period()
   && OrderSymbol()      == Symbol()
   ){
                  ChartButtonStaticAttributes(); //No issues here
                  ChartButtonFluidAttributes(clrGreen,clrCrimson,clrGray);   
   }