Fixing bug

diff --git a/models/action.go b/models/action.go
index 107d4b1057fde651a4f99141f2b1faff238e5d0f..89751b9779139989f0f4459864a00610c43a8646 100644 (file)
--- a/models/action.go
+++ b/models/action.go
@@ -79,6 +79,16 @@ func CommitRepoAction(userId int64, userName string,
                })
                return err
        }
+
+       // Update repository last update time.
+       repo, err := GetRepositoryByName(userId, repoName)
+       if err != nil {
+               return err
+       }
+       repo.Updated = time.Now()
+       if err = UpdateRepository(repo); err != nil {
+               return err
+       }
        return nil
 }
 

diff --git a/models/repo.go b/models/repo.go
index 918e5dc84ca195f225fcff723e75bd1290662f09..6a764e6c318885f3dcad3d125ed871570562627a 100644 (file)
--- a/models/repo.go
+++ b/models/repo.go
@@ -358,6 +358,11 @@ func RepoPath(userName, repoName string) string {
        return filepath.Join(UserPath(userName), repoName+".git")
 }
 
+func UpdateRepository(repo *Repository) error {
+       _, err := orm.Id(repo.Id).UseBool().Update(repo)
+       return err
+}
+
 // DeleteRepository deletes a repository for a user or orgnaztion.
 func DeleteRepository(userId, repoId int64, userName string) (err error) {
        repo := &Repository{Id: repoId, OwnerId: userId}
@@ -402,9 +407,9 @@ func DeleteRepository(userId, repoId int64, userName string) (err error) {
 }
 
 // GetRepositoryByName returns the repository by given name under user if exists.
-func GetRepositoryByName(user *User, repoName string) (*Repository, error) {
+func GetRepositoryByName(userId int64, repoName string) (*Repository, error) {
        repo := &Repository{
-               OwnerId:   user.Id,
+               OwnerId:   userId,
                LowerName: strings.ToLower(repoName),
        }
        has, err := orm.Get(repo)

diff --git a/models/user.go b/models/user.go
index 3c1109128587b7fc6c601a111581ddc936e1331b..d6dc04149073b8734b982b20622dec7e7bae1620 100644 (file)
--- a/models/user.go
+++ b/models/user.go
@@ -279,9 +279,7 @@ func GetUserByName(name string) (*User, error) {
        if len(name) == 0 {
                return nil, ErrUserNotExist
        }
-       user := &User{
-               LowerName: strings.ToLower(name),
-       }
+       user := &User{LowerName: strings.ToLower(name)}
        has, err := orm.Get(user)
        if err != nil {
                return nil, err

diff --git a/modules/middleware/repo.go b/modules/middleware/repo.go
index a9a90e3ff570ce77a0603cf80bc0c3b7bd69c400..3864caaf8000a54ce23e44b9887f6d427d2648e6 100644 (file)
--- a/modules/middleware/repo.go
+++ b/modules/middleware/repo.go
@@ -54,7 +54,7 @@ func RepoAssignment(redirect bool) martini.Handler {
                ctx.Repo.Owner = user
 
                // get repository
-               repo, err := models.GetRepositoryByName(user, params["reponame"])
+               repo, err := models.GetRepositoryByName(user.Id, params["reponame"])
                if err != nil {
                        if redirect {
                                ctx.Redirect("/")

diff --git a/serve.go b/serve.go
index 3ce8f9046cb0e4b34a24810bdbecd78f89b94dc0..be8dedc98557182fc8a457b14ef5dd64a8bdbab5 100644 (file)
--- a/serve.go
+++ b/serve.go
@@ -86,7 +86,7 @@ func runServ(*cli.Context) {
 
        os.Setenv("userName", user.Name)
        os.Setenv("userId", strconv.Itoa(int(user.Id)))
-       repo, err := models.GetRepositoryByName(user, repoName)
+       repo, err := models.GetRepositoryByName(user.Id, repoName)
        if err != nil {
                println("Unavilable repository", err)
                return