diff options
| author | Christian Wenk <christian.wenk@omicronenergy.com> | 2016-04-21 13:37:48 +0200 |
|---|---|---|
| committer | Christian Wenk <christian.wenk@omicronenergy.com> | 2016-04-21 13:37:48 +0200 |
| commit | c261875cf167e6858d052dc983fb0dcb03e3ea40 (patch) | |
| tree | 40aa22f64ca1786f4c1ff2ca665c204400da4434 /gitlab | |
| parent | 61bc24fd341e53718f8b9c06c3ac1bbcd55d2530 (diff) | |
| download | gitlab-c261875cf167e6858d052dc983fb0dcb03e3ea40.tar.gz | |
Remove 'next_url' from kwargs before passing it to the cls constructor.
The 'next_url' argument causes problems in the _construct_url method if it
doesn't belong there. E.g. if you list all projects, change an attribute
of a project and then try to save it, the _construct_url will use the
'next_url' from the list method and the save will fail.
Diffstat (limited to 'gitlab')
| -rw-r--r-- | gitlab/__init__.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gitlab/__init__.py b/gitlab/__init__.py index 6c75195..2f4b464 100644 --- a/gitlab/__init__.py +++ b/gitlab/__init__.py @@ -281,7 +281,7 @@ class Gitlab(object): get_all_results = kwargs.get('all', False) # Remove parameters from kwargs before passing it to constructor - for key in ['all', 'page', 'per_page', 'sudo']: + for key in ['all', 'page', 'per_page', 'sudo', 'next_url']: if key in cls_kwargs: del cls_kwargs[key] @@ -385,7 +385,7 @@ class Gitlab(object): get_all_results = params.get('all', False) # Remove parameters from kwargs before passing it to constructor - for key in ['all', 'page', 'per_page', 'sudo']: + for key in ['all', 'page', 'per_page', 'sudo', 'next_url']: if key in cls_kwargs: del cls_kwargs[key] |