diff options
| author | Lunny Xiao <xiaolunwen@gmail.com> | 2017-10-02 00:52:35 +0800 |
|---|---|---|
| committer | Lauris BH <lauris@nix.lv> | 2017-10-01 19:52:35 +0300 |
| commit | a8717e5e3ace4dd226547faae9c3a44616bbf6f0 (patch) | |
| tree | 524dc024dec64131406486556951a53655ad8510 /models/pull.go | |
| parent | 1ad902d5298202d5be14fd5a9c8ed6ce781a23c8 (diff) | |
| download | gitea-a8717e5e3ace4dd226547faae9c3a44616bbf6f0.tar.gz gitea-a8717e5e3ace4dd226547faae9c3a44616bbf6f0.zip | |
Use AfterLoad instead of AfterSet on Structs (#2628)
* use AfterLoad instead of AfterSet on Structs
* fix the comments on AfterLoad
* fix the comments on action AfterLoad
Diffstat (limited to 'models/pull.go')
| -rw-r--r-- | models/pull.go | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/models/pull.go b/models/pull.go index 31e1af7e8c..984a1b3038 100644 --- a/models/pull.go +++ b/models/pull.go @@ -80,17 +80,14 @@ func (pr *PullRequest) BeforeUpdate() { pr.MergedUnix = pr.Merged.Unix() } -// AfterSet is invoked from XORM after setting the value of a field of this object. +// AfterLoad is invoked from XORM after setting the values of all fields of this object. // Note: don't try to get Issue because will end up recursive querying. -func (pr *PullRequest) AfterSet(colName string, _ xorm.Cell) { - switch colName { - case "merged_unix": - if !pr.HasMerged { - return - } - - pr.Merged = time.Unix(pr.MergedUnix, 0).Local() +func (pr *PullRequest) AfterLoad() { + if !pr.HasMerged { + return } + + pr.Merged = time.Unix(pr.MergedUnix, 0).Local() } // Note: don't try to get Issue because will end up recursive querying. |