Merge pull request #814 from keemozhang/master

fix🐛: declaration of new local variable causes transactions to be ign…

Author: wenjianzhang

Diff for: app/admin/service/sys_dept.go

@@ -84,7 +84,7 @@ func (e *SysDept) Insert(c *dto.SysDeptInsertReq) error {
 	}
 	var mp = map[string]string{}
 	mp["dept_path"] = deptPath
-	if err := tx.Model(&data).Update("dept_path", deptPath).Error; err != nil {
+	if err = tx.Model(&data).Update("dept_path", deptPath).Error; err != nil {
 		e.Log.Errorf("db error:%s", err)
 		return err
 	}

Diff for: app/admin/service/sys_role.go

@@ -75,7 +75,7 @@ func (e *SysRole) Insert(c *dto.SysRoleInsertReq, cb *casbin.SyncedEnforcer) err
 	c.Generate(&data)
 	tx := e.Orm
 	if config.DatabaseConfig.Driver != "sqlite3" {
-		tx := e.Orm.Begin()
+		tx = e.Orm.Begin()
 		defer func() {
 			if err != nil {
 				tx.Rollback()
@@ -132,7 +132,7 @@ func (e *SysRole) Update(c *dto.SysRoleUpdateReq, cb *casbin.SyncedEnforcer) err
 	var err error
 	tx := e.Orm
 	if config.DatabaseConfig.Driver != "sqlite3" {
-		tx := e.Orm.Begin()
+		tx = e.Orm.Begin()
 		defer func() {
 			if err != nil {
 				tx.Rollback()
@@ -197,7 +197,7 @@ func (e *SysRole) Remove(c *dto.SysRoleDeleteReq, cb *casbin.SyncedEnforcer) err
 	var err error
 	tx := e.Orm
 	if config.DatabaseConfig.Driver != "sqlite3" {
-		tx := e.Orm.Begin()
+		tx = e.Orm.Begin()
 		defer func() {
 			if err != nil {
 				tx.Rollback()
@@ -244,7 +244,7 @@ func (e *SysRole) UpdateDataScope(c *dto.RoleDataScopeReq) *SysRole {
 	var err error
 	tx := e.Orm
 	if config.DatabaseConfig.Driver != "sqlite3" {
-		tx := e.Orm.Begin()
+		tx = e.Orm.Begin()
 		defer func() {
 			if err != nil {
 				tx.Rollback()
@@ -285,7 +285,7 @@ func (e *SysRole) UpdateStatus(c *dto.UpdateStatusReq) error {
 	var err error
 	tx := e.Orm
 	if config.DatabaseConfig.Driver != "sqlite3" {
-		tx := e.Orm.Begin()
+		tx = e.Orm.Begin()
 		defer func() {
 			if err != nil {
 				tx.Rollback()