>>>I would make GetShopOrderDetails static and call it with:
>>>

>>>dtShopOrder = ERP.GetShopOrderDetails(txtSerialNum.Text);
>>>

>>
>>Why static? Isn't an instance the desired return value?
>
>Because you wouldn't need to instantiate ERP in order to call GetShopOrderDetails()
>
>This is one object less to create and one object less to garbage collect

Thanks. That famous C trickery? ;-)