Correct me please, Click to display row data from grid into text-boxes

[Yosua]

Hi,
I've been trying to show a row data from a grid back into text-boxes,
Here's what I do

Form Method "show_data"

THISFORM.txt_id.value = tb_member.id
THISFORM.txt_name.value = tb_member.name
THISFORM.txt_birthday.value = tb_member.birthday
THISFORM.txa_address.value = tb_member.address

Grid Click

thisform.show_data

And my problem is I only return the data from top row to text-boxes, and whenever I click the other row, it shows the data from the top row. Has anyone know where I did wrong and how to fix it?
Any suggestions are appreciated and sorry for bad english.

Regards
Yosua

 

[Olaf Doschke]

Is tb_member really the table shown in the grid? Also Grid.click may be too early, record pointer has moved to the clicked line in Grid.AfterRowColChange Event, that happens, after the row has changed.

Why not use Textbox.Controlsources? That way you don't need any code, but perhaps do THISFORM.Refresh() in Grid.AfterRowColChange. That's all.

You set txt_id.Controlsource = "tb_member.id" etc. and the textboexes are bound to these table fields, like the grid is bound to the whole table via it's Recordsource.

Bye, Olaf.

 

[Mike Lewis]

Yosua, you need to do two things:

1. Put the call to thisform.show_data in the grid's AfterRowColChange, not in its Click; and

2. Put THISFORM.Refresh at the end of the Show_Data method.

I think that'lll fix it.

Mike

__________________________________
Mike Lewis (Edinburgh, Scotland)

Visual FoxPro articles, tips, training, consultancy

 

[Yosua]

Thanks Olaf for the right solution, but if I may ask some more stupid question,
where should I put txt_id.Controlsource = "tb_member.id" should I set it on properties> and how it should work when I select a row on the grid to edit or delete?
Sorry for totally clueless about this.

 

[Yosua]

Thanks Mike, I already have these in the grid's AfterRowColChange

LPARAMETERS nColIndex
thisform.tampil_data

I've tried it and it didn't work for me, did I do something wrong?

 

[Yosua]

Oops sorry my bad.

LPARAMETERS nColIndex
thisform.show_data

 

[Yosua]

After some trial and error, Mike's solution is working for me.
Thanks Mike you're a lifesaver. I should learn about control source too, thanks Olaf

 

[Mike Lewis]

Yosua, Glad to hear it's working. I used to teach something similar to this on my courses. The thing that a lot of people got wrong was to forget to refresh the form.

Mike

__________________________________
Mike Lewis (Edinburgh, Scotland)

Visual FoxPro articles, tips, training, consultancy

 

[Olaf Doschke]

Yes, you can set controlsources in property window at design time. If you have a Dateenvironment with the table, simply drag from a single field to the form, and you'll get a textbox with controlsource already set.

Bye, Olaf.