style fixes

Author: OriHoch

src/DataSources/CsvDataSource.php

@@ -95,7 +95,8 @@ public function close()
         }
     }
 
-    public function save($outputDataSource) {
+    public function save($outputDataSource)
+    {
         $file = fopen($outputDataSource, 'w');
         fputcsv($file, $this->headerRow);
         while (!$this->isEof()) {

src/DataSources/DataSourceInterface.php

@@ -27,7 +27,7 @@ public function open();
     public function getNextLine();
 
     /**
-     * iterate over all rows and save to the given output data source
+     * iterate over all rows and save to the given output data source.
      *
      * @param $outputDataSource
      */

src/Table.php

@@ -18,14 +18,14 @@ class Table implements \Iterator
      *
      * @throws Exceptions\DataSourceException
      */
-    public function __construct($dataSource, $schema=null)
+    public function __construct($dataSource, $schema = null)
     {
-        if (!is_a($dataSource, "frictionlessdata\\tableschema\\DataSources\\BaseDataSource")) {
+        if (!is_a($dataSource, 'frictionlessdata\\tableschema\\DataSources\\BaseDataSource')) {
             // TODO: more advanced data source detection
             $dataSource = new CsvDataSource($dataSource);
         }
         $this->dataSource = $dataSource;
-        if (!is_a($schema, "frictionlessdata\\tableschema\\Schema")) {
+        if (!is_a($schema, 'frictionlessdata\\tableschema\\Schema')) {
             if ($schema) {
                 $schema = new Schema($schema);
             } else {
@@ -84,12 +84,14 @@ public static function validate($dataSource, $schema, $numPeekRows = 10)
     public function schema($numPeekRows = 10)
     {
         $this->ensureInferredSchema($numPeekRows);
+
         return $this->schema;
     }
 
     public function headers($numPeekRows = 10)
     {
         $this->ensureInferredSchema($numPeekRows);
+
         return array_keys($this->schema->fields());
     }
 
@@ -99,6 +101,7 @@ public function read()
         foreach ($this as $row) {
             $rows[] = $row;
         }
+
         return $rows;
     }
 
@@ -182,7 +185,7 @@ public function valid()
 
     protected function isInferSchema()
     {
-        return is_a($this->schema, "frictionlessdata\\tableschema\\InferSchema");
+        return is_a($this->schema, 'frictionlessdata\\tableschema\\InferSchema');
     }
 
     protected function ensureInferredSchema($numPeekRows = 10)
@@ -199,6 +202,6 @@ protected function ensureInferredSchema($numPeekRows = 10)
                 // these rows will be returned by next current() call
                 $this->castRows = $this->schema->lock();
             }
-        };
+        }
     }
 }

tests/TableTest.php

@@ -102,21 +102,21 @@ public function testInferSchemaFailsAfterLock()
     public function testAutoInferSchemaWhenNullSchema()
     {
         $table = new Table($this->fixture('data.csv'));
-        $this->assertTrue(is_a($table->schema(), "frictionlessdata\\tableschema\\InferSchema"));
+        $this->assertTrue(is_a($table->schema(), 'frictionlessdata\\tableschema\\InferSchema'));
     }
 
     public function testHeaders()
     {
         $table = new Table($this->fixture('data.csv'));
-        $this->assertEquals(["first_name", "last_name", "order"], $table->headers());
+        $this->assertEquals(['first_name', 'last_name', 'order'], $table->headers());
         $this->assertEquals([
             ['first_name' => 'Foo', 'last_name' => 'Bar', 'order' => 1],
             ['first_name' => 'Baz', 'last_name' => 'Bax', 'order' => 2],
             ['first_name' => 'באך', 'last_name' => 'ביי', 'order' => 3],
         ], $table->read());
 
         $table = new Table($this->fixture('data.csv'));
-        $this->assertEquals(["first_name", "last_name", "order"], $table->headers(1));
+        $this->assertEquals(['first_name', 'last_name', 'order'], $table->headers(1));
         $this->assertEquals([
             ['first_name' => 'Foo', 'last_name' => 'Bar', 'order' => 1],
             ['first_name' => 'Baz', 'last_name' => 'Bax', 'order' => 2],
@@ -132,12 +132,12 @@ public function testHeaders()
     public function testTableSave()
     {
         $table = new Table($this->fixture('data.csv'));
-        $table->save("test-table-save-data.csv");
+        $table->save('test-table-save-data.csv');
         $this->assertEquals(
             "first_name,last_name,order\nFoo,Bar,1\nBaz,Bax,2\nבאך,ביי,3\n",
-            file_get_contents("test-table-save-data.csv")
+            file_get_contents('test-table-save-data.csv')
         );
-        unlink("test-table-save-data.csv");
+        unlink('test-table-save-data.csv');
     }
 
     public function testInferSchemaWorksWithMoreRows()
@@ -160,13 +160,13 @@ public function testInferSchemaWorksWithMoreRows()
 
     public function testSimpleInferSchema()
     {
-        $table = new Table($this->fixture("data.csv"));
-        $this->assertEquals((object)[
-            "fields" => [
-                (object)["name" => "first_name", "type" => "string"],
-                (object)["name" => "last_name", "type" => "string"],
-                (object)["name" => "order", "type" => "integer"],
-            ]
+        $table = new Table($this->fixture('data.csv'));
+        $this->assertEquals((object) [
+            'fields' => [
+                (object) ['name' => 'first_name', 'type' => 'string'],
+                (object) ['name' => 'last_name', 'type' => 'string'],
+                (object) ['name' => 'order', 'type' => 'integer'],
+            ],
         ], $table->schema()->descriptor());
     }