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

[Bug] fixes some prey and taskhunting fixes #644

Merged
merged 25 commits into from
Dec 8, 2022

Conversation

dudantas
Copy link
Member

@dudantas dudantas commented Nov 24, 2022

Description

  • Tasks hunting kill fix:
    If the player is at a party and with experience sharing enabled and active, it will kill everyone who damages the monster
    Font: https://www.tibiaqa.com/17999/how-are-prey-hunting-task-creatures-counted

  • Task hunting buy slot on store fix:
    Reorganized the order of offers by id and added the possibility to buy the hunting tasks slot
    Fixed in init.lua the redirection of the offers ids of the third slot of the hunting task and prey
    Added OFFER_TYPE_HUNTINGSLOT enum to GameStore.OfferTypes
    Fixed for the second slot to be active for premium accounts only

  • Prey fixes:
    Fixed the loot prey, it was giving the percentage of chance more to drop the items, the correct thing is to just give the percentage of chance of dropping all the loot again. For example, if the player kills a demon and drops 1 demon armor and 1 demon helmet, with the preyloot active there will be a percentage chance (defined by the prey) of dropping the loot again. As if the monster dropped loot twice in a row.
    Fixed for the second slot to be active for premium accounts only

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested

  • Tasks hunting kills
    Enter the party and activate experience sharing, everyone who attacks the monster will receive a kill in hunting tasks.
    If the party is not activated, only the last hit will receive the kill.

Test Configuration:

  • Server Version: 1291
  • Client: 1291
  • Operating System: Windows 11

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I checked the PR checks reports
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@dudantas dudantas added Status: Pending Review This PR or Issue requires a review Status: Pending Test This PR or Issue requires more testing labels Nov 25, 2022
@dudantas dudantas changed the title Fix taskhunting kills on party Some prey and taskhunting fixes Nov 29, 2022
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 8, 2022

Please retry analysis of this Pull-Request directly on SonarCloud.

@dudantas dudantas changed the title Some prey and taskhunting fixes [Bug] fixes some prey and taskhunting fixes Dec 8, 2022
@dudantas dudantas merged commit d42abca into main Dec 8, 2022
@dudantas dudantas deleted the fix-taskhunting-kills-in-party branch December 8, 2022 06:16
luan pushed a commit that referenced this pull request Jul 11, 2023
• Tasks hunting kill fix:
If the player is at a party and with experience sharing enabled and active, it will kill everyone who damages the monster
Font: https://www.tibiaqa.com/17999/how-are-prey-hunting-task-creatures-counted

• Task hunting buy slot on store fix:
Reorganized the order of offers by ID and added the possibility to buy the hunting tasks slot
Fixed in "init.lua" the redirection of the offers IDs of the third slot of the hunting task and prey
Added OFFER_TYPE_HUNTINGSLOT enum to GameStore.OfferTypes
Fixed for the second slot to be active for premium accounts only

• Prey fixes:
Fixed the loot prey, it was giving the percentage of chance more to drop the items, the correct thing is to just give the percentage of chance of dropping all the loot again. For example, if the player kills a demon and drops 1 demon armor and 1 demon helmet, with the preyloot active there will be a percentage chance (defined by the prey) of dropping the loot again. As if the monster dropped loot twice in a row.
Fixed for the second slot to be active for premium accounts only
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Datapack Core Area: Datapack Area: Source Status: Pending Review This PR or Issue requires a review Status: Pending Test This PR or Issue requires more testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants