Thanks Charlie;

I'll take your advice into account, especially if I start getting weird results...

In the meantime, to avoid making major changes to the framework I use (not created by me), I will just take the risk. So far, it seams to work.

That would be the "extenuating circumstances", I would say. Adding a custom method to the parent-of-all-dataforms seems to make sense, but I find this is too much trouble in the existing framework.

Hilmar.

>Hi Hilmar,
>Just to make it interesting and because I hear no dissenting voices...
>It seems to me that your design is flawed any time you resort to SET DATASESSION TO. Each object is responsible to know and control what goes on within it. Messages are passed to objects and the objects respond. Here I would want to send a message to the active form and tell it to reorder itself, whatever that means in the specific form's case. A toolbar button can be generic in that it knows how to send the message to any form. But when it assumes it knows how to change the datasession and set order to and refresh, etc.--well, that's quite a bit of responsiblity for a button that has incomplete knowledge of all possible forms.
>I recognize that some of the VFP foundation classes work this way, but I think the code is misplaced. Make a common interface for your forms. Talk to them through that interface. Don't sneak around behind the form's back to muck around with its innards. (obviously using language here with negative connotation)
>
>http://home.kscable.com/cschreiner/OutFoxed16.html
>
>Of course, you make have extenuating circumstances, but as a general goal, why not try to use object.method() to do work in/on an object?
>
>As to the reliablility of SET DATASESSION TO, it just depends. If the toolbar was created in the default datasession, and you have a combobox control or grid in the default datasession, your SET DATASESSION TO isn't very good. A refresh during the time you've changed the DataSessionID will be bad for that control. You can't control the internal VFP refresh, IFAIK. If you don't have those type of controls in a default datasession, you should be fine.