ENGCOM-5506: Cleaning some code gaps #23782

Author: sidolov

app/code/Magento/PageCache/Model/Config.php

@@ -214,18 +214,18 @@ protected function _getReplacements()
      */
     protected function _getAccessList()
     {
-        $result = '';
-        $tpl = "    \"%s\";";
+        $tpl = '    "%s";';
         $accessList = $this->_scopeConfig->getValue(self::XML_VARNISH_PAGECACHE_ACCESS_LIST);
         if (!empty($accessList)) {
-            $result = [];
+            $ipsList = [];
             $ips = explode(',', $accessList);
             foreach ($ips as $ip) {
-                $result[] = sprintf($tpl, trim($ip));
+                $ipsList[] = sprintf($tpl, trim($ip));
             }
-            return implode("\n", $result);
+            return implode("\n", $ipsList);
         }
-        return $result;
+
+        return '';
     }
 
     /**

app/code/Magento/Sales/Model/Order/Email/SenderBuilder.php

@@ -85,11 +85,9 @@ public function sendCopyTo()
         $copyTo = $this->identityContainer->getEmailCopyTo();
 
         if (!empty($copyTo) && $this->identityContainer->getCopyMethod() == 'copy') {
+            $this->configureEmailTemplate();
             foreach ($copyTo as $email) {
-                $this->configureEmailTemplate();
-
                 $this->transportBuilder->addTo($email);
-
                 $transport = $this->transportBuilder->getTransport();
                 $transport->sendMessage();
             }