Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loot sorting: stop moving crafts in progress #79279

Merged
merged 3 commits into from
Jan 23, 2025

Conversation

o175
Copy link
Contributor

@o175 o175 commented Jan 21, 2025

Prevent sorting activity from moving items that somebody works on

Summary

Bugfixes "Prevent sorting activity from moving items that somebody works on"

Purpose of change

Items that somebody works on should not be moved because it interrupts crafting. Fixes #74170

Describe the solution

Collect all items that are being crafted and don't move them. Skip items with "activity_var" (for example, disassembly in progres)

Describe alternatives you've considered

Testing

Autotests and manual testing.

Additional context

Set the unsorted loot zone on the workbench, set "Loot: other" somewhere else. Ask NPC to sort items and start crafting something on the workbench. Same with disassembly

Prevent sorting activity from moving items that somebody works on

fixes CleverRaven#74170
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions labels Jan 21, 2025
o175 and others added 2 commits January 21, 2025 18:15
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions and removed astyled astyled PR, label is assigned by github actions labels Jan 21, 2025
@Maleclypse Maleclypse merged commit 56b4c66 into CleverRaven:master Jan 23, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NPCs steal in progress crafts when sorting
2 participants