refactor🎨:注释未使用的对象

This commit is contained in:
zhangwenjian
2021-06-17 11:28:37 +08:00
parent d32751ea5d
commit 3358eeb54c
6 changed files with 19 additions and 14 deletions
+6 -6
View File
@@ -267,12 +267,12 @@ func (e SysRole) Update2DataScope(c *gin.Context) {
e.Error(500, err, err.Error()) e.Error(500, err, err.Error())
return return
} }
data := &models.SysRole{ //data := &models.SysRole{
RoleId: req.RoleId, // RoleId: req.RoleId,
DataScope: req.DataScope, // DataScope: req.DataScope,
DeptIds: req.DeptIds, // DeptIds: req.DeptIds,
} //}
data.UpdateBy = user.GetUserId(c) //data.UpdateBy = user.GetUserId(c)
err = s.UpdateDataScope(&req).Error err = s.UpdateDataScope(&req).Error
if err != nil { if err != nil {
e.Error(http.StatusInternalServerError, err, "") e.Error(http.StatusInternalServerError, err, "")
+1
View File
@@ -61,6 +61,7 @@ func (s *SysMenuControl) Generate(model *models.SysMenu) {
model.Paths = s.Paths model.Paths = s.Paths
model.MenuType = s.MenuType model.MenuType = s.MenuType
model.Action = s.Action model.Action = s.Action
model.Apis = s.Apis
model.SysApi = s.SysApi model.SysApi = s.SysApi
model.Permission = s.Permission model.Permission = s.Permission
model.ParentId = s.ParentId model.ParentId = s.ParentId
+3
View File
@@ -87,6 +87,9 @@ func (e *SysMenu) Insert(c *dto.SysMenuControl) *SysMenu {
var err error var err error
var data models.SysMenu var data models.SysMenu
c.Generate(&data) 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 err = e.Orm.Create(&data).Error
if err != nil { if err != nil {
e.Log.Errorf("db error:%s", err) e.Log.Errorf("db error:%s", err)
+5 -3
View File
@@ -204,8 +204,10 @@ func (e *SysRole) UpdateDataScope(c *dto.RoleDataScopeReq) *SysRole {
}() }()
var dlist = make([]models.SysDept, 0) var dlist = make([]models.SysDept, 0)
var model = models.SysRole{} var model = models.SysRole{}
tx.Preload("SysDept").First(&model, c.DeptIds) tx.Preload("SysDept").First(&model, c.RoleId)
tx.Where("id in ?", c.DeptIds).Find(&dlist) if c.DataScope == "2" {
tx.Where("dept_id in ?", c.DeptIds).Find(&dlist)
}
err = tx.Model(&model).Association("SysDept").Delete(model.SysDept) err = tx.Model(&model).Association("SysDept").Delete(model.SysDept)
if err != nil { if err != nil {
e.Log.Errorf("delete SysDept error:%s", err) e.Log.Errorf("delete SysDept error:%s", err)
@@ -290,4 +292,4 @@ func (e *SysRole) GetById(roleId int) ([]string, error) {
permissions = append(permissions, l[i].Permission) permissions = append(permissions, l[i].Permission)
} }
return permissions, nil return permissions, nil
} }
+1 -1
View File
@@ -78,7 +78,7 @@ func (e *SysJob) StartJob(c *dto.GeneralGetDto) error {
return err return err
} }
err = e.Orm.Table(data.TableName()).Where(c.Id).Updates(&data).Error err = e.Orm.Table(data.TableName()).Where("job_id = ?",c.Id).Updates(&data).Error
if err != nil { if err != nil {
e.Log.Errorf("db error: %s", err) e.Log.Errorf("db error: %s", err)
} }
+3 -4
View File
@@ -1,16 +1,15 @@
package common package common
import ( import (
"fmt"
"github.com/gin-gonic/gin" "github.com/gin-gonic/gin"
"strings" "strings"
) )
func GetClientIP(c *gin.Context) string { func GetClientIP(c *gin.Context) string {
ClientIP := c.ClientIP() ClientIP := c.ClientIP()
fmt.Println("ClientIP:", ClientIP) //fmt.Println("ClientIP:", ClientIP)
RemoteIP, _ := c.RemoteIP() RemoteIP, _ := c.RemoteIP()
fmt.Println("RemoteIP:", RemoteIP) //fmt.Println("RemoteIP:", RemoteIP)
ip := c.Request.Header.Get("X-Forwarded-For") ip := c.Request.Header.Get("X-Forwarded-For")
if strings.Contains(ip, "127.0.0.1") || ip == "" { if strings.Contains(ip, "127.0.0.1") || ip == "" {
ip = c.Request.Header.Get("X-real-ip") ip = c.Request.Header.Get("X-real-ip")
@@ -24,6 +23,6 @@ func GetClientIP(c *gin.Context) string {
if ClientIP != "127.0.0.1" { if ClientIP != "127.0.0.1" {
ip = ClientIP ip = ClientIP
} }
fmt.Println("ip:", ip) //fmt.Println("ip:", ip)
return ip return ip
} }