Jump to content

Regarding bug #5018


Ind

Recommended Posts


  • Group:  Members
  • Topic Count:  169
  • Topics Per Day:  0.04
  • Content Count:  1260
  • Reputation:   750
  • Joined:  11/19/11
  • Last Seen:  

bugreport:5018

The server doesn't save the skill flag anymore. I remember some years ago the skill table would save the skill flag, it doesn't anymore though. and without it I don't have much clue on how to find out if a skill was given permanently through script or not (behavior currently broken) any ideas?

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  11
  • Topics Per Day:  0.00
  • Content Count:  427
  • Reputation:   123
  • Joined:  11/17/11
  • Last Seen:  

Maybe related: r14861.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  12
  • Topics Per Day:  0.00
  • Content Count:  117
  • Reputation:   167
  • Joined:  11/10/11
  • Last Seen:  

Indeed, temporary skill is stored as (skill.flag - SKILL_FLAG_REPLACE_LV_0), as far as I know skills granted permanently through scripts are removed when the player skill tree is normalized. In regards to that bug-report, all issues can be resolved by appending the skill onto the player skill tree: thus not being lost upon tree normalization.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  169
  • Topics Per Day:  0.04
  • Content Count:  1260
  • Reputation:   750
  • Joined:  11/19/11
  • Last Seen:  

Indeed, temporary skill is stored as (skill.flag - SKILL_FLAG_REPLACE_LV_0), as far as I know skills granted permanently through scripts are removed when the player skill tree is normalized. In regards to that bug-report, all issues can be resolved by appending the skill onto the player skill tree: thus not being lost upon tree normalization.

well thats not quite uhm a fix to the behavior imo. anyone see anything better than having the flag saved again? (i probably wouldn't have it in the skill table since 99% of skills don't have flags, perhaps a additional table for it)

Link to comment
Share on other sites

  • 10 months later...

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

This has been fixed in r17136.

Link to comment
Share on other sites

×
×
  • Create New...