FLUID-3006
Laurel A. Williams
laurel.williams at utoronto.ca
Tue Jul 7 16:39:41 UTC 2009
The following JIRA addresses Colin's point below about processPostVariables
http://issues.fluidproject.org/browse/FLUID-3006
I've committed the changes to address this (commit 7527) if Colin and
others would like to check it out.
Laurel
>>
>> I notice that you always return true from your processPostVariables()
>> method, but then this value is checked in case any errors occurred.
>> Looks like a bug to me, or perhaps something you haven't had a chance
>> to implement yet.
>>
>> $successPost = processPostVariables();
>> if (!$successPost)
>> {
>> returnError("Cannot process input variables");
>> exit (1);
>> }
If you look carefully at processPostVariables line 37, you will note
that if the post variable does not exist, the function returns false.
However you are correct that I didn't check the other post variables and
return false if they had a problem. Thx for catching that.
--
Laurel A. Williams
Adaptive Technology Resource Centre
University of Toronto
More information about the fluid-work
mailing list