Bug 1291031

Summary: "Select All" button on Satellite 5.7 Web UI ignores filters and selects all packages
Product: Red Hat Satellite 5 Reporter: Peter <yochen>
Component: WebUIAssignee: Grant Gainey <ggainey>
Status: CLOSED ERRATA QA Contact: Ales Dujicek <adujicek>
Severity: high Docs Contact:
Priority: high    
Version: 570CC: adujicek, dyordano, igreen, jko, tlestach, xdmoon
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: spacewalk-java-2.3.8-140-sat Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-06-02 12:33:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 465198, 1127217, 1162131    

Description Peter 2015-12-13 00:50:59 UTC
Description of problem:
the issue of "Select All" button on the satellite 5.7 Web UI


Version-Release number of selected component (if applicable):
Satellite 5.7

How reproducible:
Two scenes to reproduce this issue:
(1) scene one: Erratas page
During generating own Erratas for infrastructure Packages, we searched for Packages. Example:
Looking for packages which ends with 6-6.6.0.50. The satellite found three packages. And now we have the choice to add every package separately, click the check button above (add all listed packages) or choose the button "Select All". If I choose "Select All", the Satellite starts to collect all packages available in Satellite!
(2) scene two: channel and systems pages
a) A channel admin tried to delete _some_ packages from a custom channel. After filtering the channel, if the user clicked "Select All" button and then "Delete" button, it will end up deleting _all_ packages from the channel.
b) A system admin tried to delete _some_ hosts from the satellite. After filter the hosts, if the user clicked "Select All" button and then "Delete" button, it will end up deleting _all_ hosts from the satellite.

Steps to Reproduce:
https://rhel6-sat57-2.gsslab.rdu2.redhat.com/rhn/Login.do
Errata -> Manage Errata -> Unpublished Errata
1. go to Erratas page
2. input some key words to filter, then click search button (eye button)
3. click "Select All" button
4. click "Delete Errata"
5. A new page display, it will show all errata

Actual results:
After clicking "Delete Errata", a new page display, it will show all erratas to delete 

Expected results:
It will display the filtered errata to delete


Additional info:
Case ID: 01547357

In addition, I found the link "Download CSV" has the same issue. Take users management page for example, although I have done some filter on the page, when I click "Download CSV", it will download all the users in the csv file.

Comment 1 Tomas Lestach 2015-12-14 10:02:33 UTC
This looks like a duplicate of Bug 962407

Comment 2 Peter 2015-12-15 23:05:02 UTC
(In reply to Tomas Lestach from comment #1)
> This looks like a duplicate of Bug 962407

Hi Tomas,

Thanks. You are right, it is the same bug, I guess.

But that bug was opened in 2013 and has not been fixed.

Comment 7 Tomas Lestach 2016-02-29 11:41:15 UTC
*** Bug 962407 has been marked as a duplicate of this bug. ***

Comment 10 Grant Gainey 2016-03-28 20:45:13 UTC
spacewalk.github:
083d7b6e4c3416372d7c5f44a1cb5d8b2a4d7c4f

This commit adds the requested SelectAll behavior to the New ListTag infrastructure and all pages that take advantage of it. Unfortunately, there are still ~60 JSPs that rely on the Old ListTag, and will require extensive refactoring to move them to the new-improved scheme of things.

This fix addresses the New ListTag-affected pages. More work remains to be done.

Comment 11 Grant Gainey 2016-04-27 19:00:05 UTC
The manage-errata pages specifically mentioned in this BZ migrated to NewListTag to be able to pick up the corrected behavior:

spacewalk.github:
a89aa0ec428e6a115605cb1d919c92153b6b842a
6b3c59a620bddae6bec2b482a1e637e8e652ba95

Comment 23 Grant Gainey 2016-05-18 20:36:26 UTC
spacewalk.github:
852d951c995e0e7c71183e4446dc40871abddd59

Comment 29 errata-xmlrpc 2016-06-02 12:33:26 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:1200

Comment 30 Grant Gainey 2017-02-02 15:10:31 UTC
NOTE: this fix is an invasive change to some significant infrastructure and pages. Backporting to previous releases is *not* trivial.