[QUESTION] Why not working this script?

[koragg]

1 hour ago, 50cent said:

Lol, now i have a new problem.  The script working, but sometimes the /asay xy command does not working, wtf?!  When i restart the resource many times then working again, but just for a few minutes, and does not working again. Wtf, how to fix this problem?  

No idea. Give debug errors/warnings.

Edited July 5, 2017 by koragg

[Gordon_G]

local names = {
	["Admin"] = "Tulajdonos",
	["Moderator"] = "Moderátor"
}


function adminuzi(thePlayer, commandName, ...)
	for i,v in pairs(getElementsByType( "player" )) do
        for group, serverGroupName in pairs( names ) do
            if isGuestAccount( getPlayerAccount( v ) ) then return end
		    if isObjectInACLGroup("user."..getAccountName(getPlayerAccount(thePlayer)), aclGetGroup( group ) then
		    		local message = table.concat({...}, " ")
	                outputChatBox("#0088FF["..serverGroupName.."]#d8001f  "..getPlayerName(v)..":"..message,getRootElement(),0,0,0,true)
                    break
		    end
        end
	end
end
addCommandHandler("asay", adminuzi)

Juste the thing I gave you, but "break" will stop the "for" loop. It should'nt spam anymore

@50cent

Edited July 5, 2017 by Gordon_G

[Tokio]

26 minutes ago, koragg said:

No idea. Give debug errors/warnings.

no errors/warnings

[Tokio]

2 hours ago, Gordon_G said:

local names = {
	["Admin"] = "Tulajdonos",
	["Moderator"] = "Moderátor"
}


function adminuzi(thePlayer, commandName, ...)
	for i,v in pairs(getElementsByType( "player" )) do
        for group, serverGroupName in pairs( names ) do
            if isGuestAccount( getPlayerAccount( v ) ) then return end
		    if isObjectInACLGroup("user."..getAccountName(getPlayerAccount(thePlayer)), aclGetGroup( group ) then
		    		local message = table.concat({...}, " ")
	                outputChatBox("#0088FF["..serverGroupName.."]#d8001f  "..getPlayerName(v)..":"..message,getRootElement(),0,0,0,true)
                    break
		    end
        end
	end
end
addCommandHandler("asay", adminuzi)

Juste the thing I gave you, but "break" will stop the "for" loop. It should'nt spam anymore

@50cent

this is not good, because output the text with all player's name..  

[itHyperoX]

getPlayerName(thePlayer) "v" is all player..

Edited July 5, 2017 by TheMOG

[Tokio]

1 minute ago, TheMOG said:

getPlayerName(thePlayer) "v" is all player..

here also change the "v"?

if isGuestAccount( getPlayerAccount( v ) )

[itHyperoX]

no

[Tokio]

Just now, TheMOG said:

no

 

I changed the "v" to this getPlayerName(thePlayer), and now flood the text with my name, how to fix this?

[koragg]

Use my code from first page. 

[Tokio]

11 minutes ago, koragg said:

Use my code from first page. 

i used, but sometimes does not working the cmd  

[koragg]

10 minutes ago, 50cent said:

i used, but sometimes does not working the cmd  

Well it's better than a fully non-working one anyway  Hope someone can help you, I don't know why it doesn't work from time to time. And nothing in debug...

[Tokio]

31 minutes ago, koragg said:

Well it's better than a fully non-working one anyway  Hope someone can help you, I don't know why it doesn't work from time to time. And nothing in debug...

Now i tested on a localhost server, and on localhost server working fine everytime, and just on hosted server not working everytime