Jump to content

Recommended Posts

Posted

just use L2SkillTeleport, the command definetly needs delay

dammit all of you wanna me to die D: xd I'll do that tomorrow :Y

 

Leluche check it I already uploaded that.

 

V.2

Added Timer to end the event if nothing happens.

Added reward for the pk if he survives when the time is over.

Added command .pkinfo to know if you are near or far from the pk location.

Posted

very nice, gj mate. However you could have avoided making everything static for organization and optimization of the code.

 

if it was non-static he could run multiple instances of it throuth new, however he chose to use static because the event runs once every time

Posted

There's some duration in this event?

Yes there is, check the v2.

 

if it was non-static he could run multiple instances of it throuth new, however he chose to use static because the event runs once every time

+1

 

Much error's in gameserver console,

V2 was  under test when you probably added it. Now V2 is tested and 0 errors in my gs.

V1 no errors too.

 

Posted

It could have been done like a task to begin on pk. Having everything static isn't the best way to do it. Also about running multiple events just a static boolean would solve the problem.

Posted

It could have been done like a task to begin on pk. Having everything static isn't the best way to do it. Also about running multiple events just a static boolean would solve the problem.

Ye well there are 100000 ways to code everything, I chose this. And I don't wanna run multiple events.

If someone needs to have all his patchs with the perfect code style... he'll need to modify it, I'm just sharing something.

Posted

Already added the thing about L2SkillTeleport.

We maybe should add a config to allow/or not, the GM's being the PK of the event, to test/live switch, I'll do it soon...

I tested it and I can't see any error right now.

 

If someone has any error or any suggestion to add just tell me :P

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


×
×
  • Create New...