Jump to content

OnPCStatCalcEvent


AnnieRuru

Recommended Posts


  • Group:  Members
  • Topic Count:  18
  • Topics Per Day:  0.00
  • Content Count:  2044
  • Reputation:   682
  • Joined:  10/09/12
  • Last Seen:  

since I discover this OnPCStatCalcEvent

I keep bringing it up in scripting section

and I start to get sick I have to update this patch every single time when I comply a request with it

http://rathena.org/board/topic/76088-skill-in-map/#entry166271

http://rathena.org/board/topic/77185-function-changelook-random-viewid/#entry171792

and a few unused like

http://rathena.org/board/topic/74206-close-drop-certain-item-on-certain-map/#entry155980

http://rathena.org/board/topic/72989-capture-the-flag-woe-script/#entry150735

I believe this mod is very very helpful

for example, in this topic

-	script	kdfjshksfjhksdfj	-1,{
OnPCStatCalcEvent:
bonus2 bExpAddRace,10,100; // PS: ... can someone confirm this exp is getting more than usual ?
bonus2 bExpAddRace,11,100;
end;
}

  • Upvote 3
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  40
  • Topics Per Day:  0.01
  • Content Count:  587
  • Reputation:   104
  • Joined:  11/19/11
  • Last Seen:  

agree+1 , rathena need more trigger event like OnPCStatCalcEvent OnPcuseskillevent OnScstartevent.....

Thus, it will bring more powerful and awesome scripts

Link to comment
Share on other sites


  • Group:  Developer
  • Topic Count:  48
  • Topics Per Day:  0.01
  • Content Count:  1443
  • Reputation:   337
  • Joined:  10/17/12
  • Last Seen:  

+1

Instantly I got like 30 ideas for this :)

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  9
  • Topics Per Day:  0.00
  • Content Count:  379
  • Reputation:   304
  • Joined:  11/10/11
  • Last Seen:  

Isn't it too intensive for the server to be official ? ( /str+ 255 will run the script 255 times, no ?)

  • Upvote 3
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  18
  • Topics Per Day:  0.00
  • Content Count:  2044
  • Reputation:   682
  • Joined:  10/09/12
  • Last Seen:  

as far as I concern, the script under OnPCStatCalcEvent label is very short

OnPCStatCalcEvent:
if ( strcharinfo(3) == "guild_vs2" && .Juggernaught_aid == getcharid(3) ) // conditions
bonus .... ; // put all the bonus
end;// that's all ^.^

the only things that really concern you is the condition to add the bonus ?

( /str+ 255 will run the script 255 times, no ?)
lol ? isn't the equipments { Iitem Script } field also run the item script field 255 times ?

http://www.eathena.w...dpost&p=1007915

whats the difference ?

and @str 255 will run the script field once, as well as OnPCStatCalcEvent also once

OnPCIdleEvent, OnPCUseSkill, OnPCAttackEvent, OnPCChatEvent

for these 4 I might reconsider, but this one I don't mind


still not convince enough ?

originally, if we want a donators in a server to gain a permanent status boost

we can either

1. use sc_start with high amount <-- looks noob code

2. create a new equipment with insane stats <-- more practical, and everyone currently doing this

but the equipment might broke or strip ( can counter with bUnbreakable*** and bUnstripable*** though )

or this

3. OnPCStatCalcEvent

OnPCStatCalcEvent:
   if ( getgroupid() == 1 ) { // Super Player
       bonus bAllStats, 10;
       bonus2 bExpAddRace,10,100;
       bonus2 bExpAddRace,11,100;
   }
   end;

super player gains a permanent +10 all stats and +100% exp gain by just being a Super Player

with no.2, if use /str+100 also execute donator equipment's Item script 100 times

or no.3, /str+100 also execute this label 100 times

I don't see any difference

Edited by AnnieRuru
  • Upvote 1
Link to comment
Share on other sites


  • Group:  Developer
  • Topic Count:  10
  • Topics Per Day:  0.00
  • Content Count:  2407
  • Reputation:   613
  • Joined:  07/05/12
  • Last Seen:  

Agree

It's a good idea

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  12
  • Topics Per Day:  0.00
  • Content Count:  318
  • Reputation:   37
  • Joined:  12/30/11
  • Last Seen:  

+1 very nice idea

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  9
  • Topics Per Day:  0.00
  • Content Count:  379
  • Reputation:   304
  • Joined:  11/10/11
  • Last Seen:  

( /str+ 255 will run the script 255 times, no ?)
lol ? isn't the equipments { Iitem Script } field also run the item script field 255 times ?

Yeah you're right :)

  • Upvote 2
Link to comment
Share on other sites


  • Group:  Developer
  • Topic Count:  48
  • Topics Per Day:  0.01
  • Content Count:  1443
  • Reputation:   337
  • Joined:  10/17/12
  • Last Seen:  

Ya so i really want this implemented now, I've added to my test server using AnnieRuru's patch, and it makes some script so much easier to make.

Link to comment
Share on other sites

  • 1 year later...

  • Group:  Members
  • Topic Count:  72
  • Topics Per Day:  0.02
  • Content Count:  2997
  • Reputation:   1130
  • Joined:  05/27/12
  • Last Seen:  

Implemented in 27a0f3f.

  • Upvote 1
Link to comment
Share on other sites

×
×
  • Create New...