View Issue Details

IDProjectCategoryView StatusLast Update
0003577The Dark ModCodingpublic20.10.2013 20:19
ReporterMortem Desino Assigned ToMortem Desino  
PrioritynormalSeveritynormalReproducibilityalways
Status closedResolutionfixed 
Product VersionTDM 2.00 
Target VersionTDM 2.01 
Summary0003577: mixed inv_persistent items all carry over
DescriptionMixing inv_persistent 0 & 1 items breaks inventory count in next mission. inv_persistent 1 seems to override all others in the stack.
Steps To ReproduceOpen a campaign set (e.g. the SVN tdm_campaign_sample FM)

If I "spawn atdm:playertools_health_potion inv_persistent 1", I get proper carryover in the next mission.
If I "spawn atdm:playertools_health_potion inv_persistent 0", I get proper behavior (no carryover) in the next mission.

If I spawn one of both, I get both carrying over in the next mission (i.e. I own 2 health potions [where I should have one]).
Additional InformationThe erroneous extra carryover also seems to happen with flashbombs, mines, and holy water having mixed inv_persistent set (perhaps other items too). It even shows the extra carryover in the equipment store before the mission.
TagsNo tags attached.

Activities

Mortem Desino

Mortem Desino

15.10.2013 07:17

reporter   ~0006201

Last edited: 15.10.2013 07:33

I'm thinking about this some more. The only case (that I can think of) in which we want to change inv_persistence is, e.g. the player gets caught and tossed in jail for the next mission, and he loses all his items and arrows. This bug might be superfluous, because go-to-jail lose-all-items is supposed to be handled via atdm:campaign_info (so it "filters out" all the items and weapons before the next mission.)

So we DO want all stacks of mixed persistence to always carry everything over?

EDIT: Eh. I can only think of one in which we need to support mixed persistence. "Our Hero makes his getaway with all his ill-gotten gains. But OOPS HE SLIPS AND FALLS and he drops ONLY the health potions that he found in Sir Galahad's manor, because Sir Galahad's health potions are all particularly shit for some reason."

Mortem Desino

Mortem Desino

20.10.2013 20:19

reporter   ~0006213

After further consideration -- not by itself buggy behavior, but seems to be so due to mapper error.

Issue History

Date Modified Username Field Change
15.10.2013 07:04 Mortem Desino New Issue
15.10.2013 07:10 Mortem Desino Additional Information Updated
15.10.2013 07:17 Mortem Desino Note Added: 0006201
15.10.2013 07:18 Mortem Desino Note Edited: 0006201
15.10.2013 07:33 Mortem Desino Note Edited: 0006201
20.10.2013 20:19 Mortem Desino Note Added: 0006213
20.10.2013 20:19 Mortem Desino Status new => closed
20.10.2013 20:19 Mortem Desino Assigned To => Mortem Desino
20.10.2013 20:19 Mortem Desino Resolution open => fixed