Jump to content

EQI Constant Refactor


Aleos

Recommended Posts


  • Group:  Development Manager
  • Topic Count:  56
  • Topics Per Day:  0.01
  • Content Count:  732
  • Reputation:   525
  • Joined:  12/13/11
  • Last Seen:  

EQI Constant Refactor
As of 2bf64c9, rAthena has normalized equipment constants.

This should have been announced a bit earlier and since a few bug reports have come in recently we'd like to just give a quick announcement for it now!

The purpose of this refactor was to match the database EQI constants with the ones in source. The source had it's own EQP constant structure created by Aegis. The two arrays caused issues with various things in source, so this commit clears that up!

There is one thing to note. The following script commands may be checking for the wrong position if you used an integer value for custom items or scripts (Read here for the constant value).

  • getequipid
  • getequipuniqueid
  • getequipname
  • getequipisequiped
  • getequipisenableref
  • getequiprefinerycnt
  • getequipweaponlv
  • getequippercentrefinery
  • successrefitem
  • failedrefitem
  • downrefitem
  • delequip
  • breakequip
  • getequipcardcnt
  • successremovecards
  • failedremovecards
  • getequipcardid
  • unequip

Either adjust to the constant value or make sure your integer value matches with the correct position you are checking for.

  • Upvote 5
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  192
  • Reputation:   43
  • Joined:  12/13/11
  • Last Seen:  

Thank you for the official statement.
Ran into quite some issues because i didn't know this in the first place ^^

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  6
  • Topics Per Day:  0.00
  • Content Count:  78
  • Reputation:   12
  • Joined:  05/08/16
  • Last Seen:  

Thank you guys!

 

PD: Please clan system!!!

Edited by Darknessfmy
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

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...