Egregoros

Signal feed

Timeline

Post

Remote status

Context

2

Replies

7
@sun @coolboymew If I'm already fixing plain mistakes, wouldn't it make more sense to fix the bug itself, though? It's essentially just a typo in the script.

But... at that point, it's not really a translation patch, is it? "Translation patch and i fixed this one bug." also runs into the issue that this has a really neat interaction where you can stack max MP buffs without raising HP to take advantage of the spells that cost x% of your max HP as fuel.
@All_bonesJones @coolboymew there's no objectively correct answer but for some bugs yeah you could argue it's been around so long that if you changed it now it would break people's expectations, but that is kind of a special case but you're aware of it here so it's tricky yeah.

i think for translation the current relevant fear is just some stupid asshole deliberately and grossly changing the meaning of things and it can lead to maybe overcaution in the opposite direction by well meaning translators
@coolboymew @sun I mean, the code is pretty obviously the fuckup. It gets the player's max MP, divides it by 6, adds 50, and then adds that to HP, then clears the variable. the description itself clearly says restores MP at the start of battle. every other similar mechanic that does this affects its own stat type (TP, MP, HP, etc) only does its own type with no cross-pollination. also, the MP restore is in between two things that do HP restore, and it looks like he copy/pasted the code and changed it to MP except the last step.