diff options
author | Mohammed Naser <mnaser@vexxhost.com> | 2020-02-21 15:09:03 +0100 |
---|---|---|
committer | Mohammed Naser <mnaser@vexxhost.com> | 2020-02-21 15:11:53 +0100 |
commit | b226cffc5a83d0f202020ea64628a5f6614e24fa (patch) | |
tree | 2c9deb7cc8b708798be6cc8f35e6e9733b621cb9 | |
parent | ceb12831b18877172de316070b710ea6ac697c8b (diff) | |
download | zuul-b226cffc5a83d0f202020ea64628a5f6614e24fa.tar.gz |
ui: convert all spaces to commas for filter
We replace all spaces by commas when filtering the status page, however,
due to the fact we don't use the `g` flag, it only splits the first
value and stops after the first one.
This patch adds the `g` flag to the regex so that all of the spaces are
replaced by commas when filtering.
Change-Id: I9b9cd33a7067fd00415e2ae1634e340a6df46749
-rw-r--r-- | web/src/containers/jobs/Jobs.jsx | 2 | ||||
-rw-r--r-- | web/src/containers/status/Pipeline.jsx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/web/src/containers/jobs/Jobs.jsx b/web/src/containers/jobs/Jobs.jsx index b4835cd14..9bd54ca68 100644 --- a/web/src/containers/jobs/Jobs.jsx +++ b/web/src/containers/jobs/Jobs.jsx @@ -114,7 +114,7 @@ class JobsList extends React.Component { let filtered = false if (filter) { filtered = true - let filters = filter.replace(/ +/, ',').split(',') + let filters = filter.replace(/ +/g, ',').split(',') for (let job of jobs) { filters.forEach(jobFilter => { if (jobFilter && ( diff --git a/web/src/containers/status/Pipeline.jsx b/web/src/containers/status/Pipeline.jsx index b6ce6487a..865e961a9 100644 --- a/web/src/containers/status/Pipeline.jsx +++ b/web/src/containers/status/Pipeline.jsx @@ -90,7 +90,7 @@ class Pipeline extends React.Component { filterQueue(queue, filter) { let found = false - let filters = filter.replace(/ +/, ',').split(',') + let filters = filter.replace(/ +/g, ',').split(',') queue.heads.forEach(changes => { changes.forEach(change => { filters.forEach(changeFilter => { |