Skip to content

Commit

Permalink
GitHub:9479 ([Google] User information: fixes)
Browse files Browse the repository at this point in the history
  • Loading branch information
yannicktrinh committed Sep 16, 2024
1 parent 65fd5c5 commit 8d5cc31
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 9 deletions.
13 changes: 7 additions & 6 deletions Project/Sources/Classes/GoogleUser.4dm
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
Class extends _GoogleAPI

property _internals : Object:={_defaultPersonFields: "names,emailAddresses,photos"}

Class constructor($inProvider : cs.OAuth2Provider)

Super($inProvider;"https://people.googleapis.com/v1/")

Super($inProvider)
This._internals._defaultPersonFields:="names,emailAddresses,photos"


// ----------------------------------------------------
Expand All @@ -14,7 +15,7 @@ Class constructor($inProvider : cs.OAuth2Provider)
Function _get($inResourceName : Text; $inPersonFields : Variant) : Object

Super._clearErrorStack()

var $URL : Text:=Super._getURL()
var $resourceName : Text:=Length(String($inResourceName))>0 ? String($inResourceName) : "me"
var $personFields : Text
Expand All @@ -32,7 +33,7 @@ Function _get($inResourceName : Text; $inPersonFields : Variant) : Object

var $headers : Object:={}
$headers["Content-Type"]:="application/json"
var $response : Object:=Super._sendRequestAndWaitResponse("GET"; $inURL; $headers)
var $response : Object:=Super._sendRequestAndWaitResponse("GET"; $URL; $headers)

return $response

Expand Down Expand Up @@ -61,7 +62,7 @@ Function get($inResourceName : Text; $inPersonFields : Variant) : Object
Function list($inParameter : Object) : Object

Super._clearErrorStack()

var $URL : Text:=Super._getURL()
var $personFields : Text
var $sources : Text
Expand Down Expand Up @@ -96,5 +97,5 @@ Function list($inParameter : Object) : Object
If (Type($inParameter.requestSyncToken)=Is boolean)
$URL+="&requestSyncToken="+$inParameter.requestSyncToken ? "true" : "false"
End if

return cs.GoogleUserList.new(This._getOAuth2Provider(); $URL)
4 changes: 3 additions & 1 deletion Project/Sources/Classes/OAuth2Provider.4dm
Original file line number Diff line number Diff line change
Expand Up @@ -625,7 +625,9 @@ Function _checkPrerequisites($obj : Object) : Boolean
: (Length(String($obj.clientId))=0)
This._throwError(2; {attribute: "clientId"})

: ((Length(String($obj.name))>0) && (Length(String($obj.scope))=0))
: ((Length(String($obj.name))>0) && \
((Value type($obj.scope)=Is text) && (Length(String($obj.scope))=0)) || \
((Value type($obj.scope)=Is collection) && ($obj.scope.length=0)))
This._throwError(2; {attribute: "scope"})

: (Length(String($obj.permission))=0)
Expand Down
4 changes: 2 additions & 2 deletions Project/Sources/Classes/_GoogleAPI.4dm
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
Class extends _BaseAPI

Class constructor($inProvider : cs.OAuth2Provider)
Class constructor($inProvider : cs.OAuth2Provider; $inBaseURL : Text)

Super($inProvider)

This._internals._URL:="https://gmail.googleapis.com/gmail/v1/"
This._internals._URL:=(Length(String($inBaseURL)) > 0) ? $inBaseURL : "https://gmail.googleapis.com/gmail/v1/"


// ----------------------------------------------------
Expand Down

0 comments on commit 8d5cc31

Please sign in to comment.