BetterPawnControl icon indicating copy to clipboard operation
BetterPawnControl copied to clipboard

AssignManager Paste Action Bug?

Open JiaRG opened this issue 1 year ago • 3 comments

https://github.com/voult2/BetterPawnControl/blob/71aad7a3b5c0da7e458652f4f373ea85f4c380ea/Source/Managers/AssignManager.cs#L483-L496

L488 should AssignManager.links.RemoveAll(x => x.zone == policy.id); ?

JiaRG avatar Jun 25 '24 02:06 JiaRG

https://github.com/voult2/BetterPawnControl/blob/71aad7a3b5c0da7e458652f4f373ea85f4c380ea/Source/Base/WorkLink.cs#L63-L70 settings may be null

JiaRG avatar Jun 25 '24 06:06 JiaRG

On the first one, YES. It should be assignManager... what the hell On the second one, IDK. how can it be null if all pawns have default settings when spawned?

voult2 avatar Jun 25 '24 18:06 voult2

After integrating with the Achtung mod, when I save and load the archive, the log will show a lot of the following two lines of information: Could not load reference to Verse.WorkTypeDef named Rescuing Null key while loading dictionary of Verse.WorkTypeDef and System.Int32. label=settings

JiaRG avatar Jun 26 '24 03:06 JiaRG

Fixed in 2.7.6

voult2 avatar Feb 01 '25 20:02 voult2