Merge pull request #4 from marc94rc/master

norkunas · norkunas · commit 22ec10491a65 · 2015-09-01T20:33:30.000+03:00
Setting default limit fetching notifications
diff --git a/src/Notifications.php b/src/Notifications.php
@@ -46,12 +46,13 @@ public function getOne($id)
      *
      * Application authentication key and ID must be set.
      *
-     * @param int $limit  Results offset (results are sorted by ID)
-     * @param int $offset How many notifications to return (max 50)
+     * @param int $limit How many notifications to return (max 50) 
+     * @param int $offset Results offset (results are sorted by ID)
+     * 
      *
      * @return array
      */
-    public function getAll($limit = null, $offset = null)
+    public function getAll($limit = 50, $offset = 0)
     {
         return $this->api->request('GET', '/notifications?' . http_build_query([
              'limit' => max(0, min(50, filter_var($limit, FILTER_VALIDATE_INT))),

Original file line number	Diff line number	Diff line change
`@@ -46,12 +46,13 @@ public function getOne($id)`
`46`	`46`	`*`
`47`	`47`	`* Application authentication key and ID must be set.`
`48`	`48`	`*`
`49`		`- * @param int $limit Results offset (results are sorted by ID)`
`50`		`- * @param int $offset How many notifications to return (max 50)`
	`49`	`+ * @param int $limit How many notifications to return (max 50)`
	`50`	`+ * @param int $offset Results offset (results are sorted by ID)`
	`51`	`+ *`
`51`	`52`	`*`
`52`	`53`	`* @return array`
`53`	`54`	`*/`
`54`		`- public function getAll($limit = null, $offset = null)`
	`55`	`+ public function getAll($limit = 50, $offset = 0)`
`55`	`56`	`{`
`56`	`57`	`return $this->api->request('GET', '/notifications?' . http_build_query([`
`57`	`58`	`'limit' => max(0, min(50, filter_var($limit, FILTER_VALIDATE_INT))),`