Bug 215622

Summary: Empty Data Sets not getting Rendered correctly with the New List Tag
Product: [Retired] Red Hat Network Reporter: Partha Aji <paji>
Component: RHN/Web SiteAssignee: Partha Aji <paji>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: medium Docs Contact:
Priority: medium    
Version: rhn500CC: duffy, rhn-bugs
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: https://rhn.webdev.redhat.com/rhn/systems/details/configuration/ViewModifySandboxPaths.do?sid=1006926064
Whiteboard:
Fixed In Version: sat500 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-06-26 03:08:46 UTC Type: ---
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: 237994    
Bug Blocks: 173427    
Attachments:
Description Flags
ss of problem none

Description Partha Aji 2006-11-14 21:30:06 UTC 
A couple of corrections that ought to be handled with the new list tag in the
case of an empty data set.  If you want to see how it appears right now go to

https://rhn.webdev.redhat.com/rhn/systems/details/configuration/ViewModifySandboxPaths.do?sid=1006926064

login: commandcenter/redhat

1) We want the Update List,Select All. Unselect All buttons not to appear.. 
2) We also do not want the pagination info like "1 - 0 of 0 (0 selected)".  

I tried to tinker with the SelectableDecorator without much luck.. I donot know
how to pass parameters/hints to the SelectableDecorator about the dataset from
the ListTag class...

If this too hard to do in the tag lib, then the only thing I can suggest is copy
pasting this block in every jsp

<rl:listset name="fileSet">

 <c:choose>
  <c:when test="${empty pageList}">
    <bean:message key="channelfiles.jsp.noFiles"/>
  </c:when>
  <c:otherwise>
    <rl:list dataset="..".
            .... >
        .......
    </rl:list>
  </c:otherwise>
 </c:choose>

</rl:listset>

I am not averse to this, but it does not seem to be a popular option and would
be nice to get it working with the taglib








Description of problem:


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


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Red Hat Bugzilla 2007-04-12 01:44:33 UTC 
User bnackash's account has been closed
Comment 2 Partha Aji 2007-04-20 21:39:45 UTC 
Should be resolved as with revision 115238
Comment 3 Partha Aji 2007-04-25 19:03:36 UTC 
Moving to ON_QA
Comment 4 wes hayutin 2007-05-02 12:37:02 UTC 
Partha is this ready for build 28?
I am getting ISE's after adding a config file in the sdc.
I am even getting the ISE just on listing the file now..

I logged into webdev.. I noticed the (x selected) was not working.
I'll probably want to chat w/ you to make sure I understand what the buttons
should look like too.
Comment 5 Partha Aji 2007-05-02 23:25:30 UTC 
This bug has been only resolved for satellite.. I 'll clone this for a hosted
bug....
Comment 6 wes hayutin 2007-05-03 13:22:03 UTC 
This will be available to test after the next iso.. due to a bug in the SDC
config mang sandbox
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238696
and
bug 237994
Comment 7 Partha Aji 2007-05-04 15:45:53 UTC 
Should be resolved in 5.0.0-29..
Comment 8 wes hayutin 2007-05-04 20:39:41 UTC 
Created attachment 154167 [details]
ss of problem

This still looks incorrect to me..
1. out of alignment
2. the # selected is wrong..
3. which buttons should be there.. I'm not 100% clear on this..
Comment 9 Partha Aji 2007-05-09 20:47:55 UTC 
1) Alignment has been corrected
2) The auto incrementing of the # selected has been fixed in the new list tag...
Any way in the case of an Empty List you should NOT see the page numbering ...
3) None of the Select All, Update List, Unselect All  should be present if its
an empty list..
If the list has no items selected, then Unselect all should not show up..
Comment 10 Partha Aji 2007-05-15 19:31:24 UTC 
Moving to ON_QA
Comment 11 Preethi Thomas 2007-05-16 13:51:59 UTC 
verified
Comment 12 Brandon Perkins 2007-06-26 03:08:46 UTC 
Closed for Satellite 500 Release.