KSP Recall

KSP Recall

345k Downloads

0.4.0.4 is screwing PAW

Lisias opened this issue · 4 comments

commented

Someone is screwing some PAW entries on Flight!

Fellow Kerbonaut Krazy1 reported:

Here's what I'm seeing in flight:
image

But this is what I see on my rig (KSP 1.12.5 with the latest KSP-Recall):

On Editor

Screen Shot 2024-03-31 at 14 43 33

On Flight

Screen Shot 2024-03-31 at 14 44 02

commented

Oukey, Regression Test Fest.

I downgraded KSP-Recall (on KSP 1.12.5 with a minimal install set -- WITHOUT KSPCF, just to made it clear to my future self) to see what I get.

  • Test Sesstions

Where "working fine" is the absence of the weird PAW entries reported by Krazy1 above.

commented

Oukey, my fault.

commented

When I wrote 0.4.0.3 (being specific, this commit f8afb62 ) I initially meant to tie the presence of the PAW's entries to the DebugMode on KSPe's Globals.

One would need to manually edit the Globals file in order to see that PAWs, and I found no problem on cluttering the PAW with these entries while Debugging something. It was a way to visually check what's installed or not, and if the thing is active or not. But I never bothered to check what would happen, as I had something else on the pipeline and the focus was on hidding the PAWs for the common user.

Then this happened, and I realized these PAW entries were being used by some users, and stupidly just tied the PAWs visibility to a new option, normally set to true, and then what I had intended to happen only on DebugMode started to happen to everybody and the kitchen's sink. This naïve commit caused the problem: 8389132 - as dumb as I could get, to tell you the true.

commented

Implemented on commit ef0f80a