Project

General

Profile

Bug #6766

hammer activation-key pagination failed

Added by Martin Bacovsky over 8 years ago. Updated over 4 years ago.

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

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 (diff)
Added by Martin Bačovský over 8 years ago

Fixes #6766 - Pagination is failing

Revision a050b05f
Added by mbacovsky over 8 years ago

Merge pull request #138 from mbacovsky/6766_failing_pagination

Fixes #6766 - Pagination is failing

History

#1 Updated by The Foreman Bot over 8 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
  • Pull request deleted ()

#2 Updated by Anonymous over 8 years ago

  • Target version changed from 1.8.0 to 1.7.5

#3 Updated by Anonymous over 8 years ago

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

Also available in: Atom PDF