10-29-2007, 10:14 AM
|
#4 (permalink)
|
|
Super Moderator
Join Date: Sep 2007
Posts: 165
Thanks: 0
|
I think what Karl is refering too, is that you do not have any clarity in your error reporting:
PHP Code:
if (empty($this->m_name) || empty($this->m_email) || empty($this->m_artist) || empty($this->m_album) ||
empty($this->m_release) || empty($this->m_song) || empty($this->m_lyrics)) {
$szError .= 'Please fill in all required fields.<br />'."\n";
$bError = true;
}
Errors like "You did not fill in the lyrics column" would be better.
I would hazard a guess that Karl will recommend an external validation class, as would i :) |
|
|
|