Clean up a couple things

laracasts · Feb 28, 2015 · d83213c · d83213c
1 parent 20ba013
commit d83213c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/spec/Migrations/SchemaParserSpec.php b/spec/Migrations/SchemaParserSpec.php
@@ -47,10 +47,7 @@ function it_parses_schema_with_multiple_fields_using_no_spaces()
             ['name' => 'name', 'type' => 'string', 'arguments' => [], 'options' => []],
             ['name' => 'age', 'type' => 'integer', 'arguments' => [], 'options' => []],
         ]);
-    }
 
-    function it_parses_schema_with_multiple_fields_containing_arguments_using_no_spaces()
-    {
         $this->parse('name:string,amount:decimal(5,2)')->shouldReturn([
             ['name' => 'name', 'type' => 'string', 'arguments' => [], 'options' => []],
             ['name' => 'amount', 'type' => 'decimal', 'arguments' => ['5', '2'], 'options' => []]

diff --git a/src/Migrations/SchemaParser.php b/src/Migrations/SchemaParser.php
@@ -34,9 +34,7 @@ public function parse($schema)
      */
     private function getFields($schema)
     {
-        $preg_split = preg_split('/,(?![^()]*+\\))/', $schema);
-
-        return array_map("trim", $preg_split);
+        return preg_split('/,\s?(?![^()]*\))/', $schema);
     }
 
     /**