Jump to content

  •  

Bug Tracker Migration

June 3rd
Good news everyone! The staff has decided that it is time to slowly kill off this Bug Tracker. We will begin the process of slowly migrating from this Bug Tracker over to our Github Issues which can be found here: https://github.com/HerculesWS/Hercules/issues

Over the next couple of days, I will be closing off any opportunity to create new reports. However, I still will keep the opportunity to reply to existing Bug Reports. Doing this will allow us to slowly fix any bug reports we have listed here so that we can easily migrate over to our Issue Tracker.

Update - June 7th 2015: Creating new bug posts has been disabled. Please use our https://github.com/HerculesWS/Hercules/issues tracker to post bugs. Users are still able to reply to existing bug posts.

- Administration

Issue Information

  • #006001

  • 0 - None Assigned

  • Fixed

Issue Confirmations

  • Yes (0)No (0)
Photo

Wrong script

Posted by Hercules Bot on 13 June 2012 - 02:35 AM

Originally posted by hendra814
item 18600 item description table

18600#
a cute beret that feels cat is wearing it because of its ear decoration.
ATK + 5%.
If it refined over 5, will give 1% bonus on physical attack to "Human" type and receive less damage from "Human" type.
(This bonus would work up to +12, 7%)
Class : ^777777Headgear^000000 Defense : ^7777775^000000
Location : ^777777Upper ^000000Weight : ^77777730^000000
Required Lv. : ^7777771^000000
Applicable Job : ^777777 Every Job^000000
#


and the script

18600,Cat_Ear_Beret,Cat Ear Beret,5,20,,100,,5,,0,0xFFFFFFFF,7,2,256,,0,1,761,{ if(getrefine() > 5 && getrefine() <= 12) { bonus2 bAddRace,RC_DemiHuman,2*(getrefine() - 4); bonus2 bSubRace,RC_DemiHuman,2*(getrefine() - 4); }},{},{}


if refine to +5 will not receive bonus point
but if look into the script, item will give 2 point bonus.
and for +12 will give 16 point bonus

I thinks the script suppose to be like this

18600,Cat_Ear_Beret,Cat Ear Beret,5,20,,100,,5,,0,0xFFFFFFFF,7,2,256,,0,1,761,{ if(getrefine() > 5 && getrefine() <= 12) { bonus2 bAddRace,RC_DemiHuman,1*(getrefine() - 5); bonus2 bSubRace,RC_DemiHuman,1*(getrefine() - 5); }},{},{}


Originally posted by hendra814
for item 18597

18597#
...
Aspd +3%, Critical +3
If refined to +7 or higher, Aspd +2% and Critical +2.
If refined to +9 or higher, Aspd +2% and Critical +2.
Class : ^777777Headgear^000000
Defense : ^77777710^000000
Location : ^777777Upper^000000
Weight : ^77777740^000000
Required Level : ^77777770^000000
Applicable Job :^777777 Every Job^000000
#


item script

18597,Mercury_Riser,Mercury Riser,5,40,,200,,10,,1,0xFFFFFFFF,7,2,256,,0,,759,{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() == 7 || getrefine() == 8) { bonus bAspdRate,2; bonus bCritical,2; } if(getrefine() >= 9) { bonus bAspdRate,2; bonus bCritical,2; }},{},{}


For refine +7 and +8 will recieve 2 point for aspd and critical
and refine +9 or higher recieve 2 point more, is this mean player will recieve 4 point or same with refine +7, just recieve 2 point for upgrade the item.

Originally posted by Flaid
Fixed in r16284 and r16285

Originally posted by hendra814
will be simple if the script like this

18597,Mercury_Riser,Mercury Riser,5,40,,200,,10,,1,0xFFFFFFFF,7,2,256,,0,,759,{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() >= 7 ) { bonus bAspdRate,2; bonus bCritical,2; }},{},{}


i think this is will be same efect

Originally posted by Flaid
I fixed it already, it works as it should work now.

Originally posted by hendra814
ok thanks. i'm already testing on it

Originally posted by volk
not update in sql?

Originally posted by ForteXX
Im thinking the script should continue working past +12, but stop increasing at that point, not stop working altogether (see other scripts with these kind of effects) [beret]

for the other, why wouldnt you use { bonus bAspdRate,3; bonus bCritical,3; if(getrefine() >= 7) { bonus bAspdRate,2; bonus bCritical,2; } if(getrefine() >= 9) { bonus bAspdRate,2; bonus bCritical,2; }}
(more directly in line with what the description says)

Edited by ForteXX, 13 June 2012 - 02:25 PM.


Originally posted by malufett
ForteXX idea is better... /no1

@Flaid

if(getrefine() == 7 || getrefine() == 8)

this would fail if the upgrade is greater than 9..my bad didn't see "=>9"

so ATM this is not totally fixed.....

Originally posted by ForteXX
Well, it would, due to the next part in the script. Way it is right now, if +7 or +8, it gives 2, if +9 or greater, it gives 4.

I was just saying that the description says +7 or greater gives +2, +9 or greater gives another 2, not cancelling the +7/8 bonus and giving a straight 4.

Edited by ForteXX, 13 June 2012 - 03:50 PM.


Originally posted by hendra814

Well, it would, due to the next part in the script. Way it is right now, if +7 or +8, it gives 2, if +9 or greater, it gives 4.I was just saying that the description says +7 or greater gives +2, +9 or greater gives another 2, not cancelling the +7/8 bonus and giving a straight 4.


you mean this script

{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() >= 7 ) { bonus bAspdRate,2; bonus bCritical,2; }},{},{}


will give +2 point at refine +7 or +8
and add +2 more for refine higher than +9
so character totaly receive 4 point if refine higher than +9

Originally posted by ForteXX
If you use that script, you will only ever be able to get a bonus of 5 (3+2) since it doesnt even have the refined to 9+ bonus

Originally posted by hendra814
so let me make the table

+7 player get 5 point for aspd and critical
+8 player get same as refine +7
+9 player get 7 point for aspd and critical
+10 player get same as refine +9

is it right for this script

{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() >= 7 ) { bonus bAspdRate,2; bonus bCritical,2; }},{},{}



Originally posted by ForteXX
No, that script has nothing happening extra for +9 or greater, only the extra +2 for +7 or greater.

Using
{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() >= 7) { bonus bAspdRate,2; bonus bCritical,2; } if(getrefine() >= 9) { bonus bAspdRate,2; bonus bCritical,2; }}
you get +3 of both all the time, at +7 or greater you get and additional +2 to each (+5 total), and at +9 or greater you get an additional +2 on top of that (+7 total)

^ which is exactly what
Aspd +3%, Critical +3
If refined to +7 or higher, Aspd +2% and Critical +2.
If refined to +9 or higher, Aspd +2% and Critical +2.
would be.

Originally posted by hendra814
i'm already testing, i want to know the right script

if curently my critical and aspd is 43, 183 when equip the item become 46 critical point and 186 aspd point

when i'm equip this item at refine +7 my char will have 48 critical point and 188 aspd point
and when refine into +9 my char still got a same point

18597,Mercury_Riser,Mercury Riser,5,40,,200,,10,,1,0xFFFFFFFF,7,2,256,,0,,759,{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() == 7 || getrefine() == 8) { bonus bAspdRate,2; bonus bCritical,2; } if(getrefine() >= 9) { bonus bAspdRate,2; bonus bCritical,2; }},{},{}


or will receive 50 critical point and 190 aspd point.

18597,Mercury_Riser,Mercury Riser,5,40,,200,,10,,1,0xFFFFFFFF,7,2,256,,0,,759,{ bonus bAspdRate,3; bonus bCritical,3; if(getrefine() == 7 || getrefine() == 8) { bonus bAspdRate,2; bonus bCritical,2; } if(getrefine() >= 9) { bonus bAspdRate,4; bonus bCritical,4; }},{},{}


this is the result i'm already tested at my server

Originally posted by ForteXX
Thats because using the first script, you only get +2 at refined to 9+, since the first upgrade bonus is only applied at +7 and +8 (getrefine() == 7 || getrefine() == 8), and not at anything higher. The second one gives +2 only at +7 and +8, but then changes to giving +4 at upgrades of 9+.

The script I posted also works, more fitting to the description, by giving +2 when upgraded to 7+, and then giving another +2 (rather than a straight +4) when upgraded to 9+.

Originally posted by MarkZD

Im thinking the script should continue working past +12, but stop increasing at that point, not stop working altogether (see other scripts with these kind of effects) [beret]

Bump.

Edited by MarkZD, 15 June 2012 - 06:45 PM.


Originally posted by Flaid
Fixed in r16307

Originally posted by hendra814
@flaid you did wrong fixed for cat ears beret items at sql files.

(18600,'Cat_Ear_Beret','Cat Ear Beret',5,30,NULL,200,NULL,5,NULL,1,0xFFFFFFFF,7,2,256,NULL,0,1,761,'bonus bAspdRate,3; bonus bCritical,3; if(getrefine() >= 7) { bonus bAspdRate,2; bonus bCritical,2; } if(getrefine() >= 9) { bonus bAspdRate,2; bonus bCritical,2;}',NULL,NULL);

<


i think that's script for mercury riser

Originally posted by Flaid
I highly apologize, it's kind of late, it has got the correct script now. Fixed/optimized in r16310