Author Bradley Edwards on 01-28-2002 at 00:11 (EST)
I just found a bug in the import function of the utorekeeper. No big deal, I couldn't figure out why everytime I would Import my db file, the text for the bottom HTML was missing.
in the "import_data_file" sub
there is only 22 items (0-21), it was missing the last item on import.
I just changed the last lines, from:
$key_value_pairs) == 1)
And all is great. My version, is "1.81"
|Messages In This Discussion|
| 1. RE: BUG...|
Author Bill Weiner on 01-28-2002 at 06:31 (EST)
|I took a quick look at that subroutine and you may be right. We'll check into some more. Thanks for bringing that to our attention.|
If it is missing that last field on the "Import Utility", it probably hasn't been noticed since that last field is the rarely-used "BOTTOM HTML" field... but we'll check into it.
| 2. Another Bug|
Author Bradley Edwards on 03-19-2002 at 20:00 (EST)
Here are 2 more things that I found within uStoreKeeper. Nothing major, the first is a bug that lets someone submit an order without putting in thier 'State' - No 'State' no tax!
Under 'sub review_order', I added the following:
if (!($customer_state =~ /S/))
&missing_input("Missing Input","Please enter your *State.");
between City & Zip Code.
And the other bug, will let someone submit thier 'E-check' order without inputting the 'Transit Code'.
Under the 'sub electronic_check_order', added the following:
if (!($echeck_transit_code =~ /S/))
&missing_input("Missing Input","You must enter the Transit Number.");
after all the others.
| 3. RE: Another Bug|
Author Bill Weiner on 03-19-2002 at 22:02 (EST)
|Actually, we purposely do not require the "state" because customers from other countries do not have "states".|
And same goes for the check "transit code". Based on the electronic check specs, that is not always a required field.
| 4. RE: Another Bug|
Author Bradley Edwards on 03-20-2002 at 15:50 (EST)
In my personal situation, I would probably not be dealing with other countries, so I would rather have them be required.