From 68de49481084c5866a07ec642d67473732ddd6ee Mon Sep 17 00:00:00 2001 From: Carlos Martins Date: Mon, 29 May 2023 12:31:40 -0600 Subject: [PATCH 1/3] include thread options in search --- src/libs/OptionsListUtils.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libs/OptionsListUtils.js b/src/libs/OptionsListUtils.js index 27a70e59ee5e..a340f38fd8a9 100644 --- a/src/libs/OptionsListUtils.js +++ b/src/libs/OptionsListUtils.js @@ -673,7 +673,7 @@ function getOptions( } // Check the report to see if it has a single participant and if the participant is already selected - if (reportOption.login && _.some(loginOptionsToExclude, (option) => option.login === reportOption.login)) { + if (!includeThreads && reportOption.login && _.some(loginOptionsToExclude, (option) => option.login === reportOption.login)) { continue; } From f53576ab6e9266e9652ba3b347379bb5d4526c14 Mon Sep 17 00:00:00 2001 From: Carlos Martins Date: Mon, 29 May 2023 12:48:37 -0600 Subject: [PATCH 2/3] fix style --- src/libs/OptionsListUtils.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libs/OptionsListUtils.js b/src/libs/OptionsListUtils.js index a340f38fd8a9..61d0cbf8634b 100644 --- a/src/libs/OptionsListUtils.js +++ b/src/libs/OptionsListUtils.js @@ -673,7 +673,7 @@ function getOptions( } // Check the report to see if it has a single participant and if the participant is already selected - if (!includeThreads && reportOption.login && _.some(loginOptionsToExclude, (option) => option.login === reportOption.login)) { + if (!includeThreads && reportOption.login && _.some(loginOptionsToExclude, (option) => option.login === reportOption.login)) { continue; } From 075a4d8e13e2586ab5315b2c13715421b02940f9 Mon Sep 17 00:00:00 2001 From: Carlos Martins Date: Mon, 29 May 2023 13:43:36 -0600 Subject: [PATCH 3/3] update comment --- src/libs/OptionsListUtils.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libs/OptionsListUtils.js b/src/libs/OptionsListUtils.js index 61d0cbf8634b..93465cfceee4 100644 --- a/src/libs/OptionsListUtils.js +++ b/src/libs/OptionsListUtils.js @@ -672,7 +672,7 @@ function getOptions( continue; } - // Check the report to see if it has a single participant and if the participant is already selected + // If we're excluding threads, check the report to see if it has a single participant and if the participant is already selected if (!includeThreads && reportOption.login && _.some(loginOptionsToExclude, (option) => option.login === reportOption.login)) { continue; }