This is a really minor thing but there needs to be a check for existence as well as a check for blankness for the keys in /citcon/modules/citcon_email/plugins/target/cce_select_field_lookup/cce_select_field_lookup.inc

See attached patch

I'm sorry I don't have the exact error it causes. I made the mistake of waiting to file this patch and now I've lost it.

CommentFileSizeAuthor
#1 citcon_email--keys_not_present-2173485-1.patch1.04 KBmgifford
citcon_email--keys_not_present.patch1.33 KBraychaser

Comments

mgifford’s picture

mgifford
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new citcon_email--keys_not_present-2173485-1.patch1.04 KB

I've cleaned up the formatting a bit. But ya, seems like an easy addition.

raychaser’s picture

raychaser commented

Great. Thanks for doing that.