Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Itemdb (Ragcan's JRO item) #4115

Closed
wants to merge 22 commits into from
Closed

Update Itemdb (Ragcan's JRO item) #4115

wants to merge 22 commits into from

Conversation

attackjom
Copy link
Contributor

@attackjom attackjom commented Apr 16, 2019

  • Addressed Issue(s): -
  • Server Mode: Renewal
  • Description of Pull Request:
  • Implement bLongSPGainvalue and bLongHPGainvalue
  • Add new RC2 that use in itemscript
  • Ragcan's JRO itemdb
  • Info take from

@MrLekkz
Copy link

@MrLekkz MrLekkz commented Apr 18, 2019

Thanks @attackjom
I found one mistake: Ladon's Skin (20949) should increase hit by 5 and not by 15.
Apart from that, everything seems fine.
GJ!

@attackjom
Copy link
Contributor Author

@attackjom attackjom commented Apr 18, 2019

Thanks @attackjom
I found one mistake: Ladon's Skin (20949) should increase hit by 5 and not by 15.
Apart from that, everything seems fine.
GJ!

done.

@@ -349,6 +379,19 @@
2990:2991,{ bonus bAllStats,5; bonus5 bAutoSpellWhenHit,"SC_MAELSTROM",1,100,BF_MAGIC,0; }
2995:15110,{ bonus2 bSkillAtk,"NC_AXEBOOMERANG",100; }
2997:2999,{ bonus2 bAddClass,Class_All,10; bonus bMatkRate,10; }
32242:18971,{ bonus2 bFixedCastrate,"RK_STORMBLAST",-100; }
32242:18973,{ bonus2 bSkillAtk,"NC_FLAMELAUNCHER",70; bonus2 bSkillAtk,"NC_COLDSLOWER",70; }
Copy link

@MrLekkz MrLekkz Apr 18, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on iRO, Old Detachments Ring has a set bonus with Old Mitra (18972) (50% hp bonus)
32242:18972,{ bonus bMaxHPrate,50; }
can't confirm if its iRO exclusive or not

Copy link
Contributor Author

@attackjom attackjom Apr 19, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seem I forget that haha.

Copy link
Contributor Author

@attackjom attackjom Apr 20, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

28430,Arch_Bishop_Ring,Arch Bishop Ring,4,20,,400,,0,,1,0xFFFFFFFF,63,2,136,,90,0,0,{ bonus bLuk,2; bonus bFlee2,5; bonus bUseSPrate,-5; bonus bHealPower,15; bonus bAddItemHealRate,15; },{},{}
28433,Luminous_Blue_Stone,Luminous Blue Stone,4,20,,400,,0,,1,0xFFFFFFFF,63,2,136,,90,0,0,{ bonus bAgi,2; bonus bCritical,5; bonus bAspdRate,5; bonus bCritAtkRate,2; skill "MG_SIGHT",1; if (readparam(bLuk)>=100) { bonus bCritical,10; bonus bCritAtkRate,10; } },{},{}
28434,Fist_Fighter_Glove,Fist Fighter's Glove,4,20,,400,,0,,1,0xFFFFFFFF,63,2,136,,90,0,0,{ bonus bStr,2; bonus bMaxSP,100; bonus2 bAddClass,Class_All,2; skill "MO_ABSORBSPIRITS",1; if (readparam(bStr)>99) bonus2 bAddClass,Class_All,5; },{},{}
28437,Hibram's_Gloves,Hibram's Gloves,4,20,,400,,0,,1,0xFFFFFFFF,63,2,136,,90,0,0,{ skill "AL_HEAL",3; bonus bInt,2; bonus bMatkRate,2; bonus bUseSPrate,-5; if (readparam(bInt>=100)) bonus bMatkRate,5; },{},{}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if (readparam(bInt>=100)) should be if (readparam(bInt)>=100)

Copy link
Contributor Author

@attackjom attackjom May 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@attackjom attackjom closed this May 11, 2019
@Atemo
Copy link
Contributor

@Atemo Atemo commented May 11, 2019

hmm why did you close the PR?

@attackjom
Copy link
Contributor Author

@attackjom attackjom commented May 11, 2019

Mistake in editing master branch
so, I decide to remake fork

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants