Re: [phpesp-dev] Bulk upload patch

[Matthew G. <gr...@mu...>]

The only fields I set as required are the fields that are currently
required in the single Respondent/Designer Add forms.
Which are:
Username, Password and Group.

Whoops...I neglected to do any validation on the Expiration field, but
it should at least validate as much as the single Add forms.  I'll fix
that.

As for the other fields, do we want to be more strict with validation
than the single Add forms?

And thanks again for taking the time to look at this stuff.  The more
eyes the better.


On Thu, Jul 18, 2002 at 10:08:07AM +1000, Kon Angelopoulos wrote:
> Matthew,
> 
> I've finished testing your patch and I've found the following bugs:
> 1. Bulk Importing Respondent Details:
> 	I tested this functionality with a correct list of respondents and it worked 
> well. I then changed the realm to a non existent one and as expected, it 
> displayed the table with the failed uploaded rows.
> I then removed the email address from some of the respondents and it accepted 
> them without a problem. (should this be the case?).
> I them also removed the disabled field from some of the respondents and it 
> also accepted them without reporting any errors ( i don't think that this 
> should be the case).
> I then changed the formatting of the exipration date from year/month/day to 
> year/day/month. It accepted all these without any errors ( placing 000000000 
> into the database). I don't think this should happen.
> 
> 2. Bulk upload of Designers:
> 	I tested this with a correct list of designers and it worked well.
> I then tried uploading the respondents tab delimited file in place of the 
> designers tab delimited file and it accepted it without any errors. This 
...snip...

-- 
one of the friendly folks in the IT Lab.
--------------------------------------\
The IT Lab (http://www.itlab.musc.edu) \____________________
Probably the world's premier software development center.
Serving: Programming, Tools, Ice Cream, Seminars