groups.txt - some changes

[mrm]

Hello, I have made a few changes to groups.txt, and would like somebody too look it over for mistakes. Here is the code:

OK, I ran it through Notepad++, please look again.

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


"operator"   
{
   "deny"   
   {
   }
   "allow"   
   {
      "ulx seeasay"
   }
   "inherit_from"   
   {
   }
}
"uberadmin"   
{
   "deny"   
   {
   }
   "allow"   
   {
      "ulx hiddenecho"
      "ulx rcon"
      "ulx luarun"
      "ulx cexec"
      "ulx ent"
      "ulx adduser"
      "ulx adduserid"
      "ulx removeuser"
      "ulx userallow"
      "ulx userdeny"
      "ulx addgroup"
      "ulx removegroup"
      "ulx groupallow"
      "ulx groupdeny"
      "overcomeimmunity"
   }
   "inherit_from"   
   {
      "superadmin"
   }
}
"superadmin"   
{
   "deny"   
   {
   }
   "allow"   
   {
      "ulib_passtime"
      "ulib_passtimeout"
      "ulx spawnecho"
      "ulx tsay"
      "ulx csay"
      "ulx gimp"
      "ulx mute"
      "ulx ungimp"
      "ulx unmute"
      "ulx gag"
      "ulx ungag"
      "ulx chattime"
      "ulx welcomemessage"
      "ulx slap"
      "ulx whip"
      "ulx slay"
      "ulx sslay"
      "ulx ignite"
      "ulx unignite"
      "ulx playsound"
      "ulx freeze"
      "ulx unfreeze"
      "ulx god"
      "ulx ungod"
      "ulx noclip"
      "ulx hp"
      "ulx armor"
      "ulx cloak"
      "ulx uncloak"
      "ulx blind"
      "ulx unblind"
      "ulx jail"
      "ulx unjail"
      "ulx ghost"
      "ulx unghost"
      "ulx ragdoll"
      "ulx unragdoll"
      "ulx maul"
      "ulx strip"
      "ulx adminmenu"
      "ulx clientmenu"
      "ulx mapsmenu"
      "ulx showmotd"
      "ulx banmenu"
      "ulx exec"
      "ulx rslotsmode"
      "ulx rslots"
      "ulx rslotsvisible"
      "ulx reservedslots"
      "ulx bring"
      "ulx goto"
      "ulx send"
      "ulx teleport"
      "ulx tooldeny"
      "ulx toolallow"
      "ulx tooldenyuser"
      "ulx toolallowuser"
      "ulx tooldenyoverride"
      "ulx map"
      "ulx kick"
      "ulx ban"
      "ulx banid"
      "ulx unban"
      "ulx spectate"
      "ulx physgunplayer"
      "ulx vote"
      "ulx votemap2"
      "ulx votekick"
      "ulx voteban"
      "ulx veto"
      "ulx rocket"
   }
   "inherit_from"   
   {
      "admin"
   }
}

"admin"
{
         "deny"
         {
         }
         "allow"
         {
 
      
      "ulx gimp"
      "ulx mute"
      "ulx ungimp"
      "ulx unmute"
      "ulx gag"
      "ulx ungag"
      "ulx slap"
      "ulx whip"
      "ulx slay"
      "ulx sslay"
      "ulx ignite"
      "ulx unignite"
      "ulx playsound"
      "ulx freeze"
      "ulx unfreeze"
      "ulx god"
      "ulx ungod"
      "ulx noclip"
      "ulx hp"
      "ulx armor"
      "ulx cloak"
      "ulx uncloak"
      "ulx blind"
      "ulx unblind"
      "ulx jail"
      "ulx unjail"
      "ulx ghost"
      "ulx unghost"
      "ulx ragdoll"
      "ulx unragdoll"
      "ulx maul"
      "ulx strip"
      "ulx adminmenu"
      "ulx clientmenu"
      "ulx showmotd"
      "ulx banmenu"
      "ulx bring"
      "ulx goto"
      "ulx send"
      "ulx teleport"
      "ulx tooldeny"
      "ulx toolallow"
      "ulx tooldenyuser"
      "ulx toolallowuser"
      "ulx tooldenyoverride"
      "ulx kick"
      "ulx ban"
      "ulx banid"
      "ulx unban"
      "ulx spectate"
      "ulx physgunplayer"
      "ulx vote"
      "ulx votemap2"
      "ulx votekick"
      "ulx voteban"
      "ulx veto"
               }
"inherit from"
{
"moderator"
}

}
      
             
"user"   
{
   "deny"   
   {
   }
   "allow"   
   {
      "ulx"
      "ulx help"
      "ulx psay"
      "ulx asay"
      "ulx thetime"
      "ulx menu"
      "ulx_valueupdate"
      "ulx_cvar"
      "ulx_getgamemodes"
      "ulx motd"
      "ulx_getbans"
      "ulx usermanagementhelp"
      "ulx who"
      "ulx votemap"
   }
   "inherit_from"   
   {
   }
}
"member"   
{
          "deny"
          {
          }
          "allow"
          {
                  "ulx god"
                  "ulx ungod"
                  "ulx goto"
                  "ulx teleport"
          }
          "inherit_from"
          {
          "user"
          }
}
"moderator"   
{
             "deny"
             {
             }
             "allow"
             {
                      "ulx kick"
                      "ulx jail"
                      "ulx slap"
                      "ulx unjail"
             }
"inherit from"
{
"member"
}

}





"none"   
{
   "deny"   
   {
   }
   "allow"   
   {
      "ulx logecho"
      "ulx logfile"
      "ulx logevents"
      "ulx logchat"
      "ulx logspawns"
      "ulx logspawnsecho"
      "ulx logdir"
      "ulx addgimpsay"
      "ulx addadvert"
      "ulx addcsayadvert"
      "ulx addforceddownload"
      "ulx debuginfo"
      "ulx voteecho"
      "ulx votemap2successratio"
      "ulx votemap2minvotes"
      "ulx votekicksuccessratio"
      "ulx votekickminvotes"
      "ulx votebansuccessratio"
      "ulx votebanminvotes"
      "ulx votemapenabled"
      "ulx votemapmintime"
      "ulx votemapwaittime"
      "ulx votemapsuccessratio"
      "ulx votemapminvotes"
      "ulx votemapvetotime"
      "ulx votemapmapmode"
      "ulx votemapaddmap"
   }
   "inherit_from"   
   {
   }
}

[JamminR]

Missing closing bracket of the the admin group

Code: [Select]

"inherit from"
{
"moderator"
}
<--- missing closing bracket here before....
"user"

I recommend a true text editor such as Context or Notepadd++
Even with no special programming language construct downloads with them, they have the ability to show missing brackets visually.
There may be more. I stopped looking after that.

[mrm]

Ok, I downloaded and ran it through notepad++, could you please look through it again for any errors? I want to be completely certain it will work before I switch over to it.

[JamminR]

Looks ok to me. I just took only a few moments to check it though.
One tip that might make it easier for you to 'read' later, (and, I'm not sure, may affect ULib but don't think so)
You may wish to put your groupings in order at least in that file.
Like, you have
user { }
member {}
moderator { } inherit member
Go ahead and have member after moderator top to bottom.

SHould work even not doing that, but like I said, easier readability if you edit later.

[deadeye536]

I've read the code, and ran it through Notepad++ and found no errors, everything is in quotes, the brackets are properly matched up. I also cleaned up the code a bit, doing what JamminR suggested, but from none, user -> uberadmin in what looked like a logical order.
The code you have will work, but here is this if you want it

Code: [Select]

"uberadmin"
{
"deny"
{
}
"allow"
{
"ulx hiddenecho"
"ulx rcon"
"ulx luarun"
"ulx cexec"
"ulx ent"
"ulx adduser"
"ulx adduserid"
"ulx removeuser"
"ulx userallow"
"ulx userdeny"
"ulx addgroup"
"ulx removegroup"
"ulx groupallow"
"ulx groupdeny"
"overcomeimmunity"
}
"inherit_from"
{
"superadmin"
}
}
"superadmin"
{
"deny"
{
}
"allow"
{
"ulib_passtime"
"ulib_passtimeout"
"ulx spawnecho"
"ulx tsay"
"ulx csay"
"ulx gimp"
"ulx mute"
"ulx ungimp"
"ulx unmute"
"ulx gag"
"ulx ungag"
"ulx chattime"
"ulx welcomemessage"
"ulx slap"
"ulx whip"
"ulx slay"
"ulx sslay"
"ulx ignite"
"ulx unignite"
"ulx playsound"
"ulx freeze"
"ulx unfreeze"
"ulx god"
"ulx ungod"
"ulx noclip"
"ulx hp"
"ulx armor"
"ulx cloak"
"ulx uncloak"
"ulx blind"
"ulx unblind"
"ulx jail"
"ulx unjail"
"ulx ghost"
"ulx unghost"
"ulx ragdoll"
"ulx unragdoll"
"ulx maul"
"ulx strip"
"ulx adminmenu"
"ulx clientmenu"
"ulx mapsmenu"
"ulx showmotd"
"ulx banmenu"
"ulx exec"
"ulx rslotsmode"
"ulx rslots"
"ulx rslotsvisible"
"ulx reservedslots"
"ulx bring"
"ulx goto"
"ulx send"
"ulx teleport"
"ulx tooldeny"
"ulx toolallow"
"ulx tooldenyuser"
"ulx toolallowuser"
"ulx tooldenyoverride"
"ulx map"
"ulx kick"
"ulx ban"
"ulx banid"
"ulx unban"
"ulx spectate"
"ulx physgunplayer"
"ulx vote"
"ulx votemap2"
"ulx votekick"
"ulx voteban"
"ulx veto"
"ulx rocket"
}
"inherit_from"
{
"admin"
}
}

"admin"
{
"deny"
{
}
"allow"
{
"ulx gimp"
"ulx mute"
"ulx ungimp"
"ulx unmute"
"ulx gag"
"ulx ungag"
"ulx slap"
"ulx whip"
"ulx slay"
"ulx sslay"
"ulx ignite"
"ulx unignite"
"ulx playsound"
"ulx freeze"
"ulx unfreeze"
"ulx god"
"ulx ungod"
"ulx noclip"
"ulx hp"
"ulx armor"
"ulx cloak"
"ulx uncloak"
"ulx blind"
"ulx unblind"
"ulx jail"
"ulx unjail"
"ulx ghost"
"ulx unghost"
"ulx ragdoll"
"ulx unragdoll"
"ulx maul"
"ulx strip"
"ulx adminmenu"
"ulx clientmenu"
"ulx showmotd"
"ulx banmenu"
"ulx bring"
"ulx goto"
"ulx send"
"ulx teleport"
"ulx tooldeny"
"ulx toolallow"
"ulx tooldenyuser"
"ulx toolallowuser"
"ulx tooldenyoverride"
"ulx kick"
"ulx ban"
"ulx banid"
"ulx unban"
"ulx spectate"
"ulx physgunplayer"
"ulx vote"
"ulx votemap2"
"ulx votekick"
"ulx voteban"
"ulx veto"
}
"inherit from"
{
"moderator"
}
}
"moderator"
{
"deny"
{
}
"allow"
{
"ulx kick"
"ulx jail"
"ulx slap"
"ulx unjail"
}
"inherit from"
{
"member"
}
}
"member"
{
"deny"
{
}
"allow"
{
"ulx god"
"ulx ungod"
"ulx goto"
"ulx teleport"
}
"inherit_from"
{
"user"
}
}
"operator"
{
"deny"
{
}
"allow"
{
"ulx seeasay"
}
"inherit_from"
{
}
}
"user"
{
"deny"
{
}
"allow"
{
"ulx"
"ulx help"
"ulx psay"
"ulx asay"
"ulx thetime"
"ulx menu"
"ulx_valueupdate"
"ulx_cvar"
"ulx_getgamemodes"
"ulx motd"
"ulx_getbans"
"ulx usermanagementhelp"
"ulx who"
"ulx votemap"
}
"inherit_from"
{
}
}
"none"
{
"deny"
{
}
"allow"
{
"ulx logecho"
"ulx logfile"
"ulx logevents"
"ulx logchat"
"ulx logspawns"
"ulx logspawnsecho"
"ulx logdir"
"ulx addgimpsay"
"ulx addadvert"
"ulx addcsayadvert"
"ulx addforceddownload"
"ulx debuginfo"
"ulx voteecho"
"ulx votemap2successratio"
"ulx votemap2minvotes"
"ulx votekicksuccessratio"
"ulx votekickminvotes"
"ulx votebansuccessratio"
"ulx votebanminvotes"
"ulx votemapenabled"
"ulx votemapmintime"
"ulx votemapwaittime"
"ulx votemapsuccessratio"
"ulx votemapminvotes"
"ulx votemapvetotime"
"ulx votemapmapmode"
"ulx votemapaddmap"
}
"inherit_from"
{
}
}

[mrm]

Looks ok to me. I just took only a few moments to check it though.
One tip that might make it easier for you to 'read' later, (and, I'm not sure, may affect ULib but don't think so)
You may wish to put your groupings in order at least in that file.
Like, you have
user { }
member {}
moderator { } inherit member
Go ahead and have member after moderator top to bottom.

SHould work even not doing that, but like I said, easier readability if you edit later.

Ya, that's a good idea. Thanks for looking over it again.

[mrm]

I've read the code, and ran it through Notepad++ and found no errors, everything is in quotes, the brackets are properly matched up. I also cleaned up the code a bit, doing what JamminR suggested, but from none, user -> uberadmin in what looked like a logical order.
The code you have will work, but here is this if you want it

Wow, thanks for doing all that work. I really appreciate it!