For a month ago a guy asked me to make a script for him.
I started it and was almost finish, but then I ran into some errors.
Before I fixed it the guy said he diden't need the script for the moment.
Now, he has contacted me again and wants me to finish the script.
I have it, but the errors are still there.
Since I haven't been working with the script(and perl) for a month, I am confused.
I don't know where the errors are.
If anyone could take a look at the script and tell me it would be great.
Some of the text is in norwegian, but the script is normal perl.
The script is for a computerparty.
The error I keep getting is

Software error:
Execution of dataparty.cgi aborted due to compilation errors.

I don't know what to do??
The script is a bit confusing so I hope you understand.
Please let me know if anyone know what the problem is!

- Kjetil

Does it tell you which line the error is on?

Did you upload the script using ASCII transfer mode and made sure that you CHMODed the file to 755.

HTH.

- wil

And did you test it via telnet:

perl -w script.cgi
perl -c script.cgi

That would produce more detailed errors. Or how about looking at your Perl Error Logs?
========================================
Buh Bye!

Cheers,
Me

You should really use CGI.pm for form processing.

Also I notice you are using:

my($errmsg) = @_;

...you only need shift there.

Re the error: You have a missing right bracket for starters line 153

Okey, let me resume.
I did upload in ASCII and chmoded 755. (I'm not stupid!!)
And my server does not have telnet acess.
And I don't want to use CGI.pm.
This is one of my first scripts and I don't want to "cheat" on them!
And that thing RedRum said:
Can you describe it better??
Please!!

He means you can use;

my $errmsg = shift;


Andy (mod)
andy@ultranerds.co.uk

---

Want to give me something back for my help? Please see my Amazon Wish List
GLinks ULTRA Package | GLinks ULTRA Package PRO
Links SQL Plugins | Website Design and SEO | UltraNerds | ULTRAGLobals Plugin | Pre-Made Template Sets | FREE GLinks Plugins!

You only need @_ when passing more than one argument to your error routine.

>>This is one of my first scripts and I don't want to "cheat" on them! <<

Cheating by using CGI.pm? .....I'd saying asking for us to fix your bugs is more of a cheat

I still get the same error!!
I'll give you the address.
http://www.ungdoms-start.f2s.com/cgi-bin/dataparty.cgi
There is the script.
Go there and the error will show up.
Don't know if that will help......

Did you do as suggested by Eliot and Wil and also check line 153?

Looks like you forgot a curly brace somewhere. I did see an if statement that wasn't closed but didn't look at it close enough to see where it should have been closed.



Regards,

Charlie

Hi "perlman"

I wasn't implying that you were stupid - just trying to catch a very common error.

BTW: Using modules is *not* cheating. It's the most clever thing you could do with your time and effort.

Hmm, weird Apache setup.

Cheers


- wil

OK folks!
I got it going!
I had to remove the two if codes:


Ok here's the deal.
The thing I wanted was that if the place has been picked by another one it would come up an error.
I tried to do it with a array, but that diden't work out.
What can I do???
I also want to check their name and see if the have signed in earlier.
How can do that??
Anyone have a clue??

And that cheat thing.
Okey, you can say that I cheat if I ask you, but asking is important to learn things and I don't want to use that CGI.pm thing.
Don't know why.
Just don't like the idea to use a huge script that somebody else has written. It takes the fun out of it.
(If you are a newbie!)

>>I tried to do it with a array, but that diden't work out. <<

Are you referring to:

if ($FORM{'plasser'} = '@plass') {

??

If so then surrounding @plass with ' ' is changing the way perl interprets it. You'd need:

my $plasser = $FORM{plasser};

if (grep { /^\Q$plasser\E$/ } @plass) {

error()

}

Ok. I tried what you wrote RedRum, but it diden't work.
I'll post the script so you can see.
I sorry for asking you all the questions.
It's wrong of me to bother people to help me with a script that I should have made.
But I'll hope you'll help me cause I want this script to work.
Hope you have some ideas for me.
And if you see the code everybody gets a id and I want it to stop at 30. How can I do that??
I have tried something, but when I came to 30 you still could post.
I don't know whats wrong....

The grep code does work...its likely that your code isn't working in a way to allow it to work...I was only guessing based on your sample code.