Add CreatedOnTime and UpdatedOnTime To Repository (#236)

* feat(repository): Add CreatedOnTime and UpdatedOnTime fiields with *time.time types

* docs: Remove commented code

---------

Co-authored-by: Thomas O'Neill <[email protected]>

Author: toneill818

repositories.go

@@ -3,8 +3,6 @@ package bitbucket
 import (
 	"errors"
 	"fmt"
-
-	"github.com/mitchellh/mapstructure"
 )
 
 //"github.com/k0kubun/pp"
@@ -70,10 +68,9 @@ func decodeRepositories(reposResponse interface{}) (*RepositoriesRes, error) {
 	repoArray := reposResponseMap["values"].([]interface{})
 	var repos []Repository
 	for _, repoEntry := range repoArray {
-		var repo Repository
-		err := mapstructure.Decode(repoEntry, &repo)
+		repo, err := decodeRepository(repoEntry)
 		if err == nil {
-			repos = append(repos, repo)
+			repos = append(repos, *repo)
 		}
 	}
 

repository.go

@@ -9,6 +9,7 @@ import (
 	"path"
 	"strconv"
 	"strings"
+	"time"
 
 	"github.com/mitchellh/mapstructure"
 )
@@ -29,11 +30,15 @@ type Repository struct {
 	Has_wiki    bool
 	Mainbranch  RepositoryBranch
 	Type        string
-	CreatedOn   string `mapstructure:"created_on"`
-	UpdatedOn   string `mapstructure:"updated_on"`
-	Owner       map[string]interface{}
-	Links       map[string]interface{}
-	Parent      *Repository
+	// Deprecated: CreatedOn is deprecated use CreatedOnTime
+	CreatedOn string `mapstructure:"created_on"`
+	// Deprecated: UpdatedOn is deprecated use UpdatedOnTime
+	UpdatedOn     string `mapstructure:"updated_on"`
+	Owner         map[string]interface{}
+	Links         map[string]interface{}
+	Parent        *Repository
+	CreatedOnTime *time.Time `mapstructure:"created_on"`
+	UpdatedOnTime *time.Time `mapstructure:"updated_on"`
 }
 
 type RepositoryFile struct {
@@ -249,6 +254,8 @@ type UserPermissions struct {
 	UserPermissions []UserPermission
 }
 
+var stringToTimeHookFunc = mapstructure.StringToTimeHookFunc("2006-01-02T15:04:05.000000+00:00")
+
 func (r *Repository) Create(ro *RepositoryOptions) (*Repository, error) {
 	data, err := r.buildRepositoryBody(ro)
 	if err != nil {
@@ -1190,7 +1197,15 @@ func decodeRepository(repoResponse interface{}) (*Repository, error) {
 	}
 
 	var repository = new(Repository)
-	err := mapstructure.Decode(repoMap, repository)
+	decoder, err := mapstructure.NewDecoder(&mapstructure.DecoderConfig{
+		Metadata:   nil,
+		Result:     repository,
+		DecodeHook: stringToTimeHookFunc,
+	})
+	if err != nil {
+		return nil, err
+	}
+	err = decoder.Decode(repoMap)
 	if err != nil {
 		return nil, err
 	}