XOOPS / XoopsCore

xoops_lib/Xoops/Core/ComposerUtility.php

Spacing +6 added lines, -6 removed lines

@@ -54,13 +54,13 @@ 
         $this->output = array();
         $this->errors = array();
 
-        $options = ' ' . trim($this->exeOptions) . ' ';
+        $options = ' '.trim($this->exeOptions).' ';
 
         if (empty($this->exe)) {
             $exeFinder = new PhpExecutableFinder();
             $foundExe = $exeFinder->find();
             if ($foundExe) {
-                $this->exe = $foundExe . ' composer.phar';
+                $this->exe = $foundExe.' composer.phar';
             } else {
                 $this->errors[] = 'Cannot find PHP executable';
                 return false;
@@ -73,14 +73,14 @@ 
         }
 
         set_time_limit(300); // don't want this script to timeout;
-        $command = $this->exe . $options . $command_line;
-        putenv('COMPOSER_HOME=' . \XoopsBaseConfig::get('var-path').'/composer');
+        $command = $this->exe.$options.$command_line;
+        putenv('COMPOSER_HOME='.\XoopsBaseConfig::get('var-path').'/composer');
         $process = new Process($command);
         //$process->setEnv(array('COMPOSER_HOME' => \XoopsBaseConfig::get('var-path').'/composer'));
         $process->setTimeout(120);
         try {
             $process->run(
-                function ($type, $buffer) use (&$errors, &$output) {
+                function($type, $buffer) use (&$errors, &$output) {
                     if (Process::ERR === $type) {
                         $errors[] = $buffer;
                     } else {
@@ -96,7 +96,7 @@ 
             array_unshift($this->output, $process->getErrorOutput());
             return true;
         } else {
-            $this->errors[] = 'Failed: ' . $command;
+            $this->errors[] = 'Failed: '.$command;
             $this->errors[] = sprintf(
                 "Process exit code: %s, '%s'",
                 $process->getExitCode(),

xoops_lib/Xoops/Core/Service/Manager.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
      * Service Mode constant - Multiple mode where all located services will be
      * dispatched in priority order.
      */
-    const MODE_MULTIPLE  = 8;
+    const MODE_MULTIPLE = 8;
 
     /**
      * Provider priorities
@@ -252,12 +252,12 @@ 
         } else {
             $xoops = \Xoops::getInstance();
             $provider = new Provider($this, $service);
-            $event = 'core.service.locate.' . $service;
+            $event = 'core.service.locate.'.$service;
             // locate service provider(s)
             // In response to trigger message, the contract implementor should register()
             $xoops->events()->triggerEvent($event, $provider);
             // get reference to the list of providers and prioritize it.
-            $registered=$provider->getRegistered();
+            $registered = $provider->getRegistered();
             if (count($registered)) {
                 $choices = isset($this->providerPrefs[$service]) ? $this->providerPrefs[$service] : array();
                 foreach ($registered as $p) {

xoops_lib/Xoops/Core/Service/Response.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     {
         $this->value = $value;
         $this->success = $success;
-        if ($errorMessage!==null) {
+        if ($errorMessage !== null) {
             $this->addErrorMessage($errorMessage);
         }
     }

xoops_lib/Xoops/Core/Service/Data/EmailAttachment.php

Spacing +3 added lines, -3 removed lines

@@ -70,11 +70,11 @@ 
      */
     public function __construct(?string $filename = null, ?string $mimeType = null)
     {
-        if (null!==$filename) {
+        if (null !== $filename) {
             Assert::fileExists($filename, static::MESSAGE_FILE);
             $this->filename = $filename;
         }
-        if (null!==$mimeType) {
+        if (null !== $mimeType) {
             Assert::regex($mimeType, static::MIME_REGEX, static::MESSAGE_MIME);
             $this->mimeType = $mimeType;
         }
@@ -245,6 +245,6 @@ 
      */
     public function getInlineAttribute() : bool
     {
-        return (bool) $this->inline;
+        return (bool)$this->inline;
     }
 }

xoops_lib/Xoops/Core/Service/Data/EmailAttachmentSet.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      */
     public function __construct(?array $attachments = null)
     {
-        if (null!==$attachments) {
+        if (null !== $attachments) {
             Assert::allIsInstanceOf($attachments, EmailAttachment::class, static::MESSAGE_ATTACHMENT);
             try {
                 /** @var EmailAttachment $attachment */

xoops_lib/Xoops/Core/Service/Data/Message.php

Spacing +4 added lines, -4 removed lines

@@ -66,21 +66,21 @@
      */
     public function __construct(?string $subject = null, ?string $body = null, ?int $fromId = null, ?int $toId = null)
     {
-        if (null!==$subject) {
+        if (null !== $subject) {
             $subject = trim($subject);
             Assert::stringNotEmpty($subject, static::MESSAGE_SUBJECT);
             $this->subject = $subject;
         }
-        if (null!==$body) {
+        if (null !== $body) {
             $body = trim($body);
             Assert::stringNotEmpty($body, static::MESSAGE_BODY);
             $this->body = $body;
         }
-        if (null!==$fromId) {
+        if (null !== $fromId) {
             Assert::greaterThan($fromId, 0, static::MESSAGE_FROM);
             $this->fromId = $fromId;
         }
-        if (null!==$toId) {
+        if (null !== $toId) {
             Assert::greaterThan($toId, 0, static::MESSAGE_TO);
             $this->toId = $toId;
         }

xoops_lib/Xoops/Core/Service/Data/EmailAddressList.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      */
     public function __construct(?array $addresses = null)
     {
-        if (null!==$addresses) {
+        if (null !== $addresses) {
             Assert::allIsInstanceOf($addresses, EmailAddress::class, static::MESSAGE_ADDRESS);
             try {
                 /** @var EmailAddress $address */

xoops_lib/Xoops/Core/Service/Data/EmailAddress.php

Spacing +4 added lines, -4 removed lines

@@ -58,15 +58,15 @@ 
      */
     public function __construct(?string $email = null, ?string $displayName = null)
     {
-        if (null!==$email) {
+        if (null !== $email) {
             $email = trim($email);
             Assert::true(
-                false!==filter_var($email, FILTER_VALIDATE_EMAIL, FILTER_FLAG_EMAIL_UNICODE),
+                false !== filter_var($email, FILTER_VALIDATE_EMAIL, FILTER_FLAG_EMAIL_UNICODE),
                 static::MESSAGE_ADDRESS
             );
             $this->email = $email;
         }
-        if (null!==$displayName) {
+        if (null !== $displayName) {
             $displayName = trim($displayName);
             $displayName = empty($displayName) ? null : $displayName;
             Assert::nullOrStringNotEmpty($displayName, static::MESSAGE_NAME);
@@ -85,7 +85,7 @@ 
     {
         try {
             Assert::true(
-                false!==filter_var($this->email, FILTER_VALIDATE_EMAIL, FILTER_FLAG_EMAIL_UNICODE),
+                false !== filter_var($this->email, FILTER_VALIDATE_EMAIL, FILTER_FLAG_EMAIL_UNICODE),
                 static::MESSAGE_ADDRESS
             );
         } catch (\InvalidArgumentException $e) {

xoops_lib/Xoops/Core/Service/Data/Email.php

Spacing +4 added lines, -4 removed lines

@@ -115,22 +115,22 @@
         ?EmailAddress $fromAddress = null,
         ?EmailAddress $toAddress = null
     ) {
-        if (null!==$subject) {
+        if (null !== $subject) {
             $subject = trim($subject);
             Assert::stringNotEmpty($subject, static::MESSAGE_SUBJECT);
             $this->subject = $subject;
         }
-        if (null!==$body) {
+        if (null !== $body) {
             $body = trim($body);
             Assert::stringNotEmpty($body, static::MESSAGE_BODY);
             $this->body = $body;
         }
         try {
-            if (null!==$fromAddress) {
+            if (null !== $fromAddress) {
                 $fromAddress->getEmail();
                 $this->fromAddress = $fromAddress;
             }
-            if (null!==$toAddress) {
+            if (null !== $toAddress) {
                 $toAddress->getEmail();
                 $this->toAddresses = new EmailAddressList([$toAddress]);
             }