--- Mystik Gotan <[EMAIL PROTECTED]> wrote:
> Hi.
> 
> I'm a young (14, dutch) Perl Programmer (or so however you may call, some 
> call me, some might don't ;)). Anyway, I'm getting a 500 error on my script. 
> I changed some things, like print content type in subs, and I put the HTML 
> form in the script. But no changes for the ISE error. Hope you guys can help 
> me :-)

Hallo Mystik,

If you can tell us what your script is trying to do, that might help.

I see that you are using warnings, strict, and taint checking.  This is a good start 
and it's nice
to see in people who are new to programming.

Some issues:

  use vars qw($q, $query_name, $query_email, $query_message);

Note that when you use the 'qw' operator, you should not use commas.  This is better 
written as:

  use vars qw($q $query_name $query_email $query_message);
 
However, I would probably skip this and just declare the variables with 'my' the first 
time that
you use them:

  my $q           = new CGI;
  my $query_name  = $q->param('name');
  my $query_email = $q->param('email');

And so on ...

I also notice that you are printing your content-type header multiple times.  You'll 
only need to
print in once.  You can also use your CGI object for this:

  print $q->header;

See my CGI course for more information on this (link at bottom of email).

Good luck with your programming!

Cheers,
Ovid


=====
"Ovid" on http://www.perlmonks.org/
Web Programming with Perl:  http://users.easystreet.com/ovid/cgi_course/
Silence Is Evil: http://users.easystreet.com/ovid/philosophy/decency.txt

__________________________________________________
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com

-- 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to