Disabling an NPC, instead of deleting

[Gangrenous]

I know when doing content, many are apt to just remove a spawn. If it is a duplicate, yeah that is fine. But for these spawns that are possibly a special event, or Holiday they really need to be disabled instead. Take for instance Freeport currently. In Freeport there are events, Halloween and Christmas spawns everywhere. In my opinion, to preserve the location, they just need to be disabled instead. This way they reduce load times. I am thinking if you just set the chance to zero, worlddatabase.cpp would probably still load them. Can we get a boolean column in the database that we can query against so those spawns are not loaded. Then during the holidays we could just enable them again.

Thoughts? And I do not mind doing it and just posting a diff here.

[Jabantiz]

We have just been changing spawn chance to 0 and making a list of id's and what event they go to so they could be toggled back on if some one wants them.

Yes world will load them at start up but that takes minimal time, they will not actually spawn in the zone and therefore no checks will be run against them and they won't effect a client loading into the zone at all.

[Gangrenous]

Yeah, it is not that many. I will just go that route, so I remain consistent with you.