Merge pull request #141 from Nek-/feature/new-github-filter-options

Added missing sort options on user repositories fetch

Author: cursedcoder

lib/Github/Api/CurrentUser.php

@@ -89,11 +89,19 @@ public function organizations()
     /**
      * @link http://developer.github.com/v3/repos/#list-your-repositories
      *
+     * @param  string $type      role in the repository
+     * @param  string $sort      sort by
+     * @param  string $direction direction of sort, ask or desc
+
      * @return array
      */
-    public function repositories()
+    public function repositories($type = 'owner', $sort = 'full_name', $direction = 'asc')
     {
-        return $this->get('user/repos');
+        return $this->get('user/repos', array(
+            $type,
+            $sort,
+            $direction
+        ));
     }
 
     /**

lib/Github/Api/User.php

@@ -130,12 +130,19 @@ public function subscriptions($username)
      * Get the repositories of a user
      * @link http://developer.github.com/v3/repos/
      *
-     * @param  string $username the username
-     * @return array  list of the user repositories
+     * @param  string $username  the username
+     * @param  string $type      role in the repository
+     * @param  string $sort      sort by
+     * @param  string $direction direction of sort, ask or desc
+     * @return array             list of the user repositories
      */
-    public function repositories($username)
+    public function repositories($username, $type = 'owner', $sort = 'full_name', $direction = 'asc')
     {
-        return $this->get('users/'.rawurlencode($username).'/repos');
+        return $this->get('users/'.rawurlencode($username).'/repos', array(
+            $type,
+            $sort,
+            $direction
+        ));
     }
 
     /**

test/Github/Tests/Api/UserTest.php

@@ -139,7 +139,7 @@ public function shouldGetUserRepositories()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('users/l3l0/repos')
+            ->with('users/l3l0/repos', array('owner', 'full_name', 'asc'))
             ->will($this->returnValue($expectedArray));
 
         $this->assertEquals($expectedArray, $api->repositories('l3l0'));