disable serverside destroy()...

[DustyPorViva]

Destroy() on the serverside should not work for DBNPCs(excluding putnpc2's), WNPC's, and I guess level NPC's. It's a bit silly to have to put our own prevention methods so people can't put a script in a level and destroy whole weapons. Really, when do you NEED to destroy a WNPC? I mean, I guess client-RC might use that sort of thing, but I'm sure exceptions could be made for that. Either way, I'd like to see some sort of methods worked out that don't involve having scripters make protection for each and every one of their scripts.

[cbk1994]

I agree (client-RC does not use the destroy command anyway).

[fowlplay4]

I tried to destroy a WNPC from another DBNPC/WNPC using:

PHP Code:

// substituting "npc" with the real name.
findnpc("npc").destroy();
findweapon("npc").destroy(); 


And wasn't able to o:

In theory couldn't you just.. (For now i guess)

PHP Code:

public function destroy() {
  return false;
} 


[DustyPorViva]

Originally Posted by fowlplay4

I tried to destroy a WNPC from another DBNPC/WNPC using:

PHP Code:

// substituting "npc" with the real name.
findnpc("npc").destroy();
findweapon("npc").destroy(); 


And wasn't able to o:

In theory couldn't you just.. (For now i guess)

PHP Code:

public function destroy() {
  return false;
} 


On testbed they actually distribute some script that will do a callstack and such and echo on RC if an NPC tries to destroy it(as well as what NPC it was).

Either way, I don't really see a need for such a lose security. As for whether it's possible or not I'm not sure, but it was a huge thing on Testbed so I figured it was.

[TESTRETIS]

Didn't know it was actually possible to do that to DBNPCs. For WNPCs, though I think I might disagree.

[DustyPorViva]

Originally Posted by TESTRETIS

Didn't know it was actually possible to do that to DBNPCs. For WNPCs, though I think I might disagree.

You want to be able to destroy WNPC's with scripts? Why?

[cbk1994]

Not really wanting to spread code, but you would use

PHP Code:

findWeaponNPC("test").destroy(); 


Originally Posted by TESTRETIS

Didn't know it was actually possible to do that to DBNPCs.

Absolutely, I accidently destroyed about a dozen DBNPCs on Testbed when attempting to clear out some guy's putnpc2's. There is a server option to disable it, though.

[TESTRETIS]

Originally Posted by DustyPorViva

You want to be able to destroy WNPC's with scripts? Why?

If I'm not mistaken, you can use this in substitution of removeweapon();
Or so thor claims.

I know, not the most effective or best way, but..

[cbk1994]

Originally Posted by TESTRETIS

If I'm not mistaken, you can use this in substitution of removeweapon();
Or so thor claims.

I know, not the most effective or best way, but..

You can remove weapons from a player clientside by calling destroy(). Serverside it deletes the weapon.

[Riot]

Originally Posted by DustyPorViva

Destroy() on the serverside should not work for DBNPCs(excluding putnpc2's), WNPC's, and I guess level NPC's. It's a bit silly to have to put our own prevention methods so people can't put a script in a level and destroy whole weapons. Really, when do you NEED to destroy a WNPC? I mean, I guess client-RC might use that sort of thing, but I'm sure exceptions could be made for that. Either way, I'd like to see some sort of methods worked out that don't involve having scripters make protection for each and every one of their scripts.

I'm pretty sure DBNPCs are protected by default, assuming no one has adjusted server options... At least going off of http://graal.net/index.php/Server_op...PCs_Protection they should be. I'm unable to use destroy() on DBNPCs on Delteria Dev.

I guess I should learn to read though since that was already posted.

At any rate, weapons should be protected as well.