XOOPS / XoopsCore

xoops_lib/Xoops/Core/Assets.php

Spacing +15 added lines, -15 removed lines

@@ -131,15 +131,15 @@ 
             $assetsPrefs = $xoops->cache()->read($this->assetsPrefsCacheKey);
             $file = $xoops->path($this->assetsPrefsFilename);
             $mtime = filemtime($file);
-            if ($assetsPrefs===false || !isset($assetsPrefs['mtime']) || !$mtime
-                || (isset($assetsPrefs['mtime']) && $assetsPrefs['mtime']<$mtime)) {
+            if ($assetsPrefs === false || !isset($assetsPrefs['mtime']) || !$mtime
+                || (isset($assetsPrefs['mtime']) && $assetsPrefs['mtime'] < $mtime)) {
                 if ($mtime) {
                     $assetsPrefs = Yaml::read($file);
                     if (!is_array($assetsPrefs)) {
                         $xoops->logger()->error("Invalid config in system_assets_prefs.yml");
                         $assetsPrefs = array();
                     } else {
-                        $assetsPrefs['mtime']=$mtime;
+                        $assetsPrefs['mtime'] = $mtime;
                         $xoops->cache()->write($this->assetsPrefsCacheKey, $assetsPrefs);
                         $this->copyBaseFileAssets();
                     }
@@ -212,7 +212,7 @@ 
             $assets = array($assets); // just a single path name
         }
 
-        if ($filters==='default') {
+        if ($filters === 'default') {
             if (isset($this->default_filters[$type])) {
                 $filters = $this->default_filters[$type];
             } else {
@@ -311,7 +311,7 @@ 
                 $filters
             );
             $asset_path = $asset->getTargetPath();
-            if (!is_readable($target_path . $asset_path)) {
+            if (!is_readable($target_path.$asset_path)) {
                 $assetKey = 'Asset '.$asset_path;
                 $xoops->events()->triggerEvent('debug.timer.start', $assetKey);
                 $oldumask = umask(0002);
@@ -320,7 +320,7 @@ 
                 $xoops->events()->triggerEvent('debug.timer.stop', $assetKey);
             }
 
-            return $xoops->url('assets/' . $asset_path);
+            return $xoops->url('assets/'.$asset_path);
 
         } catch (\Exception $e) {
             $xoops->events()->triggerEvent('core.exception', $e);
@@ -360,7 +360,7 @@ 
 
         try {
             if (is_scalar($assets)) {
-                $assets = array($assets);  // just a single path name
+                $assets = array($assets); // just a single path name
             }
             foreach ($assets as $a) {
                 // translate path if not a reference or absolute path
@@ -368,10 +368,10 @@ 
                     && (substr_compare($a, '/', 0, 1) != 0)) {
                     $a = $xoops->path($a);
                 }
-                if (false===strpos($a, '*')) {
+                if (false === strpos($a, '*')) {
                     $assetArray[] = new FileAsset($a); // single file
                 } else {
-                    $assetArray[] = new GlobAsset($a);  // wild card match
+                    $assetArray[] = new GlobAsset($a); // wild card match
                 }
             }
 
@@ -452,9 +452,9 @@ 
     {
         $xoops = \Xoops::getInstance();
 
-        $fromPath = $xoops->path($fromPath) . '/';
-        $toPath = $xoops->path('assets') . '/' . $output . '/';
-        $from = glob($fromPath . '/' . $pattern);
+        $fromPath = $xoops->path($fromPath).'/';
+        $toPath = $xoops->path('assets').'/'.$output.'/';
+        $from = glob($fromPath.'/'.$pattern);
 
         if (!is_dir($toPath)) {
             $oldUmask = umask(0);
@@ -468,8 +468,8 @@ 
             foreach ($from as $filepath) {
                 $xoops->events()->triggerEvent('debug.timer.start', $filepath);
                 $filename = basename($filepath);
-                $status=copy($filepath, $toPath.$filename);
-                if (false===$status) {
+                $status = copy($filepath, $toPath.$filename);
+                if (false === $status) {
                     $xoops->logger()->warning('Failed to copy asset '.$filename);
                 } else {
                     //$xoops->logger()->debug('Copied asset '.$filename);
@@ -480,7 +480,7 @@ 
             umask($oldUmask);
             return $count;
         } else {
-            $xoops->logger()->warning('Asset directory is not writable. ' . $output);
+            $xoops->logger()->warning('Asset directory is not writable. '.$output);
             return false;
         }
     }

xoops_lib/Xoops/Core/Locale/Punic/Calendar.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         switch ($count) {
             case 1:
-                return (string) ((int) $value->format('Y'));
+                return (string)((int)$value->format('Y'));
             case 2:
                 //return $value->format('y');
             default:

xoops_lib/Xoops/Core/Locale/LegacyCodes.php

Spacing +28 added lines, -28 removed lines

@@ -39,34 +39,34 @@
 class LegacyCodes
 {
     private static $rawCodes = array(
-        ['ar_SA', 'ar',      'ar-Arab-SA', ['arabic']],
-        ['bg_BG', 'bg',      'bg-Cyrl-BG', ['bulgarian']],
-        ['cs_CZ', 'cs',      'cs-Latn-CZ', ['czech']],
-        ['da_DK', 'da',      'da-Latn-DK', ['danish']],
-        ['de_DE', 'de',      'de-Latn-DE', ['german']],
-        ['el_GR', 'el',      'el-Grek-GR', ['greek']],
-        ['en_US', 'en',      'en-Latn-US', ['english']],
-        ['es_ES', 'es',      'es-Latn-ES', ['spanish']],
-        ['fa_IR', 'fa',      'fa-Arab-IR', ['persian']],
-        ['fr_FR', 'fr',      'fr-Latn-FR', ['french']],
-        ['hr_HR', 'hr',      'hr-Latn-HR', ['croatian']],
-        ['hu_HU', 'hu',      'hu-Latn-HU', ['hungarian']],
-        ['it_IT', 'it',      'it-Latn-IT', ['italian']],
-        ['ja_JP', 'ja',      'ja-Jpan-JP', ['japanese']],
-        ['ko_KR', 'ko',      'ko-Kore-KR', ['korean']],
-        ['ms_MY', 'ms',      'ms-Latn-MY', ['malaysian']],
-        ['nl_NL', 'nl',      'nl-Latn-NL', ['dutch']],
-        ['no_NO', 'no',      'no-Latn-NO', ['norwegian']],
-        ['pl_PL', 'pl',      'pl-Latn-PL', ['polish']],
-        ['pt_BR', 'pt',      'pt-Latn-BR', ['portuguesebr', 'brazilian']],
-        ['pt_PT', 'pt_PT',   'pt-Latn-PT', ['portuguese']],
-        ['ru_RU', 'ru',      'ru-Cyrl-RU', ['russian']],
-        ['sk_SK', 'sk',      'sk-Latn-SK', ['slovak']],
-        ['sl_SI', 'sl',      'sl-Latn-SI', ['slovenian']],
-        ['sv_SE', 'sv',      'sv-Latn-SE', ['swedish']],
-        ['th_TH', 'th',      'th-Thai-TH', ['thai']],
-        ['tr_TR', 'tr',      'tr-Latn-TR', ['turkish']],
-        ['vi_VN', 'vi',      'vi-Latn-VN', ['vietnamese']],
+        ['ar_SA', 'ar', 'ar-Arab-SA', ['arabic']],
+        ['bg_BG', 'bg', 'bg-Cyrl-BG', ['bulgarian']],
+        ['cs_CZ', 'cs', 'cs-Latn-CZ', ['czech']],
+        ['da_DK', 'da', 'da-Latn-DK', ['danish']],
+        ['de_DE', 'de', 'de-Latn-DE', ['german']],
+        ['el_GR', 'el', 'el-Grek-GR', ['greek']],
+        ['en_US', 'en', 'en-Latn-US', ['english']],
+        ['es_ES', 'es', 'es-Latn-ES', ['spanish']],
+        ['fa_IR', 'fa', 'fa-Arab-IR', ['persian']],
+        ['fr_FR', 'fr', 'fr-Latn-FR', ['french']],
+        ['hr_HR', 'hr', 'hr-Latn-HR', ['croatian']],
+        ['hu_HU', 'hu', 'hu-Latn-HU', ['hungarian']],
+        ['it_IT', 'it', 'it-Latn-IT', ['italian']],
+        ['ja_JP', 'ja', 'ja-Jpan-JP', ['japanese']],
+        ['ko_KR', 'ko', 'ko-Kore-KR', ['korean']],
+        ['ms_MY', 'ms', 'ms-Latn-MY', ['malaysian']],
+        ['nl_NL', 'nl', 'nl-Latn-NL', ['dutch']],
+        ['no_NO', 'no', 'no-Latn-NO', ['norwegian']],
+        ['pl_PL', 'pl', 'pl-Latn-PL', ['polish']],
+        ['pt_BR', 'pt', 'pt-Latn-BR', ['portuguesebr', 'brazilian']],
+        ['pt_PT', 'pt_PT', 'pt-Latn-PT', ['portuguese']],
+        ['ru_RU', 'ru', 'ru-Cyrl-RU', ['russian']],
+        ['sk_SK', 'sk', 'sk-Latn-SK', ['slovak']],
+        ['sl_SI', 'sl', 'sl-Latn-SI', ['slovenian']],
+        ['sv_SE', 'sv', 'sv-Latn-SE', ['swedish']],
+        ['th_TH', 'th', 'th-Thai-TH', ['thai']],
+        ['tr_TR', 'tr', 'tr-Latn-TR', ['turkish']],
+        ['vi_VN', 'vi', 'vi-Latn-VN', ['vietnamese']],
         ['zh_CN', 'zh_Hans', 'zh-Hans-CN', ['schinese']],
         ['zh_TW', 'zh_Hant', 'zh-Hant-TW', ['tchinese', 'chinese_zh']],
     );

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) {