fix redundant string slicing and explicit checks for protosynthesis and quarkdrive

[Jerick120]

PR: smogon/pokemon-showdown#10300 needs to be merged along with this

[Zarel]

You say "fix" but this just changes the protocol.

I have two objections here:

• When we change the protocol, the client needs to keep support for the old protocol. If we didn't, old replays would all be broken.
• I don't see any reason to change the protocol. The old one's string slicing may be ugly but it works. Like, I understand, I like your syntax better, too, but I don't think it's so much better to be worth having a bunch of replays using one syntax and a bunch of replays using another.

[Jerick120]

You say "fix" but this just changes the protocol.

I have two objections here:

• When we change the protocol, the client needs to keep support for the old protocol. If we didn't, old replays would all be broken.
• I don't see any reason to change the protocol. The old one's string slicing may be ugly but it works. Like, I understand, I like your syntax better, too, but I don't think it's so much better to be worth having a bunch of replays using one syntax and a bunch of replays using another.

Thats fair, however the logs are supposed to have a | delimiter for each value and every other stat changing ability does except these two and thats undocumented. The logs return protosynthesisdef while they should return protosynthesis|def.

[Zarel]

Hm. Yeah, I think that's a good point. Capitalization would also need to be fixed, and we'd need to update the volatile status display on the HP bar, but I think I could see that as being worth it.