sadger / CorsixTH

Open source clone of Theme Hospital
Other
1 stars 0 forks source link

Possible to have an epidemic with zero infected #11

Closed MarkL1961 closed 10 years ago

MarkL1961 commented 10 years ago

Not sure what really happened here, I noticed that an epidemic could not be created as one was already in progress - yet I could not see where. Then it said they had got fed up and left and then it was revealed.

2

sadger commented 10 years ago

Ha! That's amazing, wonder why it does that. I think when I changed the code to support a single epidemic it doesn't remove an epidemic with 0 patients, it's probably the reason as i'm sure it didn't happen before :P.

sadger commented 10 years ago

Apparently there is an attempt to remove an epidemic with 0 patient but only when it's not ready to reveal, i'll try relaxing that condition and see if it removes this issue. The epidemic must have been ready to reveal (he was full diagnosed) but then no cure room was found so he left. We don't want to remove the epidemic while a cover up is in progress or of course after the fax has been sent but before that it should be fine. I'll tweak the conditions this weekend to fix this.

sadger commented 10 years ago

Pushed a commit that should fix this - let me know if it happens again and i'll have to reinvestigate.

MarkL1961 commented 10 years ago

checked this against the autosave and readout is now removing patient from epidemic - got fed up and left then Epidemic has no infected patients, removing I would say that closes this one then :)

sadger commented 10 years ago

Awesome, thanks Mark, we are really getting to grips with this stuff now!