Bug #6766

hammer activation-key pagination failed

Added by Martin Bacovsky almost 4 years ago. Updated 6 days ago.

Status:Closed
Priority:Normal
Assignee:Martin Bacovsky
Category:-
Target version:1.7.5
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1109934 Found in Releases:
Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/138

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1109934
Description of problem:
hammer activation-key pagination failed

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

How reproducible:

Steps to Reproduce:
1. Create 10 different activation keys under ACME_Corporation

  1. hammer activation-key list --organization-label='ACME_Corporation' --page=1 --per-page=2
    ---|--------|----------------|-----------------------|-------------
    ID | NAME | CONSUMED | LIFECYCLE ENVIRONMENT | CONTENT VIEW
    ---|--------|----------------|-----------------------|-------------
    5 | actkey | 0 of Unlimited | |
    8 | ak1234 | 0 of Unlimited | | cv1
    ---|--------|----------------|-----------------------|-------------
    List next page? (Y/n): Y
    Error: can't convert Fixnum into String
  1. hammer activation-key list --organization-label='ACME_Corporation' --page='1' --per-page='2'
    ---|--------|----------------|-----------------------|-------------
    ID | NAME | CONSUMED | LIFECYCLE ENVIRONMENT | CONTENT VIEW
    ---|--------|----------------|-----------------------|-------------
    5 | actkey | 0 of Unlimited | |
    8 | ak1234 | 0 of Unlimited | | cv1
    ---|--------|----------------|-----------------------|-------------
    List next page? (Y/n): y
    Error: can't convert Fixnum into String

Actual results:
Pagination failed

Expected results:
Pagination should work as expected

Additional info:

Associated revisions

Revision 3f9c4266
Added by Martin Bačovský almost 4 years ago

Fixes #6766 - Pagination is failing

Revision a050b05f
Added by mbacovsky almost 4 years ago

Merge pull request #138 from mbacovsky/6766_failing_pagination

Fixes #6766 - Pagination is failing

History

#1 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 1.8.0
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/138 added

#2 Updated by Dmitri Dolguikh almost 4 years ago

  • Target version changed from 1.8.0 to 1.7.5

#3 Updated by Anonymous almost 4 years ago

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

Also available in: Atom PDF