I took a closer look at your script and you don't need CGI::Vars at all. You may want to read the documentation on CGI.pm. Your missing a lot of oppurtunities. Here are few corrections that may have you up and going quickly.
On this line we're creating %params.CGI.pm has a function for this: param. To access any parameter, just call param('any'). To use it as a function we need to change use CGI; to use CGI qw/param/;
Now that we can access param directly, let's make a few changes.
becomes:my ($command) = $params{'command'}; touch() if ($command eq "touch"); addlink() if ($command eq "add");
---------touch() if param('command') eq 'touch'; addlink() if param('command') eq 'add';
becomes:errorpage("You must fill in both fields!") if (($params{'name'} eq "") || ($params{'url'} eq ""));
---------errorpage('You must fill in both fields!') if param('name') eq '' || param('url') eq '';
In reply to Re: a quick stab at a public links page
by CharlesClarkson
in thread a quick stab at a public links page
by Anonymous Monk
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |