diff --git a/app/admin/apis/sys_role.go b/app/admin/apis/sys_role.go
index a189dd36..e5be2e17 100644
--- a/app/admin/apis/sys_role.go
+++ b/app/admin/apis/sys_role.go
@@ -267,12 +267,12 @@ func (e SysRole) Update2DataScope(c *gin.Context) {
 		e.Error(500, err, err.Error())
 		return
 	}
-	//data := &models.SysRole{
-	//	RoleId:    req.RoleId,
-	//	DataScope: req.DataScope,
-	//	DeptIds:   req.DeptIds,
-	//}
-	//data.UpdateBy = user.GetUserId(c)
+	data := &models.SysRole{
+		RoleId:    req.RoleId,
+		DataScope: req.DataScope,
+		DeptIds:   req.DeptIds,
+	}
+	data.UpdateBy = user.GetUserId(c)
 	err = s.UpdateDataScope(&req).Error
 	if err != nil {
 		e.Error(http.StatusInternalServerError, err, "")
diff --git a/app/admin/service/dto/sys_menu.go b/app/admin/service/dto/sys_menu.go
index 85169493..7447d747 100644
--- a/app/admin/service/dto/sys_menu.go
+++ b/app/admin/service/dto/sys_menu.go
@@ -61,7 +61,6 @@ func (s *SysMenuControl) Generate(model *models.SysMenu) {
 	model.Paths = s.Paths
 	model.MenuType = s.MenuType
 	model.Action = s.Action
-	model.Apis = s.Apis
 	model.SysApi = s.SysApi
 	model.Permission = s.Permission
 	model.ParentId = s.ParentId
diff --git a/app/admin/service/sys_menu.go b/app/admin/service/sys_menu.go
index 7949f749..5fb48ed9 100644
--- a/app/admin/service/sys_menu.go
+++ b/app/admin/service/sys_menu.go
@@ -87,9 +87,6 @@ func (e *SysMenu) Insert(c *dto.SysMenuControl) *SysMenu {
 	var err error
 	var data models.SysMenu
 	c.Generate(&data)
-	sysApi := make([]models.SysApi, 0)
-	e.Orm.Where("id in ?", c.Apis).Find(&sysApi)
-	data.SysApi = sysApi
 	err = e.Orm.Create(&data).Error
 	if err != nil {
 		e.Log.Errorf("db error:%s", err)
diff --git a/app/admin/service/sys_role.go b/app/admin/service/sys_role.go
index 9bcb69eb..5e40f73f 100644
--- a/app/admin/service/sys_role.go
+++ b/app/admin/service/sys_role.go
@@ -204,10 +204,8 @@ func (e *SysRole) UpdateDataScope(c *dto.RoleDataScopeReq) *SysRole {
 	}()
 	var dlist = make([]models.SysDept, 0)
 	var model = models.SysRole{}
-	tx.Preload("SysDept").First(&model, c.RoleId)
-	if c.DataScope == "2" {
-		tx.Where("dept_id in ?", c.DeptIds).Find(&dlist)
-	}
+	tx.Preload("SysDept").First(&model, c.DeptIds)
+	tx.Where("id in ?", c.DeptIds).Find(&dlist)
 	err = tx.Model(&model).Association("SysDept").Delete(model.SysDept)
 	if err != nil {
 		e.Log.Errorf("delete SysDept error:%s", err)
@@ -292,4 +290,4 @@ func (e *SysRole) GetById(roleId int) ([]string, error) {
 		permissions = append(permissions, l[i].Permission)
 	}
 	return permissions, nil
-}
+}
\ No newline at end of file
diff --git a/app/jobs/service/sys_job.go b/app/jobs/service/sys_job.go
index 4ed75ba9..4e566d00 100644
--- a/app/jobs/service/sys_job.go
+++ b/app/jobs/service/sys_job.go
@@ -78,7 +78,7 @@ func (e *SysJob) StartJob(c *dto.GeneralGetDto) error {
 		return err
 	}
 
-	err = e.Orm.Table(data.TableName()).Where("job_id = ?",c.Id).Updates(&data).Error
+	err = e.Orm.Table(data.TableName()).Where(c.Id).Updates(&data).Error
 	if err != nil {
 		e.Log.Errorf("db error: %s", err)
 	}
diff --git a/common/ip.go b/common/ip.go
index e06b9bb9..7822f259 100644
--- a/common/ip.go
+++ b/common/ip.go
@@ -1,15 +1,16 @@
 package common
 
 import (
+	"fmt"
 	"github.com/gin-gonic/gin"
 	"strings"
 )
 
 func GetClientIP(c *gin.Context) string {
 	ClientIP := c.ClientIP()
-	//fmt.Println("ClientIP:", ClientIP)
+	fmt.Println("ClientIP:", ClientIP)
 	RemoteIP, _ := c.RemoteIP()
-	//fmt.Println("RemoteIP:", RemoteIP)
+	fmt.Println("RemoteIP:", RemoteIP)
 	ip := c.Request.Header.Get("X-Forwarded-For")
 	if strings.Contains(ip, "127.0.0.1") || ip == "" {
 		ip = c.Request.Header.Get("X-real-ip")
@@ -23,6 +24,6 @@ func GetClientIP(c *gin.Context) string {
 	if ClientIP != "127.0.0.1" {
 		ip = ClientIP
 	}
-	//fmt.Println("ip:", ip)
+	fmt.Println("ip:", ip)
 	return ip
 }