search

sadger / CorsixTH

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

Spawing of emergency patients with epidemic #10

Closed MarkL1961 closed 10 years ago

MarkL1961 commented 10 years ago

I noticed this in the console screen where four patients were spawned very close to each other with infectious laughter. The emergency is for infectious laughter and I suspect these are the same patients. I am not sure if this is intended, but it could get confusing if emergency patient were to get infected too - what do you think?

Queuing emergency fax Contagious the_squits patient spawned Creating new epidemic for the_squits Contagious infectious_laughter patient spawned Cannot create epidemic infectious_laughter. Epidemic already in progress Contagious infectious_laughter patient spawned Cannot create epidemic infectious_laughter. Epidemic already in progress Contagious infectious_laughter patient spawned Cannot create epidemic infectious_laughter. Epidemic already in progress Contagious infectious_laughter patient spawned Cannot create epidemic infectious_laughter. Epidemic already in progress Destroying Inspector

sadger commented 10 years ago

It's not directly intended no, should we restrict it so emergency patients can't be contagious or get infected? I was thinking about this earlier weirdly, makes sense I think anyway or you can't tell who is who with the icons.

sadger commented 10 years ago

I have actually implemented this and pushed it to the branch already, so if you update your repo you shouldn't get emergency patients involved in epidemics. I'll leave this issue open for a bit though just so we can confirm it doesn't still happen.

MarkL1961 commented 10 years ago

The icons was one of my concerns (green bug alternating with the blue light would be confusing) and you can't currently get an emergency when there is an epidemic, so it is a waste as well. Yes, perhaps make it so they are immune from epidemics for now, maybe later it can be added if we can get a new animations for it (and run more than one timer at a time)

MarkL1961 commented 10 years ago

that was quick, is it in both branches or the main one?

sadger commented 10 years ago

It was only a small change and I knew where it went :P. I am adding everything to experimental-epidemics for now in case the change needs amending and then i'll transfer anything "stable" to the main epidemics branch. So just stay on the experimental one for now.

sadger commented 10 years ago

This is fixed as far as I know.