search

sadger / CorsixTH

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

How we could determine which diseases can be contagious #27

Closed MarkL1961 closed 9 years ago

MarkL1961 commented 10 years ago

I had an idea this morning that if we were to add disease.contagious = true to all the disease files where we want to allow them in an epidemic and disease.contagious = false with a note as to why it is false at the side; you would no longer need to worry about visuals.id. This way we can include discrete itching and sleeping illness and exclude pregnant, bloaty head and all the diseases with their own animations without having to list all the diseases in a function.

sadger commented 9 years ago

This would work I guess and then we could modify the contagiousness if we decide to try and support visual diseases. Adding this option would only mean a small change to the code so if you want we can go for it?

MarkL1961 commented 9 years ago

I say go for it. :+1: I'll try and find time this weekend to test this and the others and let you know the outcome

sadger commented 9 years ago

Ok great. Apart from the non-visual diseases which is the other that can be contagious? Is it just discrete itching and sleeping illness?

MarkL1961 commented 9 years ago

Radiation and jellyitus. Technically a visual, but at the moment they are not, but were contagious in TH.

MarkL1961 commented 9 years ago

Ones to avoid are pregnant as we don't really want men getting pregnant. As the girls might expect this in real life :) Then any with their own animation, although it might not be a problem animation wise, but I say leave them as a future option. Invis, bloaty, Elvis, hairyitus, transparency, broken bones, slack tongue and baldness all have their own animation.

Edit: forgot to mention alien dna, although it would be nice to see that spread to other patients

sadger commented 9 years ago

Ok I'll get this done probably over the weekend. By the way I have moved development back to the "epidemics" branch from the "experimental" one and copied all the changes over because I'm preparing it for getting merged for 0.4. So please work from that branch from now on.

sadger commented 9 years ago

This is done now and in the "epidemics" branch. I have tested it a bit and for example radiation patients can now spread their disease.

MarkL1961 commented 9 years ago

Is this fixed then? Seems to be okay to me

sadger commented 9 years ago

Yeah I think it's working. Closed until proven otherwise.