MineColonies

MineColonies

57M Downloads

Carpenter abilities are scalling off incorrect skills

starmun-0010 opened this issue ยท 1 comments

commented
  • I am running the latest alpha version of MineColonies and Structurize for my Minecraft version.
  • I checked the MineColonies/Structurize wiki and made sure my issue is not covered here.
  • I made sure that this issue is not a duplicate of any existing issue.

Context

From the wiki:
image

Which makes Dexterity to be primary skill and Knowledge secondary. But from the code in BuildingSawmill it is exatly the opposite.

    public Skill getPrimarySkill()
    {
        return Skill.Knowledge;
    }
    public Skill getSecondarySkill()
    {
        return Skill.Dexterity;
    }
  • Minecraft Version: 1.16.5
  • MineColonies Version: 0.15.328-ALPHA
  • Structurize Version: 0.13.208-ALPHA
  • Related Mods and their Versions:

Expected behavior

Primary skill should return Dexterity and secondary should return Knowledge.

Actual behavior

The methods return incorrect skill values.


Viewers

  • Add a thumbs-up to the bug report if you are also affected. This helps the bug report become more visible to the team and doesn't clutter the comments.
  • Add a comment if you have any insights or background information that isn't already part of the conversation.
commented

this is actually an issue with the wiki not the mod, open an issue here https://github.com/ldtteam/MinecoloniesWiki/issues