Fixes a bug w/ (none) combined with another selection.

Of course, (none) can be selected in a list of several other options, in
which case we need the empty string fix, too.
pull/834/head
Martin Czuchra 11 years ago
parent 79f6e59fbd
commit 6eb091013b
  1. 7
      app/assets/javascripts/timelines/TimelineLoader.js

@ -753,9 +753,10 @@ Timeline.TimelineLoader = (function () {
// -1 and the empty string both need to be added in the
// (none)-case, since (none) has to both match work packages
// w/ custom fields w/ an empty value and work packages with
// no custom field.
if (value.length === 1 && value[0] == "-1") {
// w/ custom values that are empty and work packages w/o
// custom values.
if (value.indexOf("-1") !== -1) {
value.push("");
}

Loading…
Cancel
Save