• Bind Menu
    46 replies, posted
[QUOTE=meharryp;49321424][url]http://wiki.garrysmod.com/index.php?title=Net_Library_Usage#Security[/url][/QUOTE] Any way thanks lol
You're creating a directory in the data folder for every player who joins your server, which contains a text file for every bind they create? Edit: apparently missed some earlier stuff. Oh boy
[QUOTE=adelaideisbur;49327763]You're creating a directory in the data folder for every player who joins your server, which contains a text file for every bind they create? Edit: apparently missed some earlier stuff. Oh boy[/QUOTE] Fixed in version 1.5 (new version :D) thanks! [editline]16th December 2015[/editline] Version 1.6 out (SO MANY UPDATES OMGGGGGG) check the changelog!
I like the auto-update idea.
[QUOTE=VeXan;49331707]I like the auto-update idea.[/QUOTE] It's not really AUTO but it lets you know when there is a new version lol thanks!
[QUOTE=XxLMM13xXx;49331836]It's not really AUTO but it lets you know when there is a new version lol thanks![/QUOTE] Ah, alright.
stop posting and fix your shit before you post it
[QUOTE=mjyelito;49334874]stop posting and fix your shit before you post it[/QUOTE] They are small little bugs that i'm just fixing to make the addon better... The addon would still work fine without them. Anything else to complain about?
Your "fix" for the bind spam exploit is pointless since the LMMBMEditBind message is just a clone of the create bind message, just without checking if the players folder exists. You're also using a completely pointless NWFloat, which 1, is the wrong data type, why aren't you using a bool, and 2, doesn't need to be networked, you can just store it as a variable on the player. Also, this isn't that bad, but I'd recommend caching the data clientside, rather than sending the data every time that someone opens the menu, since WriteTable isn't very optimized and can cause someone with a lot of binds to give the server trouble. And for the love of god remove your name at the end of each print.
Atleast he's not selling it. [sp] Oh wait, He tried 5 of his first scripts on scriptfodder and got rejected. I wonder how bad those where [/sp]
[QUOTE=meharryp;49343849]Your "fix" for the bind spam exploit is pointless since the LMMBMEditBind message is just a clone of the create bind message, just without checking if the players folder exists. You're also using a completely pointless NWFloat, which 1, is the wrong data type, why aren't you using a bool, and 2, doesn't need to be networked, you can just store it as a variable on the player. Also, this isn't that bad, but I'd recommend caching the data clientside, rather than sending the data every time that someone opens the menu, since WriteTable isn't very optimized and can cause someone with a lot of binds to give the server trouble. And for the love of god remove your name at the end of each print.[/QUOTE] I will look into all that! [QUOTE=Kevlon;49345372]Atleast he's not selling it. [sp] Oh wait, He tried 5 of his first scripts on scriptfodder and got rejected. I wonder how bad those where [/sp][/QUOTE] Yeah they were poorly made i'm still learning... We gotta start somewhere right :)
[QUOTE=XxLMM13xXx;49346614]We gotta start somewhere right :)[/QUOTE] Starting somewhere is better done by not making an ass of yourself multiple times.
Nice little menu. Will really make binding easier.
Whoops posted 2 times :( VERSION 1.7 FIXES THE NETWORK STUFF CHECK THE CHANGELOGS! [editline]19th December 2015[/editline] [QUOTE=StonedPenguin;49347642]Starting somewhere is better done by not making an ass of yourself multiple times.[/QUOTE] ... sorry? [QUOTE=Recession;49348376]Nice little menu. Will really make binding easier.[/QUOTE] Thanks!!
Love how maturely this guy is dealing with all of his hateful comments, thanks for the addon mate currently being used on my server
[QUOTE=LieutentDayum;49360686]Love how maturely this guy is dealing with all of his hateful comments, thanks for the addon mate currently being used on my server[/QUOTE] Haha thanks glad at least more than one person thought it was useful!!!
-snip-
Sorry, you need to Log In to post a reply to this thread.