Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #2723259 by yanniboi: Allow data collection input for multiple context fields #450

Open
wants to merge 1 commit into
base: 8.x-3.x
Choose a base branch
from

Conversation

yanniboi
Copy link
Contributor

This issue is mirrored on drupal.org by https://www.drupal.org/node/2723259.

Added exceptions allowing single value fields to be passed to multiple context fields.

…elds to be passed to multiple context fields.
@yanniboi
Copy link
Contributor Author

So far I have added an exception that allows submitting a single value field to a multiple context field.

Also when a single value is passed to an action or condition, expecting an array, it gets wrapped in an array.

@yanniboi
Copy link
Contributor Author

I've tested my changes by passing '@user.current_user_context:current_user.mail.value' to the SystemSendEmail 'to' field which is multiple cardinality.

Works for me, but needs testing, and I could do with someone confirming that I'm on the right track or letting me know if I am barking up the wrong tree...

@yanniboi
Copy link
Contributor Author

OK, so it looks like my exception for lists is breaking IntegrityCheckTest::testListTypeViolation() which makes sense.

I think what I've dont so far is enough of a proof of concept, but I'm going to leave it here for now, as I don't even know yet if I am going about this the right way. Feel free to comment.

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