From 4d3e2b23b86f5be8eaf649980ed3fa81b5997e3b Mon Sep 17 00:00:00 2001
From: Yarden Shoham <hrsi88@gmail.com>
Date: Fri, 24 Feb 2023 08:36:25 +0200
Subject: [PATCH] Fix SyncOnCommit always return false in API of push_mirrors
 (#23088) (#23100)

Backport #23088

Fix: #22990

---
Before, the return value of the api is always false,regrardless of
whether the entry of `sync_on_commit` is true or false.
I have confirmed that the value of `sync_on_commit` dropped into the
database is correct.
So, I think it is enough to make some small changes.

Co-authored-by: sillyguodong <33891828+sillyguodong@users.noreply.github.com>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
---
 services/convert/mirror.go | 1 +
 1 file changed, 1 insertion(+)

diff --git a/services/convert/mirror.go b/services/convert/mirror.go
index 1dcfc9b64dc..f7a8e17fd0c 100644
--- a/services/convert/mirror.go
+++ b/services/convert/mirror.go
@@ -24,6 +24,7 @@ func ToPushMirror(pm *repo_model.PushMirror) (*api.PushMirror, error) {
 		LastUpdateUnix: pm.LastUpdateUnix.FormatLong(),
 		LastError:      pm.LastError,
 		Interval:       pm.Interval.String(),
+		SyncOnCommit:   pm.SyncOnCommit,
 	}, nil
 }