OriHoch / tableschema-php

src/SchemaValidationError.php

Spacing +7 added lines, -7 removed lines

@@ -3,11 +3,11 @@ 
 
 class SchemaValidationError
 {
-    const LOAD_FAILED=1;
-    const SCHEMA_VIOLATION=8;
-    const FIELD_VALIDATION=21;
-    const ROW_FIELD_VALIDATION=22;
-    const ROW_VALIDATION=23;
+    const LOAD_FAILED = 1;
+    const SCHEMA_VIOLATION = 8;
+    const FIELD_VALIDATION = 21;
+    const ROW_FIELD_VALIDATION = 22;
+    const ROW_VALIDATION = 23;
 
     public $code;
     public $extraDetails;
@@ -16,7 +16,7 @@ 
      * @param integer $code
      * @param mixed $extraDetails
      */
-    public function __construct($code, $extraDetails=null)
+    public function __construct($code, $extraDetails = null)
     {
         $this->code = $code;
         $this->extraDetails = $extraDetails;
@@ -61,7 +61,7 @@ 
      */
     public static function getErrorMessages($validationErrors)
     {
-        return implode(", ", array_map(function($validationError){
+        return implode(", ", array_map(function($validationError) {
             /** @var SchemaValidationError $validationError */
             return $validationError->getMessage();
         }, $validationErrors));

src/InferSchema.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
  */
 class InferSchema extends Schema
 {
-    public function __construct($descriptor=null, $lenient=false)
+    public function __construct($descriptor = null, $lenient = false)
     {
         $this->descriptor = empty($descriptor) ? (object)["fields" => []] : $descriptor;
         $this->fieldsInferer = new Fields\FieldsInferrer(null, $lenient);
@@ -39,7 +39,7 @@ 
                 $this->descriptor->fields[] = $inferredField->descriptor();
             }
             $this->castRows = $this->fieldsInferer->castRows();
-            return $this->castRows[count($this->castRows)-1];
+            return $this->castRows[count($this->castRows) - 1];
         }
     }
 

src/Table.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
      * @param int $numPeekRows
      * @return array of validation errors
      */
-    public static function validate($dataSource, $schema, $numPeekRows=10)
+    public static function validate($dataSource, $schema, $numPeekRows = 10)
     {
         try {
             $table = new static($dataSource, $schema);
@@ -49,7 +49,7 @@ 
                 // validation error in one of the fields
                 return array_map(function($validationError) use ($i) {
                     return new SchemaValidationError(SchemaValidationError::ROW_FIELD_VALIDATION, [
-                        "row" => $i+1,
+                        "row" => $i + 1,
                         "field" => $validationError->extraDetails["field"],
                         "error" => $validationError->extraDetails["error"],
                         "value" => $validationError->extraDetails["value"],
@@ -73,11 +73,11 @@ 
 
     // not interesting, standard iterator functions
     // to simplify we prevent rewinding - so you can only iterate once
-    public function __destruct() {$this->dataSource->close();}
-    public function rewind() {if ($this->currentLine == 0) {$this->currentLine = 1;} else {throw new \Exception("rewind is not supported");}}
-    public function key() {return $this->currentLine;}
-    public function next() {$this->currentLine++;}
-    public function valid() {return !$this->dataSource->isEof();}
+    public function __destruct() {$this->dataSource->close(); }
+    public function rewind() {if ($this->currentLine == 0) {$this->currentLine = 1; } else {throw new \Exception("rewind is not supported"); }}
+    public function key() {return $this->currentLine; }
+    public function next() {$this->currentLine++; }
+    public function valid() {return !$this->dataSource->isEof(); }
 
     protected $currentLine = 0;
     protected $dataSource;

Braces +3 added lines, -1 removed lines

@@ -37,7 +37,9 @@
             $i = 0;
             try {
                 foreach ($table as $row) {
-                    if (++$i > $numPeekRows) break;
+                    if (++$i > $numPeekRows) {
+                        break;
+                    }
                 }
             } catch (Exceptions\DataSourceException $e) {
                 // general error in getting the next row from the data source

src/Fields/FieldsFactory.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
         throw new FieldValidationException([
             new SchemaValidationError(
                 SchemaValidationError::SCHEMA_VIOLATION,
-                "Could not find a valid field for descriptor: ".json_encode($descriptor))
+                "Could not find a valid field for descriptor: " . json_encode($descriptor))
         ]);
     }
 
@@ -43,7 +43,7 @@ 
      * @return mixed
      * @throws FieldValidationException
      */
-    public static function infer($val, $descriptor=null, $lenient=false)
+    public static function infer($val, $descriptor = null, $lenient = false)
     {
         foreach (static::$fieldClasses as $fieldClass) {
             /** @var BaseField $fieldClass */
@@ -54,7 +54,7 @@ 
         throw new FieldValidationException([
             new SchemaValidationError(
                 SchemaValidationError::SCHEMA_VIOLATION,
-                "Could not find a valid field for value: ".json_encode($val))
+                "Could not find a valid field for value: " . json_encode($val))
         ]);
     }
 }
\ No newline at end of file

src/Fields/BaseField.php

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
 
 abstract class BaseField
 {
-    public function __construct($descriptor=null)
+    public function __construct($descriptor = null)
     {
         $this->descriptor = empty($descriptor) ? (object)[] : $descriptor;
     }
@@ -44,7 +44,7 @@ 
      * @param bool @lenient
      * @return bool|BaseField
      */
-    public static function infer($val, $descriptor=null, $lenient=false)
+    public static function infer($val, $descriptor = null, $lenient = false)
     {
         $field = new static($descriptor);
         try {
@@ -94,7 +94,7 @@ 
      * @param bool @lenient
      * @return string
      */
-    public function getInferIdentifier($lenient=false)
+    public function getInferIdentifier($lenient = false)
     {
         return $this->type();
     }
@@ -110,7 +110,7 @@ 
 
     protected $descriptor;
 
-    protected function getValidationException($errorMsg, $val=null)
+    protected function getValidationException($errorMsg, $val = null)
     {
         return new FieldValidationException([
             new SchemaValidationError(SchemaValidationError::FIELD_VALIDATION, [

src/Fields/FieldsInferrer.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
     /**
      * @param null|array $rows optional initial rows to infer by, each row is an array of field name => field value
      */
-    public function __construct($rows=null, $lenient=false)
+    public function __construct($rows = null, $lenient = false)
     {
         $this->lenient = $lenient;
         if (!empty($rows)) {

Braces +3 added lines, -1 removed lines

@@ -116,7 +116,9 @@
             try {
                 $rowNum = 0;
                 foreach ($this->inputRows as $inputRow) {
-                    if (!array_key_exists($rowNum, $this->castRows)) $this->castRows[$rowNum] = [];
+                    if (!array_key_exists($rowNum, $this->castRows)) {
+                        $this->castRows[$rowNum] = [];
+                    }
                     $this->castRows[$rowNum][$fieldName] = $inferredField->castValue($inputRow[$fieldName]);
                     $rowNum++;
                 }

src/Fields/StringField.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
         return isset($this->descriptor()->format) ? $this->descriptor()->format : null;
     }
 
-    public function inferProperties($val, $lenient=false)
+    public function inferProperties($val, $lenient = false)
     {
         parent::inferProperties($val, $lenient);
         if (!$lenient) {
@@ -39,12 +39,12 @@ 
         return "string";
     }
 
-    public function getInferIdentifier($lenient=false)
+    public function getInferIdentifier($lenient = false)
     {
         $inferId = parent::getInferIdentifier();
         $format = $this->format();
         if (!$lenient && !empty($format)) {
-            $inferId .= ":".$this->format();
+            $inferId .= ":" . $this->format();
         };
         return $inferId;
     }