minetest-whynot / whynot-game

Minetest game in minetest mods collection style
GNU General Public License v3.0
14 stars 7 forks source link

[Mod Request]: `anvil` #114

Open Lazerbeak12345 opened 2 years ago

Lazerbeak12345 commented 2 years ago

Request

This is the URL for the mod I would like to be added:

https://content.minetest.net/packages/FaceDeer/anvil/

My opinion on how it fits with the whynot rules:

Refer to the Whynot Readme for full rule descriptions/reasons.

  1. In what ways might this mod take over the game?
    • Never, unless code says otherwise
  2. When could this mod be a strain on the server when no players are using the mod?
    • Never
  3. When does this mod destroy player's work?
    • Never
  4. Have you looked at the code? If so, what stood out as things that might need fixing?
    • CODE REVIEW NEEDED
  5. In what way might this mod be reduced to be more simple (as in "Keep it Simple Stupid") (ex: "the foobar mod could be made more simple by splitting into two mods, foo and bar")
    • As far as I'm aware this mod is simple enough. CODE REVIEW NEEDED
  6. Is this mod survival freindly? What items that it provides that should be craftable/obtainable, but arent?
    • CODE REVIEW NEEDED
  7. When does this mod feel like cheating?
    • If you have two hammers, you can repair any tool instantly and infinitely. (fixable with setting)
  8. Does this mod use the software "git" for version control? (note: we are asking about git. Github, Gitlab, notabug and hundreds of other git providers exist but are not specifically needed, although these do qualify).
    • Yes
  9. Upon testing this mod, what errors, odd behavior, or other incompatibilities were noticed?
    • None
  10. Legal
    • Licence is GPLv3-only. This is compatible

Other comments

Requested in this review on content db

Lazerbeak12345 commented 1 year ago

Updated legal.

Lazerbeak12345 commented 1 year ago

Currently going through survival playthrough.

So far, I don't think having both toolranks and Anvil is a problem.

I have yet to test if Anvil resets or otherwise interacts with toolranks. These interactions will effect balancing.

Lazerbeak12345 commented 1 year ago

We will need to set anvil_hammer_is_repairable false. If we don't, once a player crafts two hammers, they can infinitely repair anything at zero cost.

Lazerbeak12345 commented 1 year ago

Ok, playtest finished.

When an anvil repairs a toolranks tool, it plays nice and doesn't remove the toolranks attributes. This means that anvil can be seen as a quick and cheap, short-term fix for your tool, while toolranks can be seen as a long-term investment into that tool.