status-im/PyGithub
1
0
Fork 0
mirror of https://github.com/status-im/PyGithub.git synced 2026-09-01 11:21:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Allow None as a valid value for an optional parameter (fix testEditOrganization)

Browse Source
This commit is contained in:
Vincent Jacques
2012-05-04 14:57:46 +02:00
parent fa9e5df05a
commit 5ae4dd1233
34 changed files with 286 additions and 131 deletions
Download Patch File Download Diff File Expand all files Collapse all files
github/GithubObjects/PullRequest.py
+10 -5
View File
@@ -5,6 +5,11 @@ import Commit
import NamedUser
import PullRequestComment
import PullRequestFile
# This allows None as a valid value for an optional parameter
class DefaultValueForOptionalParametersType:
pass
DefaultValueForOptionalParameters = DefaultValueForOptionalParametersType()
class PullRequest( object ):
def __init__( self, requester, attributes, lazy ):
@@ -145,14 +150,14 @@ class PullRequest( object ):
self.__completeIfNeeded( self.__user )
return self.__user
def edit( self, title = None, body = None, state = None ):
def edit( self, title = DefaultValueForOptionalParameters, body = DefaultValueForOptionalParameters, state = DefaultValueForOptionalParameters ):
post_parameters = {
}
if title is not None:
if title is not DefaultValueForOptionalParameters:
post_parameters[ "title" ] = title
if body is not None:
if body is not DefaultValueForOptionalParameters:
post_parameters[ "body" ] = body
if state is not None:
if state is not DefaultValueForOptionalParameters:
post_parameters[ "state" ] = state
status, headers, data = self.__requester.request(
"PATCH",
@@ -210,7 +215,7 @@ class PullRequest( object ):
def is_merged( self ):
pass
def merge( self, commit_message = None ):
def merge( self, commit_message = DefaultValueForOptionalParameters ):
pass
def __initAttributes( self ):