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

Reload Monsters/Spells #93

Closed
LosRechos opened this issue Apr 25, 2021 · 8 comments · Fixed by #329
Closed

Reload Monsters/Spells #93

LosRechos opened this issue Apr 25, 2021 · 8 comments · Fixed by #329
Labels
Area: Source Priority: High Represent a high impact in key areas of the base/user experience Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@LosRechos
Copy link

The bug:
The bug happens when I use the / reload command. For some reason, when using the command /reload monster and later /reload spells, some monsters change the spells registered in their files.

The way I found to reproduce the bug:

  1. Set a monster to use sd
    {name ="sudden death", interval = 2000, chance = 100, minDamage = -2000, maxDamage = -2300, range = 5, target = true}
  1. Let him use the spell on some player
  2. use /reload monsters and right after use /reload spells
  3. The monster will start using another spell instead of sd, this can cause some crash

Screenshots

The monster is using sd as it should:
image

After using /reload monsters and /reload spells 2x:
image

2021-04-25.03-10-21.mp4

Environment

Using the last version on Windows

No error show up in the distro

@beats-dh
Copy link
Collaborator

beats-dh commented Jun 8, 2021

Has this not been resolved?

@Peixonauta2
Copy link
Contributor

Since nobody answered it, I guess it hasn't.

@github-actions
Copy link
Contributor

This issue is stale because it has been open 120 days with no activity.

@ArturKnopik
Copy link
Contributor

Is there still a problem after moving spells to revscript?

@beats-dh
Copy link
Collaborator

Is there still a problem after moving spells to revscript?

Yes, because monsters store spells and after a /reload spells or /reload scripts it breaks.

@dudantas
Copy link
Contributor

dudantas commented Oct 6, 2021

Your issue will be transferred to the canary repository

@dudantas dudantas transferred this issue from opentibiabr/otservbr-global-archived Oct 6, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2022

This issue is stale because it has been open 120 days with no activity.

@github-actions github-actions bot added the Stale No activity label Jan 5, 2022
@andersonfaaria
Copy link
Contributor

Seems to be a duplicate of #87
when you apply the reload spells monster loses the reference and hence why they are changing the spells

@andersonfaaria andersonfaaria changed the title BUG: Reload Monsters/Spells Reload Monsters/Spells Apr 6, 2022
@andersonfaaria andersonfaaria added Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors. Priority: High Represent a high impact in key areas of the base/user experience Area: Source and removed Stale No activity labels Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Source Priority: High Represent a high impact in key areas of the base/user experience Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
6 participants