Allow userId to be an optional argument. (#556)

fiasco · Nyholm · commit 65a205979ccf · 2017-04-14T10:16:12.000+02:00
diff --git a/lib/Github/Api/Integrations.php b/lib/Github/Api/Integrations.php
@@ -23,7 +23,6 @@ public function createInstallationToken($installationId, $userId = null)
         if ($userId) {
             $parameters['user_id'] = $userId;
         }
-
         return $this->post('/installations/'.rawurlencode($installationId).'/access_tokens', $parameters);
     }
 
@@ -48,9 +47,13 @@ public function findInstallations()
      *
      * @return array
      */
-    public function listRepositories($userId)
+    public function listRepositories($userId = null)
     {
-        return $this->get('/installation/repositories', ['user_id' => $userId]);
+        $parameters = array();
+        if ($userId) {
+            $parameters['user_id'] = $userId;
+        }
+        return $this->get('/installation/repositories', $parameters);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,6 @@ public function createInstallationToken($installationId, $userId = null)`
`23`	`23`	`if ($userId) {`
`24`	`24`	`$parameters['user_id'] = $userId;`
`25`	`25`	`}`
`26`		`-`
`27`	`26`	`return $this->post('/installations/'.rawurlencode($installationId).'/access_tokens', $parameters);`
`28`	`27`	`}`
`29`	`28`
`@@ -48,9 +47,13 @@ public function findInstallations()`
`48`	`47`	`*`
`49`	`48`	`* @return array`
`50`	`49`	`*/`
`51`		`- public function listRepositories($userId)`
	`50`	`+ public function listRepositories($userId = null)`
`52`	`51`	`{`
`53`		`- return $this->get('/installation/repositories', ['user_id' => $userId]);`
	`52`	`+ $parameters = array();`
	`53`	`+ if ($userId) {`
	`54`	`+ $parameters['user_id'] = $userId;`
	`55`	`+ }`
	`56`	`+ return $this->get('/installation/repositories', $parameters);`
`54`	`57`	`}`
`55`	`58`
`56`	`59`	`/**`