Appion

Just finished the database validation checking.

To test:
in wizard step 3.
Try to enter wrong information for all the user input, The wizard will display which input value is wrong and user will have to fix them before they can move to the next step.

Thanks.

Eric

Eric, looks like it will work well.
I have one comment. You are checking the db server and 2 databases (Leginon and Project) all within a single function, validateDB(). I think it might make it more general purpose to separate this into two functions, one to check the server and another to check a single db. Then we could add or remove database validations, without editing your class. It may be a little less efficient since you will want to include a db server check in each db validation, but I think the added flexibility would be worth it.

What do you think?
I know you want to be done with this file, so if you think it is a good idea, but don't feel like coding it, I can add it to my task list.

for rewrite the validateDB functionin formValidator class. I will make the other feature issue for it.
Because it might need to involve with writing an extend class to the formValidator class.

For now, test all the possible cases.

Thanks.
Eric