Project

General

Profile

Bug #6573

Satellite6 should error when the user attempts to sort on a non-sortable field

Added by Adam Price about 5 years ago. Updated about 1 year ago.

Status:
Assigned
Priority:
Normal
Assignee:
Category:
API
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1110431
Description of problem:

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

How reproducible:

Steps to Reproduce:

  1. hammer activation-key list --organization-id=1 --by='name'
    ---|--------------|----------------|-----------------------|-------------
    ID | NAME | CONSUMED | LIFECYCLE ENVIRONMENT | CONTENT VIEW
    ---|--------------|----------------|-----------------------|-------------
    8 | ak1234 | 0 of Unlimited | | cv1
    12 | ak12345678 | 0 of Unlimited | | cv1
    5 | actkey | 0 of Unlimited | DEV | cv1
    1 | actkey1 | 1 of Unlimited | DEV | cv1
    4 | test1 | 0 of Unlimited | |
    7 | ak123 | 0 of Unlimited | | cv1
    11 | ak1234567 | 0 of Unlimited | | cv1
    14 | ak1234567890 | 0 of Unlimited | | cv1
    6 | ak | 0 of Unlimited | | cv1
    9 | ak12345 | 0 of Unlimited | Library | cv1
    10 | ak123456 | 0 of Unlimited | | cv1
    13 | ak123456789 | 0 of 100 | | cv1
    ---|--------------|----------------|-----------------------|-------------
  1. hammer activation-key list --organization-id=1 --by='contentview'
    ---|--------------|----------------|-----------------------|-------------
    ID | NAME | CONSUMED | LIFECYCLE ENVIRONMENT | CONTENT VIEW
    ---|--------------|----------------|-----------------------|-------------
    8 | ak1234 | 0 of Unlimited | | cv1
    12 | ak12345678 | 0 of Unlimited | | cv1
    5 | actkey | 0 of Unlimited | DEV | cv1
    1 | actkey1 | 1 of Unlimited | DEV | cv1
    4 | test1 | 0 of Unlimited | |
    7 | ak123 | 0 of Unlimited | | cv1
    11 | ak1234567 | 0 of Unlimited | | cv1
    14 | ak1234567890 | 0 of Unlimited | | cv1
    6 | ak | 0 of Unlimited | | cv1
    9 | ak12345 | 0 of Unlimited | Library | cv1
    10 | ak123456 | 0 of Unlimited | | cv1
    13 | ak123456789 | 0 of 100 | | cv1
    ---|--------------|----------------|-----------------------|-------------

Actual results:
Note that the above two results does not change

Expected results:
The --by attribute should sort the results on the given field

Additional info:
I also tried the following commands and the result was same:

#hammer activation-key list --organization-id=1 --by='NAME'
#hammer activation-key list --organization-id=1 --by='CONTENT VIEW'

Associated revisions

Revision 3ec51c07 (diff)
Added by Adam Price almost 5 years ago

fixes #6573 - foretello should error when sorting by non-sortable field, BZ 1110431

Revision 6e867e16 (diff)
Added by Adam Price almost 5 years ago

fixes #6573 - foretello should error when sorting by non-sortable field, BZ 1110431

Revision 07075dc8
Added by Adam Price almost 5 years ago

Merge pull request #4420 from komidore64/rmi6573-error-on-non-sort-field

fixes #6573 - need better errors when sorting by a non-sortable field

Revision 5edddb73
Added by Adam Price almost 5 years ago

Merge pull request #4420 from komidore64/rmi6573-error-on-non-sort-field

fixes #6573 - need better errors when sorting by a non-sortable field

History

#1 Updated by The Foreman Bot about 5 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 49
  • Pull request https://github.com/Katello/katello/pull/4420 added
  • Pull request deleted ()

#2 Updated by Eric Helms about 5 years ago

  • Category set to API
  • Difficulty set to easy
  • Triaged changed from No to Yes

#3 Updated by Eric Helms almost 5 years ago

  • Target version changed from 49 to 54

#4 Updated by Anonymous almost 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Adam Price almost 5 years ago

  • Status changed from Closed to Assigned

#6 Updated by Eric Helms almost 5 years ago

  • Target version deleted (54)

#7 Updated by Eric Helms over 4 years ago

  • Status changed from Assigned to Closed
  • Legacy Backlogs Release (now unused) set to 14

#8 Updated by John Mitsch over 2 years ago

  • Status changed from Closed to Assigned
  • Legacy Backlogs Release (now unused) changed from 14 to 114
  • Pull request deleted (https://github.com/Katello/katello/pull/4420)

This commit was reverted upstream, so this is still an issue, changing to assigned.

Also available in: Atom PDF