From 516a097e1ad588d860b374ef1d0eff202adb286b Mon Sep 17 00:00:00 2001 From: George Goldberg Date: Fri, 1 Jun 2018 12:49:02 +0100 Subject: MM-10757: Default roles from scheme should be keyed by name not ID. (#8894) * MM-10757: Default roles from scheme should be keyed by name not ID. * Update permissions import/export. --- store/sqlstore/scheme_supplier.go | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'store/sqlstore/scheme_supplier.go') diff --git a/store/sqlstore/scheme_supplier.go b/store/sqlstore/scheme_supplier.go index f272040a6..326fb45e6 100644 --- a/store/sqlstore/scheme_supplier.go +++ b/store/sqlstore/scheme_supplier.go @@ -109,7 +109,7 @@ func (s *SqlSupplier) createScheme(ctx context.Context, scheme *model.Scheme, tr result.Err = saveRoleResult.Err return result } else { - scheme.DefaultTeamAdminRole = saveRoleResult.Data.(*model.Role).Id + scheme.DefaultTeamAdminRole = saveRoleResult.Data.(*model.Role).Name } // Team User Role @@ -124,7 +124,7 @@ func (s *SqlSupplier) createScheme(ctx context.Context, scheme *model.Scheme, tr result.Err = saveRoleResult.Err return result } else { - scheme.DefaultTeamUserRole = saveRoleResult.Data.(*model.Role).Id + scheme.DefaultTeamUserRole = saveRoleResult.Data.(*model.Role).Name } } if scheme.Scope == model.SCHEME_SCOPE_TEAM || scheme.Scope == model.SCHEME_SCOPE_CHANNEL { @@ -140,7 +140,7 @@ func (s *SqlSupplier) createScheme(ctx context.Context, scheme *model.Scheme, tr result.Err = saveRoleResult.Err return result } else { - scheme.DefaultChannelAdminRole = saveRoleResult.Data.(*model.Role).Id + scheme.DefaultChannelAdminRole = saveRoleResult.Data.(*model.Role).Name } // Channel User Role @@ -155,7 +155,7 @@ func (s *SqlSupplier) createScheme(ctx context.Context, scheme *model.Scheme, tr result.Err = saveRoleResult.Err return result } else { - scheme.DefaultChannelUserRole = saveRoleResult.Data.(*model.Role).Id + scheme.DefaultChannelUserRole = saveRoleResult.Data.(*model.Role).Name } } @@ -231,16 +231,16 @@ func (s *SqlSupplier) SchemeDelete(ctx context.Context, schemeId string, hints . } // Delete the roles belonging to the scheme. - roleIds := []string{scheme.DefaultChannelUserRole, scheme.DefaultChannelAdminRole} + roleNames := []string{scheme.DefaultChannelUserRole, scheme.DefaultChannelAdminRole} if scheme.Scope == model.SCHEME_SCOPE_TEAM { - roleIds = append(roleIds, scheme.DefaultTeamUserRole, scheme.DefaultTeamAdminRole) + roleNames = append(roleNames, scheme.DefaultTeamUserRole, scheme.DefaultTeamAdminRole) } var inQueryList []string queryArgs := make(map[string]interface{}) - for i, roleId := range roleIds { - inQueryList = append(inQueryList, fmt.Sprintf(":RoleId%v", i)) - queryArgs[fmt.Sprintf("RoleId%v", i)] = roleId + for i, roleId := range roleNames { + inQueryList = append(inQueryList, fmt.Sprintf(":RoleName%v", i)) + queryArgs[fmt.Sprintf("RoleName%v", i)] = roleId } inQuery := strings.Join(inQueryList, ", ") @@ -248,7 +248,7 @@ func (s *SqlSupplier) SchemeDelete(ctx context.Context, schemeId string, hints . queryArgs["UpdateAt"] = time queryArgs["DeleteAt"] = time - if _, err := s.GetMaster().Exec("UPDATE Roles SET UpdateAt = :UpdateAt, DeleteAt = :DeleteAt WHERE Id IN ("+inQuery+")", queryArgs); err != nil { + if _, err := s.GetMaster().Exec("UPDATE Roles SET UpdateAt = :UpdateAt, DeleteAt = :DeleteAt WHERE Name IN ("+inQuery+")", queryArgs); err != nil { result.Err = model.NewAppError("SqlSchemeStore.Delete", "store.sql_scheme.delete.role_update.app_error", nil, "Id="+schemeId+", "+err.Error(), http.StatusInternalServerError) return result } -- cgit v1.2.3-1-g7c22