Skip to content

Fix Quick Apply inventory application and FlowGraph errors#92

Open
Darojax wants to merge 1 commit into
zer0Kerbal:masterfrom
Darojax:contrib/fix-quickapply-flowgraph
Open

Fix Quick Apply inventory application and FlowGraph errors#92
Darojax wants to merge 1 commit into
zer0Kerbal:masterfrom
Darojax:contrib/fix-quickapply-flowgraph

Conversation

@Darojax
Copy link
Copy Markdown

@Darojax Darojax commented Apr 25, 2026

Addresses #81.

This fixes ScrapYard's automatic and Quick Apply inventory application path so recovered inventory parts are applied reliably instead of leaving new parts selected and producing FlowGraph errors.

Changes:

  • apply saved inventory state more reliably during Quick Apply
  • avoid mutating KSP persistent IDs while preserving ScrapYard tracker IDs
  • refresh editor costs after inventory changes
  • group inventory rows by displayed metadata
  • add optional reused-part cost percentage, defaulting to existing 0% behavior
  • guard optional local build/deploy hooks

Tested with KSP 1.12.5 in a modded test install:

  • VAB Quick Apply
  • manual part apply
  • rollout/recovery
  • inventory stacking
  • reused-part cost setting

Fix ScrapYard's automatic and Quick Apply inventory application path so recovered parts are applied reliably instead of leaving new parts selected and triggering FlowGraph errors.

Also refresh editor costs after inventory changes, preserve inventory tracker IDs without mutating KSP persistent IDs, group inventory rows by displayed metadata, and add a save setting for optional reused-part cost.

Related: zer0Kerbal#81
@Darojax Darojax changed the title Fix Quick Apply inventory application Fix Quick Apply inventory application and FlowGraph errors Apr 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant