Anonymous | Login | Signup for a new account | 2024-11-09 12:58 PST |
Main | My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||
ID | Project | Category | View Status | Date Submitted | Last Update | |
0001044 | Dwarf Fortress | Dwarf Mode -- Trade | public | 2010-04-12 06:42 | 2015-02-18 03:29 | |
Reporter | slink | |||||
Assigned To | Toady One | |||||
Priority | low | Severity | minor | Reproducibility | have not tried | |
Status | resolved | Resolution | fixed | |||
Platform | OS | OS Version | ||||
Product Version | 0.31.02 | |||||
Target Version | Fixed in Version | 0.40.06 | ||||
Summary | 0001044: Human Traders Brought "liquid" | |||||
Description | As title. Cost was 20 dwarf-bucks for 10 units of an unspecified liquid (but not an "unknown material") weighing 11 dwarf-kg in a pine barrel. I did not buy it because I had no trade goods, but I saved a copy of the game with those traders in the depot in case someone wants me to do something. | |||||
Tags | material, Save Included, Save Needs Testing, still affects migrated saves, trade | |||||
Attached Files | ||||||
Relationships | |||||||||||||||||||||
|
Notes | |
(0003051) Footkerchief (manager) 2010-04-12 08:53 |
The save may be helpful, yeah. http://dffd.wimbli.com/ [^] |
(0003092) slink (reporter) 2010-04-12 11:37 |
http://dffd.wimbli.com/file.php?id=2078 [^] |
(0004164) drunken (reporter) 2010-04-18 19:36 |
i have a liquid barrel containing 12 liquid, just confirming |
(0004850) DDR (reporter) 2010-04-24 00:14 |
Confirmed! |
(0004854) Footkerchief (manager) 2010-04-24 00:42 edited on: 2010-04-24 00:44 |
drunken and DDR: does it crash if you view the "barrels" or "liquid" sections of the Stocks menu? |
(0006940) Logical2u (manager) 2010-05-18 04:56 |
zergl from 0001940 uploaded this save - http://www.wikifortio.com/791077/region2.7z [^] |
(0007304) zergl (reporter) 2010-05-24 16:56 |
Sorry about opening a new bug for that, I forgot I still had some other filters on when searching if this was already reported. Oops. Raws were all completely vanilla in my save so it has nothing to do with modding and worldgen and playing was done completely in 0.31.04. Also in my save there's loads of trade goods in the depot so anyone trying to have a closer look at it can just buy it for further experimentation. And @Footkerchief: no crashing at all when I play around in the stocks menu before or after buying it for me. |
(0008841) ChickenLips (reporter) 2010-06-20 19:34 |
Got a save here as well (.08) with a human trader with a "+liquid barrel (palm)+", containing 10 liquid. Checked the liquid and barrel sections of the stocks, and viewed details on each item; no crash. Save available if needed. |
(0009896) ellindsey (reporter) 2010-07-10 20:02 |
Just got this bug myself, in 31.09. The humans bought a "liquid Barrel (willow)" containing liquid[10] with value 10 and weight <1, and the description "This is a stack of 10 liquid." Purchasing it did not trigger a crash. Unmodded game, started on an earlier version of DF and copied the save game files over to DF 31.09. |
(0009913) shibdib (reporter) 2010-07-10 21:44 |
could it be a barrel of water? get that barrel in a closed area with a dwarf and see if he drinks it |
(0011562) GauHelldragon (reporter) 2010-08-06 14:29 |
I also had this bug in 31.12 It is classified as a 'liquid' in the stocks screen. |
(0011569) toybasher (reporter) 2010-08-06 17:12 |
someone needs to abandon the fort, and visit it in adventure mode Then we MIGHT be able to see whats inside that barrel (if not we could still try and drink from it.) |
(0013307) Quietust (reporter) 2010-10-10 12:06 |
In a fortress with modified raws (to readd Human merchant nobles), the human Guild Representative permitted me to request "liquid" at the very bottom of the Extracts section, after all of the different types of blood and the several types of milk. |
(0013552) troas (reporter) 2010-10-29 11:32 |
This is still happening in 31.16. Bugged save available at http://dffd.wimbli.com/file.php?id=3340 [^] |
(0014519) Dwarfu (manager) 2010-12-10 21:21 |
Loaded slink's original save in 31.18. Didn't see any duplicate raw files. Was able to view the liquid without crashing. |
(0018195) Dwarfu (manager) 2011-07-10 11:35 edited on: 2011-07-10 11:49 |
I went back and tested animals with various extracts and liquids as part of their definition. If the sea lamprey's SPECIALATTACK_SUCK_BLOOD tag is removed, I no longer have been getting the generic 'liquid' on the embark screens (playing as humans for testing). I tested on the same gen seeds throughout multiple trials. Can anyone else confirm? EDIT: This would apply to nightwings also, but since they are FANCIFUL, their products don't show on embark. |
(0018602) Dwarfu (manager) 2011-08-21 09:52 |
Also requires the removal of LEECHMAN's SPECIALATTACK_SUCK_BLOOD tag (I had previously removed them from my game for testing and didn't realize it). |
(0020528) norgg (reporter) 2012-02-23 03:48 edited on: 2012-02-23 04:07 |
Some dwarven traders just turned up with a barrel of liquid for me in 34.02. Save at http://norgg.org/files/The%20Land%20of%20Oracles%20-%20Barrel%20of%20Liquid.zip [^] |
(0020534) Granite26 (reporter) 2012-02-23 05:17 |
I've seen it in Vanilla 34.02 as well |
(0021708) Caode (reporter) 2012-03-24 16:18 |
Confirmed for vanilla 34.05 as well. |
(0022393) hyp3r4ctive (reporter) 2012-04-29 02:52 |
Vanilla 34.07 confirmed as well. Can view it without crash. I can only imagine the dwarves checking it: "Not blood...not alcohol...but it's a fluid...It's a liquid everybody, my jobs done" |
(0022588) Quietust (reporter) 2012-05-18 06:18 edited on: 2012-05-18 06:21 |
Some memory hacking reveals that the liquid's material is NONE:NONE (i.e. both material type and subtype are -1) within the entity's resource lists (used directly during trade negotiations and presumably other tasks as well) - if I change the values to something more reasonable, the "Liquid" entry in the trade agreement screen changes accordingly. |
(0024312) TBeholder (reporter) 2014-01-21 21:47 edited on: 2014-01-21 21:47 |
Wait, it's interesting. Only _sea lamprey_ and _leech man_ need to be removed? Okay, _nightwing_ is [EVIL] so humans won't get it, much like sliver barb. But _tick_, _tick man_, _mosquito_, _mosquito man_ also have blood sucking, only via "[APPLY_CREATURE_VARIATION:*_SUCK_ATTACK]" (leech man have both versions which may be a munor bug in itself). They don't generate "Liquid"/NONE:NONE? |
(0024627) Dwarfu (manager) 2014-03-25 14:53 |
TBeholder, it is quite possible they do. |
(0024799) Quietust (reporter) 2014-06-04 12:00 edited on: 2014-06-04 12:04 |
After having examined 0.34.11's code for initializing entity resources, I now know exactly why this happens: If the civilization has USE_ANIMAL_PRODUCTS, it will, for each creature it has access to, attempt to add all materials used by SPECIALATTACKs to its list of tradeable extracts. However, this section fails to check whether the special attack is of type INJECT_EXTRACT, SUCK_BLOOD, or INTERACTION - it just adds all materials specified. The problem is that the other SPECIALATTACK types insert dummy values of -1 into the material/matgloss lists (in order to keep them properly aligned), so those dummy values end up getting added to the entity. In addition to not checking the SPECIALATTACK type, it is also neglecting to check the extract material's state (it's theoretically possible for the extract to be injected as a solid or a gas). By comparison, the logic immediately afterward for handling SECRETIONs correctly takes this into account. |
Issue History | |||
Date Modified | Username | Field | Change |
2010-04-12 06:42 | slink | New Issue | |
2010-04-12 08:53 | Footkerchief | Note Added: 0003051 | |
2010-04-12 08:55 | Kennel | Tag Attached: material | |
2010-04-12 08:55 | Kennel | Tag Attached: trade | |
2010-04-12 11:37 | slink | Note Added: 0003092 | |
2010-04-12 14:00 | Footkerchief | Relationship added | related to 0001052 |
2010-04-18 19:36 | drunken | Note Added: 0004164 | |
2010-04-20 19:27 | Footkerchief | Relationship replaced | child of 0001052 |
2010-04-20 19:27 | Footkerchief | Relationship deleted | child of 0001052 |
2010-04-20 19:29 | Footkerchief | Relationship added | child of 0000446 |
2010-04-24 00:14 | DDR | Note Added: 0004850 | |
2010-04-24 00:42 | Footkerchief | Note Added: 0004854 | |
2010-04-24 00:43 | Footkerchief | Issue Monitored: Footkerchief | |
2010-04-24 00:44 | Footkerchief | Note Edited: 0004854 | View Revisions |
2010-05-18 04:55 | Logical2u | Relationship added | has duplicate 0001940 |
2010-05-18 04:56 | Logical2u | Note Added: 0006940 | |
2010-05-24 16:56 | zergl | Note Added: 0007304 | |
2010-06-13 16:16 | Footkerchief | Relationship replaced | related to 0000446 |
2010-06-13 16:21 | Footkerchief | Relationship replaced | child of 0000446 |
2010-06-20 19:34 | ChickenLips | Note Added: 0008841 | |
2010-07-10 20:02 | ellindsey | Note Added: 0009896 | |
2010-07-10 21:44 | shibdib | Note Added: 0009913 | |
2010-07-11 10:07 | shibdib | Issue Monitored: shibdib | |
2010-08-06 14:29 | GauHelldragon | Note Added: 0011562 | |
2010-08-06 17:12 | toybasher | Note Added: 0011569 | |
2010-10-10 12:06 | Quietust | Note Added: 0013307 | |
2010-10-29 11:32 | troas | Note Added: 0013552 | |
2010-10-29 13:36 | Logical2u | Relationship added | has duplicate 0003481 |
2010-12-10 20:44 | Dwarfu | Tag Attached: Save Included | |
2010-12-10 21:21 | Dwarfu | Note Added: 0014519 | |
2010-12-10 21:21 | Dwarfu | Tag Attached: Save Needs Testing | |
2010-12-11 12:15 | Dwarfu | Relationship deleted | child of 0000446 |
2010-12-11 12:16 | Dwarfu | Relationship added | parent of 0002311 |
2011-03-22 08:43 | Footkerchief | Relationship added | has duplicate 0004304 |
2011-07-10 11:35 | Dwarfu | Note Added: 0018195 | |
2011-07-10 11:35 | Dwarfu | Issue Monitored: Dwarfu | |
2011-07-10 11:35 | Dwarfu | Note Edited: 0018195 | View Revisions |
2011-07-10 11:39 | Dwarfu | Note Edited: 0018195 | View Revisions |
2011-07-10 11:49 | Dwarfu | Note Edited: 0018195 | View Revisions |
2011-07-10 12:11 | Dwarfu | Relationship deleted | parent of 0002311 |
2011-08-21 09:52 | Dwarfu | Note Added: 0018602 | |
2012-02-23 03:48 | norgg | Note Added: 0020528 | |
2012-02-23 04:07 | norgg | Note Edited: 0020528 | View Revisions |
2012-02-23 05:17 | Granite26 | Note Added: 0020534 | |
2012-03-24 16:18 | Caode | Note Added: 0021708 | |
2012-04-29 02:52 | hyp3r4ctive | Note Added: 0022393 | |
2012-05-18 06:18 | Quietust | Note Added: 0022588 | |
2012-05-18 06:20 | Quietust | Note Edited: 0022588 | View Revisions |
2012-05-18 06:21 | Quietust | Note Edited: 0022588 | View Revisions |
2014-01-21 21:47 | TBeholder | Note Added: 0024312 | |
2014-01-21 21:47 | TBeholder | Note Edited: 0024312 | View Revisions |
2014-03-25 14:53 | Dwarfu | Note Added: 0024627 | |
2014-03-25 14:53 | Dwarfu | Assigned To | => Dwarfu |
2014-03-25 14:53 | Dwarfu | Status | new => acknowledged |
2014-06-04 12:00 | Quietust | Note Added: 0024799 | |
2014-06-04 12:01 | Quietust | Note Edited: 0024799 | View Revisions |
2014-06-04 12:04 | Quietust | Note Edited: 0024799 | View Revisions |
2014-06-04 12:04 | Quietust | Note Edited: 0024799 | View Revisions |
2014-07-18 08:48 | Footkerchief | Relationship added | related to 0000753 |
2014-07-28 15:54 | Dwarfu | Status | acknowledged => confirmed |
2014-07-29 11:06 | Toady One | Status | confirmed => resolved |
2014-07-29 11:06 | Toady One | Fixed in Version | => Next Version |
2014-07-29 11:06 | Toady One | Resolution | open => fixed |
2014-07-29 11:06 | Toady One | Assigned To | Dwarfu => Toady One |
2014-07-29 11:17 | Dwarfu | Issue End Monitor: Dwarfu | |
2014-08-09 13:19 | Loci | Tag Attached: not fixed | |
2014-08-09 13:38 | Dwarfu | Issue Monitored: Dwarfu | |
2014-08-09 13:44 | Loci | Tag Detached: not fixed | |
2014-08-09 13:45 | Loci | Tag Attached: still affects migrated saves | |
2015-02-18 03:29 | Dwarfu | Issue End Monitor: Dwarfu |
Copyright © 2000 - 2010 MantisBT Group |