sjs/retrogit
1
0
Fork 0
mirror of https://github.com/samsonjs/retrogit.git synced 2026-03-25 09:25:49 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Unify Account initialization code paths.

Browse source 
We were not initializing the timezone when loading all accounts, which was
making the cron job fail.
This commit is contained in:
Mihai Parparita 2014-08-05 22:06:05 -07:00
parent faf41a425d
commit c4e55c5553
1 changed files with 12 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
app/account.go
View file

@ -29,21 +29,27 @@ func getAccount(c appengine.Context, gitHubUserId int) (*Account, error) {
return nil, err
}
r := bytes.NewBuffer(account.OAuthTokenSerialized)
err = gob.NewDecoder(r).Decode(&account.OAuthToken)
err = initAccount(account)
if err != nil {
return nil, err
}
return account, nil
}
func initAccount(account *Account) error {
r := bytes.NewBuffer(account.OAuthTokenSerialized)
err := gob.NewDecoder(r).Decode(&account.OAuthToken)
if err != nil {
return err
}
if len(account.TimezoneName) == 0 {
account.TimezoneName = "America/Los_Angeles"
}
account.TimezoneLocation, err = time.LoadLocation(account.TimezoneName)
if err != nil {
return nil, err
return err
}
return account, nil
return nil
}
func getAllAccounts(c appengine.Context, accounts *[]Account) error {
@ -53,8 +59,7 @@ func getAllAccounts(c appengine.Context, accounts *[]Account) error {
return err
}
for i, _ := range *accounts {
r := bytes.NewBuffer((*accounts)[i].OAuthTokenSerialized)
err = gob.NewDecoder(r).Decode(&(*accounts)[i].OAuthToken)
err = initAccount(&(*accounts)[i])
if err != nil {
return err
}