|
Schwanke
New Member
77 Posts |
 Posted - 25 August 2002 : 00:16:09
|
In post_info.asp around line 1202 and 1451 you have the code:
if (left(lcase(txtAddress), 7) <> "http://" and left(lcase(txtAddress), 8) <> "https://" and left(lcase(txtAddress), 8) <> "file:///") and (txtAddress <> "") then
Err_Msg = Err_Msg & "<li>You Must prefix the Address with <b>http://</b>, <b>https://</b> or <b>file:///</b></li>"
end if
But prior to thate code you have code that gives an error "You must enter an address for the new url."
Wouldnt it make more sense to oput the the last test of txtAddress <> "" in the previos set with its brother testing for " "?
I only say this cuz some mods I am doing I have to allow for adding local pages to the web links and so I am commenting out these tests. But I still dont want a user submiting a blank name. (And I would think I would want the error to be the previos error anyway.)
It just seems logically correct this way to me. Feel free to explain it to me differently.
:)
Its not gunna break anything as is but if you ever edit or move that part of the code you might loose the test for "" or not remember where you put it. :)
|
<>< |
Edited by - Schwanke on 25 August 2002 00:58:05 |
|
Topic Locked
