Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-18128

Filtering build queue/executors does not work in nested view

      The checkboxes "Filter build queue" and "Filter build executors" do not really make sense at the moment in a nested view. As a nested view contains no jobs at all, the checkboxes simply completely remove the build queue/executor list.

      Suggestions:

      • Remove the checkboxes from the "Edit View" page for nested views
      • Or make them work by recursively looking for the jobs inside the nested view

          [JENKINS-18128] Filtering build queue/executors does not work in nested view

          There is another problem on filtering.

          Yoichi Nakayama added a comment - There is another problem on filtering.

          At first, I've tried to solve this by the change of NestedView class:
          https://github.com/jenkinsci/nested-view-plugin/pull/16
          But the approach requires duplicate code and increase maintenance cost of the Nested View plugin.
          I noticed that it is better to extend the filter logic defined in View class to respect ViewGroup.

          Yoichi Nakayama added a comment - At first, I've tried to solve this by the change of NestedView class: https://github.com/jenkinsci/nested-view-plugin/pull/16 But the approach requires duplicate code and increase maintenance cost of the Nested View plugin. I noticed that it is better to extend the filter logic defined in View class to respect ViewGroup.

          I've sent a pull request
          https://github.com/jenkinsci/jenkins/pull/1030
          It also fixes similar problem with Folder under ListView

          Yoichi Nakayama added a comment - I've sent a pull request https://github.com/jenkinsci/jenkins/pull/1030 It also fixes similar problem with Folder under ListView

          Jesse Glick added a comment -

          After fixing JENKINS-20500 I see no problems with folders.

          Jesse Glick added a comment - After fixing JENKINS-20500 I see no problems with folders.

            yoichi Yoichi Nakayama
            marc_guenther Marc Günther
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: