Skip to content
This repository was archived by the owner on May 24, 2024. It is now read-only.

Commit 5c9eb15

Browse files
authored
Merge pull request #25 from LF-Engineering/fix-profile-by-username-bug
Fix profile by username bug
2 parents ca8c5e3 + 8dd3158 commit 5c9eb15

File tree

1 file changed

+19
-2
lines changed

1 file changed

+19
-2
lines changed

affiliation/identity.go

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ import (
1313
"github.com/LF-Engineering/dev-analytics-libraries/http"
1414
)
1515

16+
var unknown string = "Unknown"
17+
var genderAcc int64 = 0
18+
1619
// Affiliations interface
1720
type Affiliations interface {
1821
AddIdentity(identity *Identity) bool
@@ -309,18 +312,32 @@ func (a *Affiliation) GetProfileByUsername(username string, projectSlug string)
309312

310313
if profileIdentity.Name != nil {
311314
identity.Name = *profileIdentity.Name
315+
} else {
316+
identity.Name = unknown
312317
}
313318

314319
identity.Username = username
315320

316321
if profileIdentity.Email != nil {
317322
identity.Email = *profileIdentity.Email
318323
}
324+
319325
identity.ID = &profileIdentity.ID
320326

321327
identity.IsBot = profile.Profile.IsBot
322-
identity.Gender = profile.Profile.Gender
323-
identity.GenderACC = profile.Profile.GenderAcc
328+
329+
if profile.Profile.Gender == nil {
330+
identity.Gender = profile.Profile.Gender
331+
identity.GenderACC = profile.Profile.GenderAcc
332+
} else {
333+
identity.Gender = &unknown
334+
identity.GenderACC = &genderAcc
335+
}
336+
337+
if profile.Enrollments == nil {
338+
identity.OrgName = &unknown
339+
identity.MultiOrgNames = make([]string, 0)
340+
}
324341

325342
if len(profile.Enrollments) > 1 {
326343
identity.OrgName = a.getUserOrg(profile.Enrollments)

0 commit comments

Comments
 (0)