AlliedModders

AlliedModders (https://forums.alliedmods.net/index.php)
-   Plugins (https://forums.alliedmods.net/forumdisplay.php?f=108)
-   -   [CS:S] Jail Awards with credits system (https://forums.alliedmods.net/showthread.php?t=174843)

RedSword 01-03-2013 15:08

Re: [CS:S] Jail Awards with credits system v1.1
 
Hi there,

the following should be changed for approval :
  • Your code is prone to error. i.e. calling "sm_credits" from server console. It seems to be the case for almost all your commands.
  • While your "sm_awards" currently produce an error while trying to PrintToClient to the server, it does send a menu to the server just before (L213 / L297). While this doesn't produce a crash or an error, it is an unwanted behaviour (I did file a bug report on this).
  • You load "common.phrases", but there is no %t/%T in all your code !

Beside, you should also remove FCVAR_REPLICATED, as this can produce console error for the client.

Other than that, I don't see any real problem.

Franc1sco 01-03-2013 16:17

Re: [CS:S] Jail Awards with credits system v1.1
 
Quote:

Originally Posted by RedSword (Post 1866348)
Hi there,

the following should be changed for approval :
  • Your code is prone to error. i.e. calling "sm_credits" from server console. It seems to be the case for almost all your commands.
  • While your "sm_awards" currently produce an error while trying to PrintToClient to the server, it does send a menu to the server just before (L213 / L297). While this doesn't produce a crash or an error, it is an unwanted behaviour (I did file a bug report on this).
  • You load "common.phrases", but there is no %t/%T in all your code !

Beside, you should also remove FCVAR_REPLICATED, as this can produce console error for the client.

Other than that, I don't see any real problem.

1. fixed
2. fixed
I added:
PHP Code:

    if(client == 0// if is console
    
{
        
PrintToServer("you need to be ingame for use this"); // reply to console
        
return; // dont continue
    


3. removed
4. removed



--------------------------------------------------------------------

new version 1.2

-bugfixed for approval

RedSword 01-03-2013 16:48

Re: [CS:S] Jail Awards with credits system v1.2
 
It is still possible to get an error using sm_setcredits from server console.

EDIT : It would possible by the way to use "Command is in-game only" from common.phrases, rather than creating your own phrase, that way your plugin is (partially) multilingual.

Franc1sco 01-03-2013 16:52

Re: [CS:S] Jail Awards with credits system v1.2
 
Quote:

Originally Posted by RedSword (Post 1866414)
It is still possible to get an error using sm_setcredits from server console.

ups ok fixed

RedSword 01-03-2013 16:54

Re: [CS:S] Jail Awards with credits system v1.2
 
Approved.

Franc1sco 01-03-2013 17:05

Re: [CS:S] Jail Awards with credits system v1.2
 
Quote:

Originally Posted by RedSword (Post 1866414)
EDIT : It would possible by the way to use "Command is in-game only" from common.phrases, rather than creating your own phrase, that way your plugin is (partially) multilingual.

Added.


Quote:

Originally Posted by RedSword (Post 1866422)
Approved.

ok thanks

healasuk 01-04-2013 21:28

Re: [CS:S] Jail Awards with credits system v1.2
 
How do you edit the amount of credits that is required to buy a certain award?

Franc1sco 01-04-2013 21:29

Re: [CS:S] Jail Awards with credits system v1.2
 
Quote:

Originally Posted by healasuk (Post 1867249)
How do you edit the amount of credits that is required to buy a certain award?

Soon I will add cvars for it

healasuk 01-05-2013 23:44

Re: [CS:S] Jail Awards with credits system v1.2
 
I'll be waiting for the cvars to be added

sjvcs 01-06-2013 11:19

Re: [CS:S] Jail Awards with credits system v1.2
 
hello, do it to the T and CT are different credits ..
I think it would be good


All times are GMT -4. The time now is 17:16.

Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.