Use OptionsResolver in Groups::projects

Author: fbourigault

UPGRADE.md

@@ -24,6 +24,7 @@ See [documentation](doc/customize.md) to know how to customize the client timeou
 * The `all` method now take a single argument which is an associative array of query string parameters.
 * The `search` method have been removed. Use `all` method instead.
 * The `members` method second and subsequent arguments have been replaced by a single associative array of query string parameters.
+* The `projects` method second and subsequent arguments have been replaced by a single associative array of query string parameters.
 
 ## `Gitlab\Api\Issues` changes
 

lib/Gitlab/Api/Groups.php

@@ -162,15 +162,55 @@ public function removeMember($group_id, $user_id)
 
     /**
      * @param $id
-     * @param int $page
-     * @param int $per_page
+     * @param array $parameters (
+     *
+     *     @var bool   $archived   Limit by archived status.
+     *     @var string $visibility Limit by visibility public, internal, or private.
+     *     @var string $order_by   Return projects ordered by id, name, path, created_at, updated_at, or last_activity_at fields.
+     *                             Default is created_at.
+     *     @var string $sort       Return projects sorted in asc or desc order. Default is desc.
+     *     @var string $search     Return list of authorized projects matching the search criteria.
+     *     @var bool   $simple     Return only the ID, URL, name, and path of each project.
+     *     @var bool   $owned      Limit by projects owned by the current user.
+     *     @var bool   $starred    Limit by projects starred by the current user.
+     * )
+     *
      * @return mixed
      */
-    public function projects($id, $page = 1, $per_page = self::PER_PAGE)
+    public function projects($id, array $parameters = [])
     {
-        return $this->get('groups/'.$this->encodePath($id).'/projects', array(
-            'page' => $page,
-            'per_page' => $per_page
-        ));
+        $resolver = $this->createOptionsResolver();
+        $booleanNormalizer = function ($value) {
+            return $value ? 'true' : 'false';
+        };
+
+        $resolver->setDefined('archived')
+            ->setAllowedTypes('archived', 'bool')
+            ->setNormalizer('archived', $booleanNormalizer)
+        ;
+        $resolver->setDefined('visibility')
+            ->setAllowedValues('visibility', ['public', 'internal', 'private'])
+        ;
+        $resolver->setDefined('order_by')
+            ->setAllowedValues('order_by', ['id', 'name', 'path', 'created_at', 'updated_at', 'last_activity_at'])
+        ;
+        $resolver->setDefined('sort')
+            ->setAllowedValues('sort', ['asc', 'desc'])
+        ;
+        $resolver->setDefined('search');
+        $resolver->setDefined('simple')
+            ->setAllowedTypes('simple', 'bool')
+            ->setNormalizer('simple', $booleanNormalizer)
+        ;
+        $resolver->setDefined('owned')
+            ->setAllowedTypes('owned', 'bool')
+            ->setNormalizer('owned', $booleanNormalizer)
+        ;
+        $resolver->setDefined('starred')
+            ->setAllowedTypes('starred', 'bool')
+            ->setNormalizer('starred', $booleanNormalizer)
+        ;
+
+        return $this->get('groups/'.$this->encodePath($id).'/projects', $resolver->resolve($parameters));
     }
 }