Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.How to reproduce:
* Create a relationship with optional required context
* Leave that context empty
Relationship will never be called then.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | ctools-2399313-1-Relationship-optional-context.patch | 1.28 KB | geek-merlin |
Comments
Comment #1
geek-merlinPatch flying in that fixes this for me.
Comment #3
geek-merlinThis makes #1588212: Add "optional" checkbox to require contexts within mini panels break existing sites so setting major and improving title.
Comment #4
sylus CreditAttribution: sylus commentedWorking well for me too too bad didn't get this to RTBC before release.
Comment #5
geek-merlinIt looks like there is another issue with similar but not same code: #2341363: Relationship with optional contexts does not work
Didn't check if it's a dup.
Comment #6
rivimeyPatch in #1 applies cleanly to 7.x-1.x
Patch passes ctools tests run locally. If @axel.rutz is watching still it would be good to see a mod to the patch that includes a bit more comment in the code; it is not obvious (to me at least) what is happening.
It would appear to me that this patch has at least two other related issues, and one of them (https://www.drupal.org/node/1588212) has in #31 (https://www.drupal.org/node/1588212#comment-10538470) been determined to be closed.
Could someone please clarify whether the problem here is fixed or not?
In the meantime, this looks like a duplicate.
Comment #7
DamienMcKennaIt wasn't committed to 1.11.
Comment #8
MustangGB CreditAttribution: MustangGB commentedComment #9
joelpittet@geek-merlin Thanks for the patch, I've committed it to the latest dev. Minor change on commit I merged the elseif at the end.