MM-27407 Return archived channels even if view archived config is false for sysadmin endpoints (#15149)
Co-authored-by: Mattermod <mattermod@users.noreply.github.com>
Этот коммит содержится в:
коммит произвёл
GitHub
родитель
339c5bae76
Коммит
d4dac31b04
@@ -969,15 +969,6 @@ func TestGetAllChannels(t *testing.T) {
|
|||||||
defer th.TearDown()
|
defer th.TearDown()
|
||||||
Client := th.Client
|
Client := th.Client
|
||||||
|
|
||||||
var originalConfigVal bool
|
|
||||||
th.App.UpdateConfig(func(cfg *model.Config) {
|
|
||||||
originalConfigVal = *cfg.TeamSettings.ExperimentalViewArchivedChannels
|
|
||||||
*cfg.TeamSettings.ExperimentalViewArchivedChannels = true
|
|
||||||
})
|
|
||||||
defer th.App.UpdateConfig(func(cfg *model.Config) {
|
|
||||||
*cfg.TeamSettings.ExperimentalViewArchivedChannels = originalConfigVal
|
|
||||||
})
|
|
||||||
|
|
||||||
th.TestForSystemAdminAndLocal(t, func(t *testing.T, client *model.Client4) {
|
th.TestForSystemAdminAndLocal(t, func(t *testing.T, client *model.Client4) {
|
||||||
channels, resp := client.GetAllChannels(0, 20, "")
|
channels, resp := client.GetAllChannels(0, 20, "")
|
||||||
CheckNoError(t, resp)
|
CheckNoError(t, resp)
|
||||||
@@ -1265,10 +1256,6 @@ func TestSearchAllChannels(t *testing.T) {
|
|||||||
})
|
})
|
||||||
CheckNoError(t, groupErr)
|
CheckNoError(t, groupErr)
|
||||||
|
|
||||||
th.App.UpdateConfig(func(cfg *model.Config) {
|
|
||||||
*cfg.TeamSettings.ExperimentalViewArchivedChannels = true
|
|
||||||
})
|
|
||||||
|
|
||||||
testCases := []struct {
|
testCases := []struct {
|
||||||
Description string
|
Description string
|
||||||
Search *model.ChannelSearch
|
Search *model.ChannelSearch
|
||||||
|
|||||||
@@ -1570,7 +1570,6 @@ func (a *App) GetChannelsForUser(teamId string, userId string, includeDeleted bo
|
|||||||
}
|
}
|
||||||
|
|
||||||
func (a *App) GetAllChannels(page, perPage int, opts model.ChannelSearchOpts) (*model.ChannelListWithTeamData, *model.AppError) {
|
func (a *App) GetAllChannels(page, perPage int, opts model.ChannelSearchOpts) (*model.ChannelListWithTeamData, *model.AppError) {
|
||||||
opts.IncludeDeleted = *a.Config().TeamSettings.ExperimentalViewArchivedChannels && opts.IncludeDeleted
|
|
||||||
if opts.ExcludeDefaultChannels {
|
if opts.ExcludeDefaultChannels {
|
||||||
opts.ExcludeChannelNames = a.DefaultChannelNames()
|
opts.ExcludeChannelNames = a.DefaultChannelNames()
|
||||||
}
|
}
|
||||||
@@ -2187,7 +2186,6 @@ func (a *App) AutocompleteChannelsForSearch(teamId string, userId string, term s
|
|||||||
|
|
||||||
// SearchAllChannels returns a list of channels, the total count of the results of the search (if the paginate search option is true), and an error.
|
// SearchAllChannels returns a list of channels, the total count of the results of the search (if the paginate search option is true), and an error.
|
||||||
func (a *App) SearchAllChannels(term string, opts model.ChannelSearchOpts) (*model.ChannelListWithTeamData, int64, *model.AppError) {
|
func (a *App) SearchAllChannels(term string, opts model.ChannelSearchOpts) (*model.ChannelListWithTeamData, int64, *model.AppError) {
|
||||||
opts.IncludeDeleted = *a.Config().TeamSettings.ExperimentalViewArchivedChannels && opts.IncludeDeleted
|
|
||||||
if opts.ExcludeDefaultChannels {
|
if opts.ExcludeDefaultChannels {
|
||||||
opts.ExcludeChannelNames = a.DefaultChannelNames()
|
opts.ExcludeChannelNames = a.DefaultChannelNames()
|
||||||
}
|
}
|
||||||
|
|||||||
Ссылка в новой задаче
Block a user