0000423: Leather Armor Material Type - Dwarf Fortress Bug Tracker

Notes
(0007092) derigo (reporter) 2010-05-20 14:36	I don't think this is a bug. "Leather leather armor" means an item of the type 'leather armor' made out of any kind of leather, as opposed to say cow leather leather armor. I realize there is an 'any material' option which effectively does the same thing, but thats ok. That only happens because its such a specific item to begin with(it can only be made out of leather). And no, you can't specify 'metal leather armor' because the item leather armor has the [LEATHER] token. It can neither be made out of metal nor specified to be metal in the materials assignments. Nor can you make it out of adamantine or shell.

(0007094) Footkerchief (manager) 2010-05-20 14:49	This is still a bug in my opinion, if only because that screen is misusing [MATERIAL_PLACEHOLDER:leather] (see item_armor.txt). It should only use the placeholder when the material is completely ambiguous, but on this screen, it's using it alongside a material category, resulting in the awkward "leather leather".

(0007095) derigo (reporter) 2010-05-20 15:02	Oh I see what you mean. There's no such thing as leather armor, really. Its just called 'armor' with the material placeholder in front. Yea I agree, the placeholder should be replaced by the material, unless the material is completely unspecified.

(0031649) Detros (manager) 2015-01-02 19:14	Still in 40.23. There is both "armor" and "leather armor" in "A: Armor" category of [m]-[n] uniforms screen. There should be just the first one, the second one should be done only through the combination of Armor: "armor" and Material: "leather".

(0031652) Rafal99 (reporter) 2015-01-02 21:04 edited on: 2015-01-02 21:06	"armor" in "A: Armor" category means any item worn on upper body, just like "headwear" in "H: Helm" category means any headwear. So "armor" and "leather armor" are different options. There is a potential problem, such that "armor" with material set to "leather" will probably look the same as "leather armor" with no material set, but will have different meaning. Both with accept "leather leather armor", but the first one will also accept leather shirts or the like, while the second one won't.

(0031662) Detros (manager) 2015-01-03 08:48	Sure, "armor" and "leather armor" are different options. And yes, "leather armor" and "leather"+"armor" look the same. The main problem here IMO is that material part is offered out of Material section of uniform settings. I don't know about any such thing there so I think "leather armor" should be removed from Armor options. As it is now, it is not systematic.

Issue History
Date Modified	Username	Field	Change
2010-04-04 15:09	Draco18s	New Issue
2010-04-04 16:37	Qloos	Tag Attached: text
2010-04-04 16:37	Qloos	Tag Attached: Uniform
2010-05-20 11:37	Footkerchief	Relationship added	related to 0001995
2010-05-20 14:36	derigo	Note Added: 0007092
2010-05-20 14:49	Footkerchief	Note Added: 0007094
2010-05-20 15:02	derigo	Note Added: 0007095
2011-03-21 12:58	Footkerchief	Relationship added	related to 0001300
2014-11-10 02:38	Larix2	Note Added: 0030938
2014-11-10 02:40	Larix2	Note Deleted: 0030938
2015-01-02 19:14	Detros	Note Added: 0031649
2015-01-02 19:19	Detros	Issue Monitored: Detros
2015-01-02 19:50	Footkerchief	Assigned To	=> Footkerchief
2015-01-02 19:50	Footkerchief	Status	new => confirmed
2015-01-02 21:04	Rafal99	Note Added: 0031652
2015-01-02 21:05	Rafal99	Note Edited: 0031652	View Revisions
2015-01-02 21:05	Rafal99	Note Edited: 0031652	View Revisions
2015-01-02 21:06	Rafal99	Note Edited: 0031652	View Revisions
2015-01-03 08:48	Detros	Note Added: 0031662