Bug #938
closed"Required" fields currently must be > 2 characters -- lastName=Ng fails
0%
Description
There are currently 2 fields in this form that Bob wanted deleted at one time.
They are :
What use do you anticipate making the VegBank archive? name="useVegbank"
Briefly describe your prior experience with other vegetation plot database
and/or analytical tools name="plotdbDoc"
These need to be deleted from this html page, from the User database, and from
the servlet that deals with this page
(vegclass\veg_plot\exp\servlet\usermanagement-servlet\UserManagementServlet.java)
Additionally, this class contains the method : handleCertification
that deals with validating the data from the form and making sure required
fields are filled in:
Line 426 (currently):
// check that the required paramteres are upto snuff
if ( surName.length() > 2 && givenName.length() > 2 && phoneNumber.length()
&& degreeInst.length() > 2 && currentInst.length() > 2 && esaPos.length()2 && currentCertLevel.length() > 0
&& vegAnalysisDoc.length() > 2 && submittedEmail.length() > 20 && vegSamplingDoc.length() > 2
&& vegSamplingDoc.length() > 2 && vegAnalysisDoc.length() > 2 &&
usnvcExpDoc.length() >2
&& vegbankExpDoc.length() > 2 && useVegbank.length() > 2 &&
plotdbDoc.length() > 2)
My test certification failed, showing a curious html page that says some of my
fields are required and I didn't fill them in. However, I did fill them if, but
they were short (i.e. Name was M Lee). These >2 conditions should probably be
rewritten as >0. After I saw this and lengthened fields, the certification
worked fine.
See attached document for the error message I received.
Files
Updated by Michael Lee almost 22 years ago
We no longer want to delete the 2 fields in question. These can be left as they
are. The only change necessary with this bug now is changing the java class to
make sure that required fields are accepted if they have characters entered into
the field. See the html doc that I attached for the confusing error message I
received. Something to the effect of "The following fields are required, but
you left some blank:" then all fields listed were filled in. Some were just
too short for the java code.
Updated by John Harris almost 22 years ago
This bug is fixed on the test system.