1.0.2 Update Causing Error about acf-flexible-content-preview

[benvoynick]

I've committed 946bc90 to resolve this and related issues.

As @paulmist noted in #4 the nesting level for the subfield loop was not right, and so a lot of subfields were potentially getting missed.

The warning was caused because acf_get_fields() requires a WordPress post ID (which anything below the Field Group level does not have). Thankfully it is not necessary to call that function on the individual layouts: while acf_get_field_groups() does not populate field data, acf_get_fields() does return recursive field group data.

I've refactored the code to split the recursive portion out into another method that calls itself, which I think makes things a little easier to read. It also means that even deeper nesting levels are supported. While going deeper probably wouldn't be a good user experience, the plugin will allow it for anyone who wants. :)

[jameelmoses]

Thanks, @benvoynick. @paulmist this is now resolved and released. Thanks for reporting!

[paulmist]

@benvoynick brilliant! Thank you for tidying up my mess.

Perhaps you can have a stab at Covid19 next?

I was looking for some good documentation on the ACF functions but struggled.

[jameelmoses]

@paulmist while you’re here, check out https://github.com/jameelmoses/acf-flexible-content-extended. It provides the same functionality for image previews of layouts with the added improved UX of editing layouts in a modal so your backend pages aren’t so long