Main | My View | View Issues | Change Log | Wiki | Tavern | News
  

Viewing Issue Advanced Details [ Jump to Notes ] [ Wiki ] [ View Simple ] [ Issue History ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0011692 [DCSS] Bug Report text always 2018-09-23 22:42 2019-10-09 02:17
Reporter damerell View Status public  
Assigned To gammafunk
Priority normal Resolution done Local or Remote Both
Status closed   Operating System All
Projection none   Console or Tiles Both
ETA none Fixed in Branch 0.24 old branch Product Branch 0.22 ancient branch
  Product Version
Summary 0011692: Amulet of the gourmand's message not always appropriate.
Description The amulet of the gourmand, when donned, always says: "You feel a craving for the dungeon's cuisine."

This does not seem appropriate either in the case of already-carnivorous species like kitties, or mummies who cannot eat at all.

While there's a Watsonian argument that a mummy can feel a craving for food it cannot eat, I expect Crawl to generally take a Doylist approach; other items of jewellery (eg faith, teleportation) vary their message when they provide no effect.
Steps To Reproduce
Additional Information
Tags No tags attached.
Attached Files

- Relationships

-  Notes
(0033497)
john_science (reporter)
2019-10-08 20:47

 This feels like a design choice to me. The current approach everywhere in this game is to have an item/spell/ability tell you what it does, and it is up to you to know if that means anything for your character.

Do we want amulets lying around that say "This amulet does nothing for you"? What if at some point there is a spell that makes you temporarily carnivorous, should the text on the amulet change as you go in and out of a carnivorous state? To me, that sounds confusing.

Alternately, we could change the name from "Amulet of the gourmand" to "Amulet of Uselessness" for Felids and Mummies, and call it a day.
(0033498)
Goratrix (reporter)
2019-10-08 21:25

 This is actually already fixed:

https://github.com/crawl/crawl/blob/bfa02ace7b0344c25e6f2ac9491636ff137fe48b/crawl-ref/source/player-equip.cc#L1274 [^]

By the way, you should be aware there are tons of issues on this tracker that were already fixed, it's always better to check the source or verify in wizard mode :) Or search the commits on github: https://github.com/crawl/crawl/commit/394fb5221806c9948a325b1f74aee27f6a8f31d0 [^]
(0033499)
gammafunk (administrator)
2019-10-09 01:58

 Fixed in 0.23-a0-400-g394fb52

- Issue History
Date Modified Username Field Change
2018-09-23 22:42 damerell New Issue
2019-10-08 20:47 john_science Note Added: 0033497
2019-10-08 21:25 Goratrix Note Added: 0033498
2019-10-09 01:58 gammafunk Note Added: 0033499
2019-10-09 01:58 gammafunk Status new => resolved
2019-10-09 01:58 gammafunk Fixed in Branch => 0.24 development branch
2019-10-09 01:58 gammafunk Resolution open => done
2019-10-09 01:58 gammafunk Assigned To => gammafunk
2019-10-09 02:16 damerell Status resolved => closed

Mantis 1.1.8[^]
Copyright © 2000 - 2009 Mantis Group
Powered by Mantis Bugtracker