Bug 1722491 - Sorting tables doesn't work in service request page
Summary: Sorting tables doesn't work in service request page
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Reporting
Version: 5.10.6
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.11.0
Assignee: Keenan Brock
QA Contact: Niyaz Akhtar Ansari
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1728033
TreeView+ depends on / blocked
 
Reported: 2019-06-20 13:19 UTC by Niyaz Akhtar Ansari
Modified: 2019-12-13 15:00 UTC (History)
9 users (show)

Fixed In Version: 5.11.0.14
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1728033 (view as bug list)
Environment:
Last Closed: 2019-12-13 15:00:50 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
service apper at bottom of the table (201.59 KB, image/png)
2019-06-20 13:19 UTC, Niyaz Akhtar Ansari
no flags Details

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(-)


Note You need to log in before you can comment on or make changes to this bug.