Commit9c58013

Gauvain Pocentek

committed

Fix GET/POST for project files

1 parent3270865 commit9c58013Copy full SHA for 9c58013

File tree

-5

lines changed

-5

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -675,6 +675,8 @@ class GitlabObject(object):`
`675`	`675`	`requiredUpdateAttrs=None`
`676`	`676`	`#: Attributes that are optional when updating an object`
`677`	`677`	`optionalUpdateAttrs=None`
	`678`	`+#: Whether the object ID is required in the GET url`
	`679`	`+getRequiresId=True`
`678`	`680`
`679`	`681`	`idAttr='id'`
`680`	`682`	`shortPrintAttr=None`
`@@ -1134,7 +1136,8 @@ class ProjectFile(GitlabObject):`
`1134`	`1136`	`optionalCreateAttrs= ['encoding']`
`1135`	`1137`	`requiredDeleteAttrs= ['branch_name','commit_message']`
`1136`	`1138`	`getListWhenNoId=False`
`1137`		`-shortPrintAttr='name'`
	`1139`	`+shortPrintAttr='file_path'`
	`1140`	`+getRequiresId=False`
`1138`	`1141`
`1139`	`1142`
`1140`	`1143`	`classProjectSnippetNote(GitlabObject):`

Lines changed: 5 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -82,9 +82,10 @@ def populate_sub_parser_by_class(cls, sub_parser):`
`82`	`82`
`83`	`83`	`elifaction_namein [GET,DELETE]:`
`84`	`84`	`ifclsnotin [gitlab.CurrentUser]:`
`85`		`-id_attr=cls.idAttr.replace('_','-')`
`86`		`-sub_parser_action.add_argument("--%s"%id_attr,`
`87`		`-required=True)`
	`85`	`+ifcls.getRequiresId:`
	`86`	`+id_attr=cls.idAttr.replace('_','-')`
	`87`	`+sub_parser_action.add_argument("--%s"%id_attr,`
	`88`	`+required=True)`
`88`	`89`	`[sub_parser_action.add_argument("--%s"%x.replace('_','-'),`
`89`	`90`	`required=True)`
`90`	`91`	`forxincls.requiredGetAttrs]`
`@@ -172,7 +173,7 @@ def do_get(cls, gl, what, args):`
`172`	`173`	`die("%s objects can't be retrieved"%what)`
`173`	`174`
`174`	`175`	`id=None`
`175`		`-ifclsnotin [gitlab.CurrentUser]:`
	`176`	`+ifclsnotin [gitlab.CurrentUser]andcls.getRequiresId:`
`176`	`177`	`id=get_id(cls,args)`
`177`	`178`
`178`	`179`	`try:`

Comments

(0)