scope should probably be page instead of view.
Updating the table directly is dangerous... Because im guessing you want the tables data to be filled by a binding it self. This will cause the binding to rewrite the data, so the users edit will be lost. Either update the sql directly here, or if they have to press a confirm button somewhere first, save all the changes seperately and make your table combine the pulled data and the changes
I dont think this will do anything because the event comes from the combobox and not from the tables onEdit, so you can delete this.