Merge pull request #15 from ProcessMaker/feature/FOUR-6588

Applying php-cs-fixer formatting

Author: ryancooley

.gitignore

@@ -1,2 +1,4 @@
 src/vendor
 /sdk
+.php-cs-fixer.php
+.php-cs-fixer.cache

src/Api.php

@@ -1,17 +1,17 @@
 <?php
+
 namespace Executor;
 
 /**
  * This helper allows you to use Api classes in the namespace
  * ProcessMaker\Client\Api\ by calling its name as a function
  * on an instance of this class. The class name is lowercase first.
- * 
+ *
  * For example, to call \ProcessMaker\Client\Api\ProcessesApi
  * call $api->processes()
- * 
  */
-class Api {
-
+class Api
+{
     /**
      * Reusable instances of api objects
      *
@@ -37,9 +37,10 @@ class Api {
      * Constructor
      *
      * @param \ProcessMaker\Client\Configuration $config
-     * @param boolean $ssl_verify
+     * @param bool $ssl_verify
      */
-    public function __construct($config, $ssl_verify) {
+    public function __construct($config, $ssl_verify)
+    {
         $this->client = new \GuzzleHttp\Client(['verify' => $ssl_verify]);
         $this->config = $config;
     }
@@ -54,18 +55,17 @@ public function __construct($config, $ssl_verify) {
     public function __call($name, $arguments)
     {
         if (count($arguments) > 0) {
-            throw new \BadMethodCallException("Arguments should not be passed");
+            throw new \BadMethodCallException('Arguments should not be passed');
         }
 
         $class_name = $this->getClassName($name);
 
         if (array_key_exists($class_name, $this->instances)) {
             return $this->instances[$class_name];
-
         } elseif (class_exists($this->getClassName($name))) {
             $this->instances[$class_name] = new $class_name($this->client, $this->config);
-            return $this->instances[$class_name];
 
+            return $this->instances[$class_name];
         } else {
             throw new \BadMethodCallException("class $class_name does not exist");
         }
@@ -79,6 +79,6 @@ public function __call($name, $arguments)
      */
     private function getClassName($name)
     {
-        return '\\ProcessMaker\\Client\Api\\' . ucfirst($name) . "Api";
+        return '\\ProcessMaker\\Client\Api\\' . ucfirst($name) . 'Api';
     }
-}
+}

src/DockerExecutorPhpServiceProvider.php

@@ -1,10 +1,11 @@
 <?php
+
 namespace ProcessMaker\Package\DockerExecutorPhp;
 
 use Illuminate\Support\Facades\Route;
 use Illuminate\Support\ServiceProvider;
-use ProcessMaker\Traits\PluginServiceProviderTrait;
 use ProcessMaker\Models\ScriptExecutor;
+use ProcessMaker\Traits\PluginServiceProviderTrait;
 
 class DockerExecutorPhpServiceProvider extends ServiceProvider
 {
@@ -23,12 +24,12 @@ public function boot()
                 'language' => 'php',
                 'title' => 'PHP Executor',
                 'description' => 'Default PHP Executor',
-                'config' => 'RUN composer require aws/aws-sdk-php:3.226.0'
+                'config' => 'RUN composer require aws/aws-sdk-php:3.226.0',
             ]);
-            
+
             // Build the instance image. This is the same as if you were to build it from the admin UI
             \Artisan::call('processmaker:build-script-executor ' . $scriptExecutor->id);
-            
+
             // Restart the workers so they know about the new supported language
             // \Artisan::call('horizon:terminate');
         });
@@ -39,7 +40,7 @@ public function boot()
             'name' => 'PHP',
             'runner' => 'PhpRunner',
             'mime_type' => 'application/x-php',
-            'options' => ['invokerPackage' => "ProcessMaker\\Client"],
+            'options' => ['invokerPackage' => 'ProcessMaker\\Client'],
             'init_dockerfile' => [
                 'ARG SDK_DIR',
                 'COPY $SDK_DIR /opt/sdk-php',

src/ScriptRunners/PhpRunner.php

@@ -17,6 +17,7 @@ public function config($code, array $dockerConfig)
         $dockerConfig['image'] = config('script-runners.php.image');
         $dockerConfig['command'] = 'php /opt/executor/bootstrap.php';
         $dockerConfig['inputs']['/opt/executor/script.php'] = $code;
+
         return $dockerConfig;
     }
 }

src/TestDocs.php

@@ -52,14 +52,14 @@ public function handle()
         }
 
         try {
-            foreach($codes as $code) {
+            foreach ($codes as $code) {
                 $this->runCode($code);
             }
-        } catch(\ProcessMaker\Exception\ScriptException $e) {
+        } catch (\ProcessMaker\Exception\ScriptException $e) {
             $stack = $e->getMessage();
             $stack = explode("\n", $stack);
             $stack = array_slice($stack, 0, 15);
-            $stack = join("\n", $stack);
+            $stack = implode("\n", $stack);
 
             $this->info($stack);
         }
@@ -75,7 +75,7 @@ private function runCode($code)
             'timeout' => 60,
         ]);
 
-        $result = $script->runScript([],[]);
+        $result = $script->runScript([], []);
 
         $script->delete();
 

src/bootstrap.php

@@ -1,4 +1,5 @@
 <?php
+
 // Include our composer libraries
 require __DIR__ . '/vendor/autoload.php';
 
@@ -13,25 +14,24 @@
 const CONFIG_JSON_INVALID = 101;
 const SCRIPT_PATH_INVALID = 102;
 
-
 $data = json_decode(@file_get_contents(DATA_JSON_PATH), true);
-if($data === null) {
+if ($data === null) {
     // Terminate script, the data is invalid json or cannot be ready
-    fwrite(STDERR, "Data JSON file (" . DATA_JSON_PATH . ") does not exist or is invalid.");
+    fwrite(STDERR, 'Data JSON file (' . DATA_JSON_PATH . ') does not exist or is invalid.');
     exit(DATA_JSON_INVALID);
 }
 
 $config = json_decode(@file_get_contents(CONFIG_JSON_PATH), true);
-if($config === null) {
+if ($config === null) {
     // Terminate script, the config is invalid json or cannot be ready
-    fwrite(STDERR, "Config JSON file (" . CONFIG_JSON_PATH . ") does not exist or is invalid.");
+    fwrite(STDERR, 'Config JSON file (' . CONFIG_JSON_PATH . ') does not exist or is invalid.');
     exit(CONFIG_JSON_INVALID);
 }
 
 // Check for existence of script
-if(!file_exists(SCRIPT_PATH)) {
+if (!file_exists(SCRIPT_PATH)) {
     // Terminate script, the config is invalid json or cannot be ready
-    fwrite(STDERR, "Script file (" . SCRIPT_PATH . ") does not exist or is invalid.");
+    fwrite(STDERR, 'Script file (' . SCRIPT_PATH . ') does not exist or is invalid.');
     exit(SCRIPT_PATH_INVALID);
 }
 
@@ -42,9 +42,7 @@
     $api = new Executor\Api($api_config, isset($_ENV['API_SSL_VERIFY']) ? (bool) $_ENV['API_SSL_VERIFY'] : true);
 }
 
-$response = require(SCRIPT_PATH);
+$response = require SCRIPT_PATH;
 
 // Finally store the output of our script into our output JSON path
 file_put_contents(OUTPUT_JSON_PATH, json_encode($response));
-
-

src/tests/ApiTest.php

@@ -1,10 +1,12 @@
 <?php
+
 use PHPUnit\Framework\TestCase;
 
 class ApiTest extends TestCase
 {
-    const API_TOKEN = "eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImp0aSI6IjQyN2RlODA3MzhiZWNmYzgyZGFiMDA4MTExYzQxYTFkZTM3NzYyYjg5MDYyYWQyZjcxZTAzYzAwNzk3YTc5NDBjY2U5MGFlM2IzZTE1MWE1In0.eyJhdWQiOiIxIiwianRpIjoiNDI3ZGU4MDczOGJlY2ZjODJkYWIwMDgxMTFjNDFhMWRlMzc3NjJiODkwNjJhZDJmNzFlMDNjMDA3OTdhNzk0MGNjZTkwYWUzYjNlMTUxYTUiLCJpYXQiOjE1NzkxMzY2NzQsIm5iZiI6MTU3OTEzNjY3NCwiZXhwIjoxNjEwNzU5MDc0LCJzdWIiOiIxIiwic2NvcGVzIjpbXX0.mQtWB8xclNpWMn6bpX6BnceJey8xu-IMicrzkLcbsTT9vXwvrn4e25Cbyzfahz7vwjqi10s2j7CeLJZwSkeF_k4f4uZsfJpDoTt08K1tYa_cRfpuoInP_2PhJnBIVZO2kdljJBwC3K_9gWgBzsGKS0I2fglSEycWaiIXWx5jAy8QS8hPYGPsrNG4YDTiNiUnYvbmY8c9JHbKS-yLtRZFEnkF7k-zUgSFKVz6WAwO2TcqCB_YX8RVKmnzG1KMgzXaGV-B4kCH5zdY74ZFLIwJalflTmA9jAgtaR3GsXeWqWXkxnPIIYHQdNlNONDqsJ1troo-3rH6hwZETRHKPVtrBAlR9q2k1SYRC9GUhC1otLJH0PXKiabUryIHPXC3cBavCYQ6MGEv2ou5ZfAuAAcKovWZd7-GDqwnDQxx-uMyIEiGrsZAt8EZiDFETHq1eBIGQLVezFbYhnOACYARWrMarZrQ6Q5yCKLrZ4KqIeaoJoPJOf8UJIQcpqhRAonfX_0RDLRWdbAW9ltfYkmfzWU2uJe15YqZu2wi9Bss0iGXWuRq2ASeNERgg_tkNe9ELQMxSKTx8D7tDe-g0Ja1QViLawmHK8XYDsmmKgOSMC9Xj-jd-Dg-ZjA3sMXWTEomudQyjQIBfdu68T4rGd_1lfneZAEEqtw9y2dKR7if13Fwar4";
-    const API_HOST = "http://pm.local.processmaker.com/api/1.0";
+    const API_TOKEN = 'eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImp0aSI6IjQyN2RlODA3MzhiZWNmYzgyZGFiMDA4MTExYzQxYTFkZTM3NzYyYjg5MDYyYWQyZjcxZTAzYzAwNzk3YTc5NDBjY2U5MGFlM2IzZTE1MWE1In0.eyJhdWQiOiIxIiwianRpIjoiNDI3ZGU4MDczOGJlY2ZjODJkYWIwMDgxMTFjNDFhMWRlMzc3NjJiODkwNjJhZDJmNzFlMDNjMDA3OTdhNzk0MGNjZTkwYWUzYjNlMTUxYTUiLCJpYXQiOjE1NzkxMzY2NzQsIm5iZiI6MTU3OTEzNjY3NCwiZXhwIjoxNjEwNzU5MDc0LCJzdWIiOiIxIiwic2NvcGVzIjpbXX0.mQtWB8xclNpWMn6bpX6BnceJey8xu-IMicrzkLcbsTT9vXwvrn4e25Cbyzfahz7vwjqi10s2j7CeLJZwSkeF_k4f4uZsfJpDoTt08K1tYa_cRfpuoInP_2PhJnBIVZO2kdljJBwC3K_9gWgBzsGKS0I2fglSEycWaiIXWx5jAy8QS8hPYGPsrNG4YDTiNiUnYvbmY8c9JHbKS-yLtRZFEnkF7k-zUgSFKVz6WAwO2TcqCB_YX8RVKmnzG1KMgzXaGV-B4kCH5zdY74ZFLIwJalflTmA9jAgtaR3GsXeWqWXkxnPIIYHQdNlNONDqsJ1troo-3rH6hwZETRHKPVtrBAlR9q2k1SYRC9GUhC1otLJH0PXKiabUryIHPXC3cBavCYQ6MGEv2ou5ZfAuAAcKovWZd7-GDqwnDQxx-uMyIEiGrsZAt8EZiDFETHq1eBIGQLVezFbYhnOACYARWrMarZrQ6Q5yCKLrZ4KqIeaoJoPJOf8UJIQcpqhRAonfX_0RDLRWdbAW9ltfYkmfzWU2uJe15YqZu2wi9Bss0iGXWuRq2ASeNERgg_tkNe9ELQMxSKTx8D7tDe-g0Ja1QViLawmHK8XYDsmmKgOSMC9Xj-jd-Dg-ZjA3sMXWTEomudQyjQIBfdu68T4rGd_1lfneZAEEqtw9y2dKR7if13Fwar4';
+
+    const API_HOST = 'http://pm.local.processmaker.com/api/1.0';
 
     public function testCallUserApi()
     {
@@ -23,4 +25,4 @@ public function testApiDoesNotExist()
         $this->expectException(BadMethodCallException::class);
         $this->assertEquals('ok', $api->foo()->getUsers());
     }
-}
+}