Allow choosing an Options email as recipient

Thanks for adding this option! Generally looks fine but just a couple things and one question

1. +++ b/src/Entity/ContactEmail.php
   @@ -506,13 +506,23 @@ class ContactEmail extends ContentEntityBase implements ContactEmailInterface {
   +      $allowedValues = $definition->getSetting('allowed_values');
   

   Coding standards should be $allowed_values = ... as its not a property of the class

2. +++ b/src/Entity/ContactEmail.php
   @@ -506,13 +506,23 @@ class ContactEmail extends ContentEntityBase implements ContactEmailInterface {
   +              $results += explode(',', $allowedValues[$email['value']]['emails']);
   

   Is it possible that this is empty and causes a notice?

3. Needs test coverage added along with the new option, perhaps could go here in the ContactEmailsRecipientTest

I need this as I use Options email. I could not get the patch in #3 to apply to either 8.x-1.x-dev or 8.x-1.20 (no surprise - it's two years old!)

I got the following error on 8.x-1.20

error: patch failed: src/ContactEmailsTestBase.php:2
error: src/ContactEmailsTestBase.php: patch does not apply
error: patch failed: src/Tests/ContactEmailsRecipientTest.php:84
error: src/Tests/ContactEmailsRecipientTest.php: patch does not apply

So I removed the ContactEmailsTestBase and ContactEmailsRecipientTest elements from the patch and it then applied. I've tested it and it all works fine. I'd have been happy to move this to RTBC if I hadn't hacked the patch.

I've attached the 'hacked' patch file, but you really need to re-instate the test stuff I guess?

Could whoever maintains this module move this forward please?

Has this been applied?

Apologies for the huge delay here. The clients I work for have all switched to Webform, so I will mark this module as seeking co-maintainers to help speed things up in the future.

Thank you!