Skip to content

Commit

Permalink
Now that KSP-Recall is in charge of preventing KSP's Editor from scre…
Browse files Browse the repository at this point in the history
…wing with us, a lot of code is now useless. So instead of fixing things, commenting out code is the most sensible thing to be done!

"FIXES" #261
  • Loading branch information
Lisias committed Sep 4, 2022
1 parent f47899c commit 0347574
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
10 changes: 9 additions & 1 deletion Source/PartDB14x/PartDB/VariantPartScaler.cs
Expand Up @@ -75,10 +75,17 @@ internal void OnEditorVariantApplied(Part part, PartVariant partVariant)

// And since the super's Move code doesn't works right when you are "scaling back" a part, we need to
// "fix" them now.
this.MoveParts();
// this.MoveParts();

this.OnChange();
}
#if false
// This whole section of code is now useless once KSP-Recall is in charge of preventing Editor from screwing up with us.
//
// As a matter of fact, this whole code was **never** really necessary - in the end, I was just playing
// a tug-of-war with Editor, redoing things unnecessarily
//
// I expect to have "fixed" a lot of problems by removing this krapload of code!!!

protected override AttachNode[] FindBaseNodesWithSameId(AttachNode node)
{
Expand Down Expand Up @@ -175,5 +182,6 @@ private AttachNode[] FindAttachingNode(Part part, Part attachedPart)

return attachingNodes;
}
#endif
}
}
2 changes: 1 addition & 1 deletion Source/PartDB14x/PartDB/VariantPartScaler14x.cs
Expand Up @@ -28,7 +28,7 @@ internal partial class VariantPartScaler : StandardPartScaler
protected override void DoCopyUpdate()
{
base.DoCopyUpdate();
this.MoveParts();
//this.MoveParts();
}
}
}

0 comments on commit 0347574

Please sign in to comment.