Bug 1722491

Summary: Sorting tables doesn't work in service request page
Product: Red Hat CloudForms Management Engine Reporter: Niyaz Akhtar Ansari <nansari>
Component: ReportingAssignee: Keenan Brock <kbrock>
Status: CLOSED CURRENTRELEASE QA Contact: Niyaz Akhtar Ansari <nansari>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.10.6CC: bmidwood, dmetzger, hkataria, lavenel, mpovolny, mshriver, obarenbo, simaishi, yrudman
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.11.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.11.0.14 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1728033 (view as bug list) Environment:
Last Closed: 2019-12-13 15:00:50 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1728033    
Attachments:
Description Flags
service apper at bottom of the table none

Description Niyaz Akhtar Ansari 2019-06-20 13:19:06 UTC
Created attachment 1582675 [details]
service apper at bottom of the table

Description of problem:
When I provision the service, latest services appear at bottom of the table and sorting tables doesn't work as expected  

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

How reproducible:
100%

Steps to Reproduce:
1.Create catalog item
2.Order the catalog item
3.In service request page 

Actual results:
Service request footer filter not working properly 

Expected results:
When I provision the service, Latest services should appear at top of the table

Additional info:

Comment 8 Keenan Brock 2019-07-02 15:31:30 UTC
Looks like this bug is across all screens for 5.11.0
It is due to the way that the inner select logic was written.

This is also a regression in 5.10.6 /via https://github.com/ManageIQ/manageiq-ui-classic/pull/5366

Comment 11 CFME Bot 2019-07-08 20:12:08 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/a72c9b458ae5afa2123f6ca824d345815ad6e689
commit a72c9b458ae5afa2123f6ca824d345815ad6e689
Author:     Keenan Brock <keenan>
AuthorDate: Tue Jul  2 11:40:54 2019 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Tue Jul  2 11:40:54 2019 -0400

    Revert "Merge pull request #5366 from kbrock/faster_services"

    This reverts commit 3f609dab512cf49350cb10545d2a80ca9b343a0d, reversing
    changes made to 2f31e65a702754ef962432a3f1e10bd4ff8b1b55.

    Fixes:  https://bugzilla.redhat.com/show_bug.cgi?id=1722491

 product/views/Service.yaml | 3 -
 1 file changed, 3 deletions(-)

Comment 13 Niyaz Akhtar Ansari 2019-07-15 12:12:50 UTC
Verified in Version 5.11.0.14.20190710225033_cb17ff0

Comment 14 CFME Bot 2019-07-22 15:31:23 UTC
New commit detected on ManageIQ/manageiq/master:

https://github.com/ManageIQ/manageiq/commit/6d5a1613c2b63b326e5196140a979eb1ad5bf777
commit 6d5a1613c2b63b326e5196140a979eb1ad5bf777
Author:     Keenan Brock <keenan>
AuthorDate: Mon Jul  1 20:11:21 2019 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Mon Jul  1 20:11:21 2019 -0400

    Fix turbo order

    Surprisingly, this is not ordered:

    select * from (
      select *
      from vms
      order by id
      limit 10
    )

    Instead, we needed the order also in both clauses:

    select * from (
      select *
      from vms
      order by id
      limit 10
    )
    order by id

    https://bugzilla.redhat.com/show_bug.cgi?id=1722491
 lib/rbac/filterer.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)