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

Show 1 Hp/Sp regen when sitting #2910

Closed
AnnieRuru opened this issue Nov 29, 2020 · 3 comments
Closed

Show 1 Hp/Sp regen when sitting #2910

AnnieRuru opened this issue Nov 29, 2020 · 3 comments

Comments

@AnnieRuru
Copy link
Contributor

Describe the bug
https://herc.ws/board/topic/19152-show-1-regen-hpsp-how-to-remove-it/#comment-99462

To Reproduce

  1. @heal -100
  2. sit
  3. profit

Expected behavior
don't show the '1' '1' '1' when sitting

Screenshots

System specs (please complete the following information):

  • OS: Windows 10 pro 64-bit
  • Hercules Version: latest (always pull latest git)
  • Packet version: 20190530
  • Mode: Renewal
  • Client type: 2019-05-30aRagexeRE
  • Branch: Main

Plugins used or source modifications
many people has reproduce this issue, so don't need to tell isn't it ?

Additional context
I actually also reproduce this issue since 2 months ago, just didn't bother to report this

the answer is on the forum, but dev needs to take a look

@Daraen1
Copy link

Daraen1 commented Nov 30, 2020

I am the author of the initial problem. My teammate and I found the solution it is available on the forum directly as AnnieRuru says.
I also specify that the problem happened on 2018-04-18bRagexeRE and that I personally use the pre-renewal mechanism and that I also have the latest version of Hercules

@dastgirp
Copy link
Member

I noticed this happen normal regen as well, not only when sitting.

@Daraen1
Copy link

Daraen1 commented Dec 16, 2020

Yes, but by changing the original code
if (regen->skill != NULL) { regen->skill->hp = cap_value(regen->skill->hp, 1, INT16_MAX); regen->skill->sp = cap_value(regen->skill->sp, 1, INT16_MAX); } if (regen->sitting != NULL) { regen->sitting->hp = cap_value(regen->sitting->hp, 1, INT16_MAX); regen->sitting->sp = cap_value(regen->sitting->sp, 1, INT16_MAX); }
to
if (regen->skill != NULL) { regen->skill->hp = cap_value(regen->skill->hp, 0, INT16_MAX); regen->skill->sp = cap_value(regen->skill->sp, 0, INT16_MAX); }
if (regen->sitting != NULL) { regen->sitting->hp = cap_value(regen->sitting->hp, 0, INT16_MAX); regen->sitting->sp = cap_value(regen->sitting->sp, 0, INT16_MAX); }

In hercules / src / map / status.c
The problem will be corrected for both, i.e. sitting and standing

hemagx added a commit that referenced this issue Feb 16, 2021
@MishimaHaruna MishimaHaruna added this to the Release v2021.03.08 milestone Mar 9, 2021
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

No branches or pull requests

4 participants