phptek / silverstripe-exodus

external-content/code/transform/QueuedExternalContentImporter.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         $target = null,
         $includeParent = false,
         $includeChildren = true,
-        $duplicateStrategy='overwrite',
+        $duplicateStrategy = 'overwrite',
         $params = array()
     ) {
         if ($contentItem) {

external-content/code/transform/ExternalContentImporter.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * 			How to handle duplication
      * @param array $params All parameters passed with the import request.
      */
-    public function import($contentItem, $target, $includeParent = false, $includeChildren = true, $duplicateStrategy='Overwrite', $params = array())
+    public function import($contentItem, $target, $includeParent = false, $includeChildren = true, $duplicateStrategy = 'Overwrite', $params = array())
     {
         $this->runOnImportStart();
         $this->params = $params;

external-content/code/controllers/ExternalContentAdmin.php

Spacing +13 added lines, -14 removed lines

@@ -140,7 +140,7 @@ 
             }
 
             if (!$filterFunction) {
-                $filterFunction = function ($node) use ($filter) {
+                $filterFunction = function($node) use ($filter) {
                     return $filter->isPageIncluded($node);
                 };
             }
@@ -176,7 +176,7 @@ 
 
         // Render using full-subtree template
         return $markingSet->renderChildren(
-            [ self::class . '_SubTree', 'type' => 'Includes' ],
+            [self::class . '_SubTree', 'type' => 'Includes'],
             $this->getTreeNodeCustomisations()
         );
     }
@@ -189,7 +189,7 @@ 
     protected function getTreeNodeCustomisations()
     {
         $rootTitle = $this->getCMSTreeTitle();
-        return function ($node) use ($rootTitle) {
+        return function($node) use ($rootTitle) {
             return [
                 'listViewLink' => $this->LinkListViewChildren($node->ID),
                 'rootTitle' => $rootTitle,
@@ -338,11 +338,11 @@ 
     {
         $form = $this->getEditForm();
         $migrationTarget 		= isset($request['MigrationTarget']) ? $request['MigrationTarget'] : '';
-        $fileMigrationTarget 	= isset($request['FileMigrationTarget']) ? $request['FileMigrationTarget'] : '';
+        $fileMigrationTarget = isset($request['FileMigrationTarget']) ? $request['FileMigrationTarget'] : '';
         $includeSelected 		= isset($request['IncludeSelected']) ? $request['IncludeSelected'] : 0;
         $includeChildren 		= isset($request['IncludeChildren']) ? $request['IncludeChildren'] : 0;
-        $duplicates 			= isset($request['DuplicateMethod']) ? $request['DuplicateMethod'] : ExternalContentTransformer::DS_OVERWRITE;
-        $selected 				= isset($request['ID']) ? $request['ID'] : 0;
+        $duplicates = isset($request['DuplicateMethod']) ? $request['DuplicateMethod'] : ExternalContentTransformer::DS_OVERWRITE;
+        $selected = isset($request['ID']) ? $request['ID'] : 0;
 
         if (!$selected) {
             $messageType = 'bad';
@@ -635,7 +635,7 @@ 
             }
 
             $visible = explode('\\', $class);
-            $classes[$key] = FormField::name_to_label($visible[count($visible)-1]);
+            $classes[$key] = FormField::name_to_label($visible[count($visible) - 1]);
         }
 
         $fields = FieldList::create(
@@ -668,11 +668,10 @@ 
         // Providers are ALWAYS at the root
         $parent = 0;
         $name = (isset($_REQUEST['Name'])) ?
-            basename($_REQUEST['Name']) :
-            _t('ExternalContent.NEWCONNECTOR', "New Connector");
+            basename($_REQUEST['Name']) : _t('ExternalContent.NEWCONNECTOR', "New Connector");
         $type = $_REQUEST['ProviderType'];
         $providerClasses = array_map(
-            function ($item) {
+            function($item) {
                 return strtolower($item);
             },
             ClassInfo::subclassesFor(self::$tree_class)
@@ -832,9 +831,9 @@ 
     {
         $css = '';
 
-        $sourceClasses 	= ClassInfo::subclassesFor(ExternalContentSource::class);
-        $itemClasses 	= ClassInfo::subclassesFor(ExternalContentItem::class);
-        $classes 		= array_merge($sourceClasses, $itemClasses);
+        $sourceClasses = ClassInfo::subclassesFor(ExternalContentSource::class);
+        $itemClasses = ClassInfo::subclassesFor(ExternalContentItem::class);
+        $classes = array_merge($sourceClasses, $itemClasses);
 
         foreach ($classes as $class) {
             $obj = singleton($class);
@@ -922,7 +921,7 @@ 
         }
 
         if (!$record || !$record->ID) {
-            $this->httpError(404, "Bad record ID #" . (int)$data['ID']);
+            $this->httpError(404, "Bad record ID #" . (int) $data['ID']);
         }
 
         $type = $record->ClassName;

external-content/code/cache/CacheService.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
      * @param int $expiry
      * 			How many seconds to cache this object for (no value uses the configured default)
      */
-    public function store($key, $value, $expiry=0)
+    public function store($key, $value, $expiry = 0)
     {
         if ($expiry == 0) {
             $expiry = $this->expiry;

external-content/remoteclients/WebApiClient.php

Indentation +4 added lines, -6 removed lines

@@ -5,11 +5,10 @@ 
 All rights reserved.
 
 Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are met:
-
-    * Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer.
-    * Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the
+ * Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer.
+ * Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the
       documentation and/or other materials provided with the distribution.
-    * Neither the name of SilverStripe nor the names of its contributors may be used to endorse or promote products derived from this software
+ * Neither the name of SilverStripe nor the names of its contributors may be used to endorse or promote products derived from this software
       without specific prior written permission.
 
 THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
@@ -18,8 +17,7 @@ 
 GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
 STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY
 OF SUCH DAMAGE.
-
-*/
+ */
 
 /**
  * An object that acts as a client to a web API of some sort.

Spacing +7 added lines, -7 removed lines

@@ -163,7 +163,7 @@ 
      * Create a new webapi client
      *
      */
-    public function __construct($url, $methods=null, $globalParams=null)
+    public function __construct($url, $methods = null, $globalParams = null)
     {
         $this->baseUrl = $url;
         $this->methods = $methods;
@@ -210,7 +210,7 @@ 
         if (preg_match_all('/{(\w+)}/', $uri, $matches)) {
             foreach ($matches[1] as $match) {
                 if (isset($args[$match])) {
-                    $uri = str_replace('{'.$match.'}', $args[$match], $uri);
+                    $uri = str_replace('{' . $match . '}', $args[$match], $uri);
                 }
             }
         }
@@ -280,10 +280,10 @@ 
                 }
             } else {
                 if ($response->getStatus() == 500) {
-                    error_log("Failure: ".$response->getBody());
+                    error_log("Failure: " . $response->getBody());
                     error_log(var_export($client, true));
                 }
-                throw new FailedRequestException("Failed executing $method: ".$response->getMessage()." for request to $uri (".$client->getUri(true).')', $response->getBody());
+                throw new FailedRequestException("Failed executing $method: " . $response->getMessage() . " for request to $uri (" . $client->getUri(true) . ')', $response->getBody());
             }
         }
 
@@ -335,7 +335,7 @@ 
             $client->setMethod($requestType);
             // set the encoding type
             $client->setEncType($enctype);
-            $paramMethod = 'setParameter'.$requestType;
+            $paramMethod = 'setParameter' . $requestType;
             // make sure to add the alfTicket parameter
             if ($this->globalParams) {
                 foreach ($this->globalParams as $key => $value) {
@@ -357,10 +357,10 @@ 
                 }
             } else {
                 if ($response->getStatus() == 500) {
-                    error_log("Failure: ".$response->getBody());
+                    error_log("Failure: " . $response->getBody());
                     error_log(var_export($client, true));
                 }
-                throw new FailedRequestException("Failed executing $url: ".$response->getMessage()." for request to $uri (".$client->getUri(true).')', $response->getBody());
+                throw new FailedRequestException("Failed executing $url: " . $response->getMessage() . " for request to $uri (" . $client->getUri(true) . ')', $response->getBody());
             }
         }