OriHoch / tableschema-php

src/Table.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace frictionlessdata\tableschema;
 use frictionlessdata\tableschema\Exceptions\DataSourceException;
-use frictionlessdata\tableschema\Fields\BaseField;
 
 /**
  * represents a data source which validates against a table schema

src/SchemaValidator.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
      * @param integer $code
      * @param mixed $extraDetails
      */
-    protected function addError($code, $extraDetails=null)
+    protected function addError($code, $extraDetails = null)
     {
         $this->errors[] = new SchemaValidationError($code, $extraDetails);
     }
@@ -53,7 +53,7 @@ 
         $this->applyForeignKeysResourceHack($descriptor);
         $validator->validate(
             $descriptor,
-            (object)['$ref' => 'file://' . realpath(dirname(__FILE__)).'/schemas/table-schema.json']
+            (object)['$ref' => 'file://' . realpath(dirname(__FILE__)) . '/schemas/table-schema.json']
         );
         if (!$validator->isValid()) {
             foreach ($validator->getErrors() as $error) {

src/InferSchema.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
      * @param null $descriptor optional descriptor object - will be used as an initial descriptor
      * @param bool $lenient if true - infer just basic types, without strict format requirements
      */
-    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);
@@ -44,7 +44,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/Schema.php

Braces +3 added lines, -1 removed lines

@@ -143,7 +143,9 @@
         $validationErrors = [];
         foreach ($this->fields() as $fieldName => $field) {
             $value = array_key_exists($fieldName, $row) ? $row[$fieldName] : null;
-            if (in_array($value, $this->missingValues())) $value = null;
+            if (in_array($value, $this->missingValues())) {
+                $value = null;
+            }
             try {
                 $outRow[$fieldName] = $field->castValue($value);
             } catch (Exceptions\FieldValidationException $e) {

src/Fields/BaseField.php

Spacing +5 added lines, -5 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;
     }
@@ -91,7 +91,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 {
@@ -142,7 +142,7 @@ 
      * @param bool @lenient
      * @return string
      */
-    public function getInferIdentifier($lenient=false)
+    public function getInferIdentifier($lenient = false)
     {
         return $this->type();
     }
@@ -159,7 +159,7 @@ 
     protected $descriptor;
     protected $constraintsDisabled = false;
 
-    protected function getValidationException($errorMsg, $val=null)
+    protected function getValidationException($errorMsg, $val = null)
     {
         return new FieldValidationException([
             new SchemaValidationError(SchemaValidationError::FIELD_VALIDATION, [
@@ -260,7 +260,7 @@ 
 
     protected function checkPatternConstraint($val, $pattern)
     {
-        return preg_match("/^".$pattern."\$/", $val) === 1;
+        return preg_match("/^" . $pattern . "\$/", $val) === 1;
     }
 
     protected function checkMinimumConstraint($val, $minConstraint)

Braces +3 added lines, -1 removed lines

@@ -118,7 +118,9 @@
     final public function castValue($val)
     {
         if ($this->isEmptyValue($val)) {
-            if ($this->required()) throw $this->getValidationException("field is required", $val);
+            if ($this->required()) {
+                throw $this->getValidationException("field is required", $val);
+            }
             return null;
         } else {
             return $this->validateCastValue($val);

src/Fields/StringField.php

Spacing +3 added lines, -3 removed lines

@@ -5,7 +5,7 @@ 
 
 class StringField extends BaseField
 {
-    public function inferProperties($val, $lenient=false)
+    public function inferProperties($val, $lenient = false)
     {
         parent::inferProperties($val, $lenient);
         if (!$lenient) {
@@ -35,12 +35,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;
     }