On our project that's built on Lightning, we had to use the patch in #2653574-69: Unable to keep nested IEF data separate with multivalue fields. in order to fix the bug where nested IEF forms broke IEF's behavior.

I'm fairly certain that patch (or something architecturally similar) will make it into the next release of IEF... hopefully. If it does, it looks like it will break Lightning's schedulePublication() Behat step definition inside lightning.behat.inc.

I have a patch that fixes the to-be-broken test, and will attach it in this issue. For our project, in order to not require a patched Lightning instance, we're actually using a copied-but-modified custom Behat step definition until things are fixed upstream (better than relying on multiple patches to test core functionality).

CommentFileSizeAuthor
#2 2846867-2-ief-scheduled-update-fix.patch1.17 KBgeerlingguy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geerlingguy created an issue. See original summary.

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy at Midwestern Mac, LLC commented
FileSize
2846867-2-ief-scheduled-update-fix.patch1.17 KB
geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy at Midwestern Mac, LLC commented

Leaving postponed since #2653574: Unable to keep nested IEF data separate with multivalue fields. is not merged upstream.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Postponed » Reviewed & tested by the community

The upstream patch is RTBC, so we could bring it into Lightning now. I'm RTBCing this on the assumption that it will pass our functional tests on Travis CI (which we'll verify before we merge it).

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Reviewed & tested by the community » Closed (outdated)

Lightning no longer uses Scheduled Updates, and it will be removed as a dependency in #2927527: Remove unused/deprecated modules from codebase. So I'm closing this as outdated.