MineColonies

MineColonies

Mods
56M Downloads

Add Tinkers Construct Weaponry to Guards [$15]

kedamono opened this issue · 5 comments

commented

Minecolonies version

version 0.4.1710

Expected behavior

  • The Guards colonists would use Tinkers Construct Weapons

Actual behaviour

  • They ignore them when placed into their inventory.

Steps to reproduce the problem

  1. Create a Tinkers Flint Sword
  2. Recall a Guard and remove his current weapon from his inventory.
  3. Add the Tinkers Flint Sword to his inventory
  4. Result: He requests a weapon and doesn't specify what kind.

Did you help close this issue? Go claim the $15 bounty on Bountysource.

commented

Yep, can their weapon classes be added to the Guards "acceptable weapons" list as well?
I was gonna suggest it too. The Longsword, Broadsword, Rapier and Cleaver?

commented

That's very strange. I guess Tinker construct didn't define their swords as
weapons following minecraft classes.

2016-11-12 14:13 GMT-02:00 kedamono [email protected]:

Minecolonies version

version 0.4.1710
Expected behavior

  • The Guards colonists would use Tinkers Construct Weapons

Actual behaviour

  • They ignore them when placed into their inventory.

Steps to reproduce the problem

  1. Create a Tinkers Flint Sword
  2. Recall a Guard and remove his current weapon from his inventory.
  3. Add the Tinkers Flint Sword to his inventory
  4. Result: He requests a weapon and doesn't specify what kind.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#279, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AGI9y-f_6yiriX1lrzFPJ1d2F9Rm-uioks5q9eWMgaJpZM4KwdGK
.

  • 😄1
commented

I just had one of my Miner characters not recognize the flint pickaxe that she had been using just a moment before. I think it's just a glitch, but...

commented

Maybe the materials are confusing them? Did they request a certain mining level and flint isn't up to that mining level?

I definitely had a team of citizens using bronze TiCon tools (bronze because I had forestry running with the testpack and it adds that alloy) and they accepted the tools as valid.

commented

As Kostronor stated in another issue: "We now have tinkers integration so having a check for tinkers slime leaves is no problem"

Would this also make it possible now for the guards to recognize and use the Tinkers weapons?
Longbow, Crossbow, Longsword, Broadsword, Rapier and Cleaver?