Lua Error

Garry's Mod coding
#1

OK, here is the original code:



 concommand.Add( "amg_start", function( ply, cmd, args, string )
	if AMG.UseTimer then
		print( "AMG.UseTimer must be set to false." )
	else
		StartMinigame( args[1], args[2] )
	end
end)

And I got the error of:

[ERROR] addons/aminigames/lua/autorun/fcp_autorun.lua:20: attempt to call global ‘StartMinigame’ (a nil value)

  1. unknown - addons/aminigames/lua/autorun/fcp_autorun.lua:20
  2. unknown - lua/includes/modules/concommand.lua:69

So I tried getting help and I changed the code up a little and got:



concommand.Add( "amg_start", function( ply, cmd, args, string )
	if AMG.UseTimer then
		print( "AMG.UseTimer must be set to false." )
	else
		StartMinigame( ply, cmd )
	end
end)

And got the same error. So I’m really lost here. Would appreciate some help.

I also tried:



StartMinigame( ply, cmd, args )
#2

Can you post the StartMinigame function?

#3 


function StartMinigame( type, length )

	print( "Starting minigame!" )

	if not table.HasValue( gameTypes, type ) then return end
	mgType = table.Random( AMG.miniGames )
	theType = type
	mgActive = true
	for k, v in pairs( player.GetAll() ) do
		v.Earned = 0
		net.Start( "NotifyPlayers" )
		net.Send( v )
	end

I’m guessing I would change the code to:



concommand.Add( "amg_start", function( ply, cmd, args, string )
	if AMG.UseTimer then
		print( "AMG.UseTimer must be set to false." )
	else
		StartMinigame( type, length )
	end
end)

???

#4

StartMinigame is probably a serverside function you’re trying to call clientside with the command. Make sure the concommand.Add is serverside. Your first code of



 concommand.Add( "amg_start", function( ply, cmd, args, string )
    if AMG.UseTimer then
        print( "AMG.UseTimer must be set to false." )
    else
        StartMinigame( args[1], args[2] )
    end
end)

Should work.

#5

The filename which contains the StartMinigame function is sv_amg.lua.
Would that mean it’s a serverside command?

#6

Yes. Where is the concommand.Add located?. Also, you shouldn’t have changed the function arguments, they were fine before.

#7

The concommand is in fcp_autorun.lua

#8

Is that being called clientside or serverside? What is the location?

#9

Pretty sure it’s serverside. The location is minigames/lua/autorun/fcp_autorun.lua

[editline]26th August 2014[/editline]

Not really getting any solutions…

#10

So it isn’t serverside, but shared. Wrap the concommand in a SERVER if statement.

#11

The lua file already has a server if statement at the top. Do I have to make a separate one for the concommand?

#12

No, just make sure that the concommand is inside the if statement still. If that checks out, then the only other issue would be that the function is being created after the concommand.

#13

Still not working… Here’e the code:



if SERVER then
 concommand.Add( "amg_start", function( ply, cmd, args, string )
    if AMG.UseTimer then
        print( "AMG.UseTimer must be set to false." )
    else
        StartMinigame( args[1], args[2] )
    end
end
end)

[editline]26th August 2014[/editline]

I get the same error when I type it in console but when I restart I get this error from it:

[ERROR] addons/aminigames/lua/autorun/fcp_autorun.lua:24: ‘)’ expected (to close ‘(’ at line 17) near ‘end’

  1. unknown - addons/aminigames/lua/autorun/fcp_autorun.lua:0
#14

You don’t need the last ‘)’ on the end to close the if SERVER statement.

#15

I removed the ‘)’ but the command doesn’t work anymore…
Unknown command: amg_start

#16

Put it on the end above it, this is why you need indentation so you can see which end corresponds to each part.

#17

Put it back.

It seems as if the error isn’t even generating from the given code, considering there’s nowhere that needs a ) to close off that isn’t already there. Maybe there’s an error earlier on in the file

#18

If I put it back I get the error of:

[ERROR] addons/aminigames/lua/autorun/fcp_autorun.lua:24: ‘)’ expected (to close ‘(’ at line 17) near ‘end’

  1. unknown - addons/aminigames/lua/autorun/fcp_autorun.lua:0

If I don’t have it, the command doesn’t work.
I’m really consused.
Here’s my current code:



	if SERVER then

 concommand.Add( "amg_start", function( ply, cmd, args, string )
    if AMG.UseTimer then
        print( "AMG.UseTimer must be set to false." )
    else
        StartMinigame( args[1], args[2] )
    end
		end
	end
end
#19

Now you’re missing a ) at the end for the concommand.Add

#20

What you posted earlier, you only had to move the bracket up one line.

[lua]if SERVER then
concommand.Add( “amg_start”, function( ply, cmd, args, string )
if AMG.UseTimer then
print( “AMG.UseTimer must be set to false.” )
else
StartMinigame( args[1], args[2] )
end
end)
end[/lua]