Index email/scopes config by scope email

tr33m4n · Jan 7, 2023 · a4467b3 · a4467b3
1 parent d27ad56
commit a4467b3
Show file tree

Hide file tree

Showing 2 changed files with 53 additions and 7 deletions.
diff --git a/src/Block/Adminhtml/Form/Field/ImpersonatedEmails.php b/src/Block/Adminhtml/Form/Field/ImpersonatedEmails.php
@@ -7,6 +7,7 @@
 use Magento\Config\Block\System\Config\Form\Field\FieldArray\AbstractFieldArray;
 use Magento\Framework\DataObject;
 use Magento\Framework\View\Element\BlockInterface;
+use tr33m4n\OauthGmail\Model\Config\Provider;
 
 class ImpersonatedEmails extends AbstractFieldArray
 {
@@ -20,15 +21,15 @@ class ImpersonatedEmails extends AbstractFieldArray
     protected function _prepareToRender()
     {
         $this->addColumn(
-            'email',
+            Provider::EMAIL_KEY,
             [
                 'label' => __('Email'),
                 'class' => 'required-entry validate-email'
             ]
         );
 
         $this->addColumn(
-            'scopes',
+            Provider::SCOPES_KEY,
             [
                 'label' => __('Scopes'),
                 'renderer' => $this->getScopesRenderer()

diff --git a/src/Model/Config/Provider.php b/src/Model/Config/Provider.php
@@ -8,6 +8,8 @@
 use Magento\Framework\App\Filesystem\DirectoryList;
 use Magento\Framework\Filesystem;
 use Magento\Framework\Filesystem\Directory\ReadInterface;
+use Magento\Framework\Mail\Template\SenderResolverInterface;
+use Magento\Framework\Serialize\SerializerInterface;
 use tr33m4n\OauthGmail\Exception\ConfigException;
 
 class Provider
@@ -26,19 +28,36 @@ class Provider
 
     private const AUTH_PATH_TEMPLATE = 'oauth_gmail' . DIRECTORY_SEPARATOR . '%s';
 
+    public const EMAIL_KEY = 'email';
+
+    public const SCOPES_KEY = 'scopes';
+
+    private SerializerInterface $serializer;
+
     private ScopeConfigInterface $scopeConfig;
 
     private ReadInterface $varDirectory;
 
+    private SenderResolverInterface $senderResolver;
+
+    /**
+     * @var array<string, string>
+     */
+    private ?array $impersonatedEmails = null;
+
     /**
      * Provider constructor.
      */
     public function __construct(
+        SerializerInterface $serializer,
         ScopeConfigInterface $scopeConfig,
-        Filesystem $filesystem
+        Filesystem $filesystem,
+        SenderResolverInterface $senderResolver
     ) {
+        $this->serializer = $serializer;
         $this->scopeConfig = $scopeConfig;
         $this->varDirectory = $filesystem->getDirectoryRead(DirectoryList::VAR_DIR);
+        $this->senderResolver = $senderResolver;
     }
 
     /**
@@ -118,16 +137,42 @@ public function shouldUseImpersonated(): bool
      * Get impersonated emails
      *
      * @throws \tr33m4n\OauthGmail\Exception\ConfigException
+     * @throws \Magento\Framework\Exception\MailException
      * @return array<string, string>
      */
     public function getImpersonatedEmails(): array
     {
-        $impersonatedEmail = $this->scopeConfig->getValue(self::XML_CONFIG_IMPERSONATED_EMAILS);
-        if (!filter_var($impersonatedEmail, FILTER_VALIDATE_EMAIL)) {
-            throw new ConfigException(__('Impersonated email is not a valid email'));
+        if (null !== $this->impersonatedEmails) {
+            return $this->impersonatedEmails;
+        }
+
+        $impersonatedEmails = $this->scopeConfig->getValue(self::XML_CONFIG_IMPERSONATED_EMAILS);
+        if (is_string($impersonatedEmails)) {
+            $impersonatedEmails = $this->serializer->unserialize($impersonatedEmails);
+        }
+
+        if (empty($impersonatedEmails)) {
+            throw new ConfigException(__('No impersonated emails have been set'));
+        }
+
+        $indexedEmails = [];
+        foreach ($impersonatedEmails as $impersonatedEmail) {
+            $scopes = $impersonatedEmail[self::SCOPES_KEY] ?? [];
+            if (empty($scopes)) {
+                continue;
+            }
+
+            foreach ($scopes as $scope) {
+                $resolvedEmail = $this->senderResolver->resolve($scope)[self::EMAIL_KEY] ?? null;
+                if (!$resolvedEmail) {
+                    continue;
+                }
+
+                $indexedEmails[$resolvedEmail] = $impersonatedEmails[self::EMAIL_KEY];
+            }
         }
 
-        return $impersonatedEmail;
+        return $this->impersonatedEmails = $indexedEmails;
     }
 
     /**