apply suggestions from code review

Author: a1012112796

models/package.go

@@ -85,12 +85,18 @@ func getPackage(e Engine, repoID int64, typ PackageType, name string) (*Package,
 	return pkg, nil
 }
 
-// UpdateCols updates some columns
-func (pkg *Package) UpdateCols(cols ...string) error {
+// updateCols updates some columns
+func (pkg *Package) updateCols(cols ...string) error {
 	_, err := x.ID(pkg.ID).Cols(cols...).Update(pkg)
 	return err
 }
 
+// UpdateLastUpdated update last update time
+func (pkg *Package) UpdateLastUpdated(updateTime timeutil.TimeStamp) error {
+	pkg.UpdatedUnix = updateTime
+	return pkg.updateCols("updated_unix")
+}
+
 // LoadRepo load package's repo
 func (pkg *Package) LoadRepo(cols ...string) (err error) {
 	if pkg.Repo != nil {

modules/packages/docker/event_hook.go

@@ -68,8 +68,7 @@ func HandleEvents(data *notifications.Envelope) error {
 			}
 
 			// update update time
-			pkg.UpdatedUnix = timeutil.TimeStamp(event.Timestamp.Unix())
-			err = pkg.UpdateCols("updated_unix")
+			err = pkg.UpdateLastUpdated(timeutil.TimeStamp(event.Timestamp.Unix()))
 			if err != nil {
 				return err
 			}