Project

General

Custom queries

Profile

Actions

Bug #15257

closed

Default per_page configuration enables pagination, including in CSV mode

Added by Dominic Cleal almost 9 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Normal
Category:
Foreman commands (obsolete)
Target version:
-
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Pagination was disabled in both table and CSV formats by default since #10534 (in 0.3.0), but #14530 in master changed hammer_cli_foreman to paginate according to the config file's per_page value if set.

This value is set in the default hammer_cli config file (https://github.com/theforeman/hammer-cli/blob/0.6.1/config/cli_config.template.yml), so pagination is now enabled by default, even in CSV mode.

I think the configured setting should probably be ignored in CSV output mode (it is described as a 'ui' setting too), or remove the default per_page value of 20.


Related issues 3 (0 open3 closed)

Related to Hammer CLI - Bug #14530: hammer queries defaults to 1000 items per pageClosedMartin Bacovsky04/07/2016Actions
Related to Katello - Bug #15542: SAT6.1 Pagination control not supported for listing content hosts in host collection with Satellite 6 command-lineRejected06/29/2016Actions
Has duplicate Katello - Bug #15274: [Sat6] hammer content-view filter rule list shows only 20 entriesDuplicate06/02/2016Actions
#1

Updated by Dominic Cleal almost 9 years ago

  • Related to Bug #14530: hammer queries defaults to 1000 items per page added
#3

Updated by Martin Bacovsky almost 9 years ago

  • Status changed from New to Assigned
  • Assignee set to Martin Bacovsky
#5

Updated by Martin Bacovsky almost 9 years ago

  • Assignee changed from Martin Bacovsky to Tomáš Strachota
#6

Updated by The Foreman Bot almost 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli/pull/208 added
#7

Updated by The Foreman Bot almost 9 years ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/243 added
#8

Updated by Anonymous almost 9 years ago

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

Updated by Andrew Kofink almost 9 years ago

  • Has duplicate Bug #15274: [Sat6] hammer content-view filter rule list shows only 20 entries added
#10

Updated by Andrew Kofink almost 9 years ago

  • Related to Bug #15542: SAT6.1 Pagination control not supported for listing content hosts in host collection with Satellite 6 command-line added
Actions

Also available in: Atom PDF