Skip to content

Guard Assets Manager option saves#90

Merged
mehul0810 merged 1 commit into
release/1.6.0from
fix/release-1.6.0-assets-manager-notices
May 21, 2026
Merged

Guard Assets Manager option saves#90
mehul0810 merged 1 commit into
release/1.6.0from
fix/release-1.6.0-assets-manager-notices

Conversation

@mehul0810

Copy link
Copy Markdown
Collaborator

Summary

  • Guards Assets Manager save payloads and stored option reads before nested access.
  • Preserves the existing perform_assets_manager_options storage shape and option names.
  • Fixes archive exception persistence to store the selected archive value instead of reading an unrelated variable.

Validation

  • php -l src/Modules/Assets/AssetsManager.php
  • composer check-cs -- src/Modules/Assets/AssetsManager.php
  • git diff --check
  • Local Studio save-flow smoke on ?perform=1 persisted {"disabled":{"js":{"hoverintent-js":{"current":[0]}}}}, then test options were removed.

Closes #26.

@github-actions

Copy link
Copy Markdown

Test on Playground
Test this pull request on the Playground

@mehul0810 mehul0810 merged commit cc55e27 into release/1.6.0 May 21, 2026
1 of 7 checks passed
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