code style: replaced traditional syntax array literals

pushed minimum required PHP version to 7.2

Author: onlime

app/ConfigLoader.php

@@ -21,10 +21,10 @@ class ConfigLoader
     public function __construct()
     {
         $globalConfig = APP_ROOT . '/config.ini';
-        $extraConfigs = array(
+        $extraConfigs = [
             APP_ROOT . '/config.local.ini',
             APP_ROOT . '/config.private.ini'
-        );
+        ];
 
         // load global config
         $config = parse_ini_file($globalConfig, true);
@@ -80,7 +80,7 @@ protected function _arrayToObject(array $arr)
             * Using __FUNCTION__ (Magic constant)
             * for recursive call
             */
-            return (object)array_map(array($this, __FUNCTION__), $arr);
+            return (object)array_map([$this, __FUNCTION__], $arr);
         } else {
             // Return object
             return $arr;

app/SendmailWrapper.php

@@ -60,15 +60,15 @@ public function run()
 
         // count total number of recipients
         $rcptCount   = 0;
-        $rcptHeaders = array('to', 'cc', 'bcc');
+        $rcptHeaders = ['to', 'cc', 'bcc'];
         foreach ($rcptHeaders as $rcptHeader) {
             if (isset($headerArr[$rcptHeader])) {
                 // parse recipient headers according to RFC2822 (http://www.faqs.org/rfcs/rfc2822.html)
                 $rcptCount += count(imap_rfc822_parse_adrlist($headerArr[$rcptHeader], $defaultHost));
             }
         }
 
-        $messageInfo   = array(
+        $messageInfo   = [
             'uid'     => `whoami`,
             'msgid'   => $msgId,
             'from'    => @$headerArr['from'],
@@ -79,7 +79,7 @@ public function run()
             'site'    => @$_SERVER["HTTP_HOST"],
             'client'  => @$_SERVER["REMOTE_ADDR"],
             'script'  => getenv('SCRIPT_FILENAME')
-        );
+        ];
 
         // throttling
         if ($throttleOn) {
@@ -92,15 +92,15 @@ public function run()
             // redirect STDIN data to throttle command
             // We're going to use the whole email message including some
             // extra headers.
-            $throttleMsg = $this->buildMessage(array(
+            $throttleMsg = $this->buildMessage([
                 'X-Meta-MsgID'  => $messageInfo['msgid'],
                 'X-Meta-Site'   => $messageInfo['site'],
                 'X-Meta-Client' => $messageInfo['client'],
                 'X-Meta-Script' => $messageInfo['script'],
-            ));
-            $descriptorSpec = array(
-                0 => array('pipe', 'r')
-            );
+            ]);
+            $descriptorSpec = [
+                0 => ['pipe', 'r']
+            ];
             $proc = proc_open($throttleCmd, $descriptorSpec, $pipes);
             if (is_resource($proc)) {
                 fwrite($pipes[0], $throttleMsg);

app/StdinMailParser.php

@@ -21,7 +21,7 @@ abstract class StdinMailParser
     /**
      * @var array
      */
-    protected $_additionalHeaders = array();
+    protected $_additionalHeaders = [];
 
     /**
      * @var string
@@ -34,7 +34,7 @@ abstract class StdinMailParser
      *
      * @var array
      */
-    protected $_rfc5322MultiHeaders = array(
+    protected $_rfc5322MultiHeaders = [
         'trace',
         'resent-date',
         'resent-from',
@@ -46,7 +46,7 @@ abstract class StdinMailParser
         'comments',
         'keywords',
         'optional-field'
-    );
+    ];
 
     /**
      * Constructor
@@ -106,7 +106,7 @@ public function setHeader($name, $value)
     public function getParsedHeaderArr()
     {
         $headerLines = explode(PHP_EOL, $this->_header);
-        $headerArr   = array();
+        $headerArr   = [];
         foreach ($headerLines as $line) {
             list($key, $value) = explode(":", $line);
             $key   = strtolower(trim($key));

composer.json

@@ -16,7 +16,7 @@
         "issues": "https://github.com/onlime/sendmail-wrapper/issues"
     },
     "require": {
-        "php": ">=5.4.0",
+        "php": ">=7.2.0",
         "ext-mbstring": "*",
         "ext-pdo": "*",
         "ext-imap": "*"

prepend.php

@@ -1,12 +1,12 @@
 <?php
 // environment variables that should be available in child processes
-$envVars = array(
+$envVars = [
     'HTTP_HOST',
     'SCRIPT_NAME',
     'SCRIPT_FILENAME',
     'DOCUMENT_ROOT',
     'REMOTE_ADDR'
-);
+];
 
 // sanitizing environment variables for Bash ShellShock mitigation
 // (CVE-2014-6271, CVE-2014-7169, CVE-2014-7186, CVE-2014-7187, CVE-2014-6277)