19 replies to this topic

[DelphiToday]

Hi Bojan,

 

When I save my Columns and Load it back The Columns are not showing correctly like I want to. 

(36 columns and about 18 not visible)

 

I must load it twice and then it will be alright. So there is a problem in de Update procedure?

procedure TfrmFinalDupeChecker.btnLoadCFGClick(Sender: TObject);
begin
  NextVirtualGrid.Serialize.LoadFromINI(ChangeFileExt(Application.ExeName, '.ngr'));
  NextVirtualGrid.Columns.Update;

  // And Again

  NextVirtualGrid.Serialize.LoadFromINI(ChangeFileExt(Application.ExeName, '.ngr'));
  NextVirtualGrid.Columns.Update;
end;

Can you fix this?

I have no demo because when less Columns and probably less/more visible there is no problem.

it's random that it goes wrong.

 

Regards,

Eduard.

 

 

[DelphiToday]

Hello?

[Boki (Berg)]

Hello Eduard,

 

I will inspect this and fix it as soon is possible.

[DelphiToday]

Thank you.

Will you show solution in this topic because I need it fast?

[DelphiToday]

Any Luck?

[Boki (Berg)]

Hello,

 

I didn't managed to reproduce it yet. While I tested I Saved and Loaded different number of columns and that caused mismatch. I will try to test it little bit more.

[DelphiToday]

What I did is in Design time add 5 fields (0..4). The rest is dynamically.

 

 BOKI 2024-03-08_094425.jpg   46.88KB   0 downloads

 

Thinking: If I remove the last icon field (4) there is no problem by my previous version.

Maybe accidentally.

 

In my FinalPlayer I use only two design time Icon field and the rest dynamically and no problem there.

 

Maybe this can help?

[Boki (Berg)]

Hello,

 

I think that there should be no problems with design and run-time columns. Only thing that is required is that the order and count of Saved and Loaded columns are equal.

[DelphiToday]

I always save it when ended the program and load when started.

Always after INIT the NextGrid.

[DelphiToday]

Found kind of error. First change this in your 'procedure TNxGridSerialization.ReadFromINI'

 

// NOT CORRECT? Columns[i].Visible := IniFile.ReadBool('Visible Columns' + ID, 'Column' + IntToStr(i), True);
Must be:  Columns[i].Visible := IniFile.ReadBool('Visible Columns' + ID, 'Column' + IntToStr(i), Columns[i].Visible); // <<<< Columns[i].Visible (not default TRUE)

 

Because I create some columns with default visible FALSE. As you can see you set this at first to TRUE when reading non existing INI file.

 

Now to the problem...

I created a demo and can reproduce the fault.

 

 VirtualGrid_TEST.zip   2.46KB   5 downloads

 

Try it several times and some columns are not correct after a re-run.

 

Hope you can find this little problem

 

 

[Boki (Berg)]

Hello,

 

This is probably the problem. Does it work well after you added this fix?

 

I will include it in the official releases then.

[DelphiToday]

Hi,

 

No this is not the problem it's only a small fix. The visibility was not the main problem but de order of columns after a restart. 

 

Just check the demo and move some columns and close it and start up again than you can see that the order is not correct.

In de demo i hide some columns so you must run it after the first fix.

[DelphiToday]

Here is a screenshot.

 

 BEFORE-2024-03-16_142156.jpg   11.39KB   1 downloads

 

Now I close the demo and start it up again.

 

 AFTER-2024-03-16_142302.jpg   12KB   1 downloads

 

Now you can see that the column order is different. 

 

Hope it's now more clearly.

[DelphiToday]

Please let me now ?

[Boki (Berg)]

Hello,

 

I have downloaded your demo project. It will take some time to solve the issue.

[DelphiToday]

And have you been able to find the problem yet?

[Boki (Berg)]

Hello Eduard,

 

I have added the visibility fix, but not yet managed to solve the position problem. I’m hoping to fix it for the upcoming release.

[DelphiToday]

We will wait

[Boki (Berg)]

Hello,

 

I think that I have fixed it. You can re-download latest PRE release from the users section.

[DelphiToday]

I update the file 'NxGridSerialization6.pas' and seems to be good now.

Thank you