I suppose it depends if you have an edited version of globals.gd that shifts the line numbers. I'm actually the one that reported the bug and the fix I was using to solve it. After line 1278 isn't an absolute necessity if you prefer it elsewhere. I just choose to put it last thing before "gameloaded = true". However, if you were thinking it is counter intuitive because you are thinking the directions mean to include it "inside" the previous else, well, that's not the intent.
The farmmana dictionary entry didn't exist on old saves and so I added it when loading. I never did check to see if a game started from scratch in 0.5.25 would create the entry or not (either way it would be fine after a save then load, but if new games also fail, then an additional fix would be needed [and may exist in the bug fix]). At some point the game tries to test the value of farmmana and errors if it doesn't already exist.