From 6957e4fb8ef372d95457fada61186ec9dc3b99df Mon Sep 17 00:00:00 2001
From: Jacky <me@jackyu.cn>
Date: Thu, 23 Jan 2025 15:29:33 +0800
Subject: [PATCH] fix: sync config is not enabled #820

---
 .../StdDesign/StdDataDisplay/StdTable.vue     |  6 -----
 app/src/views/config/ConfigList.vue           |  1 -
 app/src/views/site/site_list/SiteList.vue     |  2 --
 app/src/views/stream/StreamList.vue           |  2 --
 internal/site/save.go                         | 22 ++++++++++++-------
 5 files changed, 14 insertions(+), 19 deletions(-)

diff --git a/app/src/components/StdDesign/StdDataDisplay/StdTable.vue b/app/src/components/StdDesign/StdDataDisplay/StdTable.vue
index 64f28a5b..b5a41384 100644
--- a/app/src/components/StdDesign/StdDataDisplay/StdTable.vue
+++ b/app/src/components/StdDesign/StdDataDisplay/StdTable.vue
@@ -493,10 +493,6 @@ const paginationSize = computed(() => {
             >
               {{ $gettext('View') }}
             </AButton>
-            <ADivider
-              v-if="!props.disableModify"
-              type="vertical"
-            />
           </template>
 
           <template v-if="!props.disableModify && !inTrash">
@@ -507,7 +503,6 @@ const paginationSize = computed(() => {
             >
               {{ $gettext('Modify') }}
             </AButton>
-            <ADivider type="vertical" />
           </template>
 
           <slot
@@ -544,7 +539,6 @@ const paginationSize = computed(() => {
                 {{ $gettext('Recover') }}
               </AButton>
             </APopconfirm>
-            <ADivider type="vertical" />
             <APopconfirm
               v-if="inTrash"
               :cancel-text="$gettext('No')"
diff --git a/app/src/views/config/ConfigList.vue b/app/src/views/config/ConfigList.vue
index a3068c78..24944ead 100644
--- a/app/src/views/config/ConfigList.vue
+++ b/app/src/views/config/ConfigList.vue
@@ -157,7 +157,6 @@ const refRename = useTemplateRef('refRename')
         >
           {{ $gettext('Modify') }}
         </AButton>
-        <ADivider type="vertical" />
         <AButton
           type="link"
           size="small"
diff --git a/app/src/views/site/site_list/SiteList.vue b/app/src/views/site/site_list/SiteList.vue
index 31fa876d..b32034fe 100644
--- a/app/src/views/site/site_list/SiteList.vue
+++ b/app/src/views/site/site_list/SiteList.vue
@@ -135,7 +135,6 @@ function handleBatchUpdated() {
         >
           {{ $gettext('Enable') }}
         </AButton>
-        <ADivider type="vertical" />
         <AButton
           type="link"
           size="small"
@@ -143,7 +142,6 @@ function handleBatchUpdated() {
         >
           {{ $gettext('Duplicate') }}
         </AButton>
-        <ADivider type="vertical" />
         <APopconfirm
           :cancel-text="$gettext('No')"
           :ok-text="$gettext('OK')"
diff --git a/app/src/views/stream/StreamList.vue b/app/src/views/stream/StreamList.vue
index dd952e5a..6b71ab32 100644
--- a/app/src/views/stream/StreamList.vue
+++ b/app/src/views/stream/StreamList.vue
@@ -151,7 +151,6 @@ function handleAddStream() {
         >
           {{ $gettext('Enable') }}
         </AButton>
-        <ADivider type="vertical" />
         <AButton
           type="link"
           size="small"
@@ -159,7 +158,6 @@ function handleAddStream() {
         >
           {{ $gettext('Duplicate') }}
         </AButton>
-        <ADivider type="vertical" />
         <APopconfirm
           :cancel-text="$gettext('No')"
           :ok-text="$gettext('OK')"
diff --git a/internal/site/save.go b/internal/site/save.go
index 45522d08..587e699a 100644
--- a/internal/site/save.go
+++ b/internal/site/save.go
@@ -46,10 +46,10 @@ func Save(name string, content string, overwrite bool, siteCategoryId uint64, sy
 	s := query.Site
 	_, err = s.Where(s.Path.Eq(path)).
 		Select(s.SiteCategoryID, s.SyncNodeIDs).
-			Updates(&model.Site{
-				SiteCategoryID: siteCategoryId,
-				SyncNodeIDs:    syncNodeIds,
-			})
+		Updates(&model.Site{
+			SiteCategoryID: siteCategoryId,
+			SyncNodeIDs:    syncNodeIds,
+		})
 	if err != nil {
 		return
 	}
@@ -80,10 +80,10 @@ func syncSave(name string, content string) {
 			client.SetBaseURL(node.URL)
 			resp, err := client.R().
 				SetHeader("X-Node-Secret", node.Token).
-					SetBody(map[string]interface{}{
-						"content":   content,
-						"overwrite": true,
-					}).
+				SetBody(map[string]interface{}{
+					"content":   content,
+					"overwrite": true,
+				}).
 				Post(fmt.Sprintf("/api/sites/%s", name))
 			if err != nil {
 				notification.Error("Save Remote Site Error", err.Error())
@@ -94,6 +94,12 @@ func syncSave(name string, content string) {
 				return
 			}
 			notification.Success("Save Remote Site Success", NewSyncResult(node.Name, name, resp).String())
+
+			// Check if the site is enabled, if so then enable it on the remote node
+			enabledConfigFilePath := nginx.GetConfPath("sites-enabled", name)
+			if helper.FileExists(enabledConfigFilePath) {
+				syncEnable(name)
+			}
 		}()
 	}