Skip to content

Commit

Permalink
Make the delete() method handle / in ids
Browse files Browse the repository at this point in the history
Replace the / with the HTTP %2F as is done with other methods.

Closes #337
  • Loading branch information
Gauvain Pocentek committed Oct 8, 2017
1 parent ac430a3 commit 8764903
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions gitlab/mixins.py
Expand Up @@ -235,6 +235,9 @@ def delete(self, id, **kwargs):
GitlabAuthenticationError: If authentication is not correct
GitlabDeleteError: If the server cannot perform the request
"""
import pdb; pdb.set_trace()
if not isinstance(id, int):
id = id.replace('/', '%2F')
path = '%s/%s' % (self.path, id)
self.gitlab.http_delete(path, **kwargs)

Expand Down

0 comments on commit 8764903

Please sign in to comment.