Forbid empty id for get()

Gauvain Pocentek · Gauvain Pocentek · commit 18415fe34f44 · 2017-01-02T09:14:20.000+01:00
Unless the class explicitly defines it's OK (getRequiresId set to True).
diff --git a/gitlab/objects.py b/gitlab/objects.py
@@ -99,6 +99,8 @@ def get(self, id=None, **kwargs):
         args = self._set_parent_args(**kwargs)
         if not self.obj_cls.canGet:
             raise NotImplementedError
+        if id is None and self.obj_cls.getRequiresId is True:
+            raise ValueError('The id argument must be defined.')
         return self.obj_cls.get(self.gitlab, id, **args)
 
     def list(self, **kwargs):
