Merge pull request #31 from dalibert42/1.2

Fixes order by with multiple colums (backport from 1.1)

Author: moufmouf

src/SQLParser/Query/Select.php

@@ -350,7 +350,7 @@ public function toSql(array $parameters = array(), Connection $dbConnection = nu
         }
 
         if (!empty($this->group)) {
-            $groupBy = NodeFactory::toSql($this->group, $dbConnection, $parameters, ' ', false, $indent + 2, $conditionsMode);
+            $groupBy = NodeFactory::toSql($this->group, $dbConnection, $parameters, ',', false, $indent + 2, $conditionsMode);
             if ($groupBy) {
                 $sql .= "\nGROUP BY ".$groupBy;
             }

tests/Mouf/Database/MagicQueryTest.php

@@ -121,6 +121,8 @@ public function testStandardSelect()
         $sql = "SELECT Substring_index(ee.`data`, ':', -1) as foo FROM users";
         $this->assertEquals("SELECT Substring_index(ee.data, ':', -1) AS foo FROM users", self::simplifySql($magicQuery->build($sql)));
 
+        $sql = 'SELECT * FROM users GROUP BY id, login';
+        $this->assertEquals('SELECT * FROM users GROUP BY id, login', self::simplifySql($magicQuery->build($sql)));
     }
 
     /**