Skip to content

Fix ACF group mapping validation in push sync#55

Open
manuel-be-one wants to merge 1 commit into
Bynder:mainfrom
manuel-be-one:fix/sync-push-acf-group-mapping
Open

Fix ACF group mapping validation in push sync#55
manuel-be-one wants to merge 1 commit into
Bynder:mainfrom
manuel-be-one:fix/sync-push-acf-group-mapping

Conversation

@manuel-be-one
Copy link
Copy Markdown

Summary

This PR hardens the push sync when building ACF group mappings.

What changed

  • Skip non-array group entries before mapping.
  • Guard against key/value count mismatches before calling array_combine().
  • Log mapping mismatches to simplify debugging malformed data.

Why

  • array_combine() can generate invalid mappings or warnings when the ACF group shape is not what we expect.
  • This fix prevents sync failures caused by malformed or inconsistent field data.

Testing

  • Verified the push sync path with valid group data.
  • Verified malformed group entries are skipped safely.
  • Verified mismatched key/value counts no longer break the sync.

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