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

Spouse rooms may not be updated correctly #896

Open
atravita-mods opened this issue Jan 15, 2023 · 2 comments
Open

Spouse rooms may not be updated correctly #896

atravita-mods opened this issue Jan 15, 2023 · 2 comments
Labels
bug This is an unintended error or behavior that can be addressed with specific development changes.
Milestone

Comments

@atravita-mods
Copy link
Contributor

atravita-mods commented Jan 15, 2023

Describe the bug
When using Content Patcher to change spouse rooms at a rate of DayUpdated or higher, no changes to the spouseroom seems to take effect. This seems to be the case with both vanilla and mod-added NPCs.

To Reproduce

  1. Install the attach mods.
  2. Marry Lyell.
  3. Check the lower table in his spouse room. There should be a number and a picture.
  4. Sleep a night.
  5. Check the lower table in his spouse room again. The number will not have changed (when it should have)
  6. Alternatively, the same test steps with Abigail, except her spot is one tile to the right of her chair.

Log file
Upload your SMAPI log to https://smapi.io/log and post a link here.

This is using the map export function in my PR.

Mods.zip
SpouseRoomAssetPropagationTest.zip
lyell
image

@Pathoschild Pathoschild added the bug This is an unintended error or behavior that can be addressed with specific development changes. label Jan 15, 2023
@Pathoschild Pathoschild added this to the 4.0.x milestone Jan 15, 2023
@Pathoschild
Copy link
Owner

(Normally SMAPI handles that with asset propagation, but it doesn't currently handle the case where you change a map which is referenced from Data/SpouseRooms and was indirectly applied to the farmhouse.)

@atravita-mods
Copy link
Contributor Author

Similar issue for the texture files Data/HairData refers to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is an unintended error or behavior that can be addressed with specific development changes.
Projects
None yet
Development

No branches or pull requests

2 participants