Project

General

Profile

Bug #18352

`hammer content-view remove` has unnecessary option `--content-view-version-content-view-ids`

Added by Andrew Kofink over 2 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Content Views
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1416862
Description of problem:
The option `--content-view-version-content-view-ids` is not needed and can be specified with `--id` or `--name`.

Version-Release number of selected component (if applicable):
6.3.0
How reproducible:
Always
Steps to Reproduce:
1. `hammer content-view remove -h`
2. `hammer content-view remove --content-view-versions '1.2,3.0,4.6' --id 1`
3.
Actual results:
(1.) The option `--content-view-version-content-view-ids` is listed, and (2.) the following error is shown:
Could not remove objects from content view:
Error: Could not find content_view_version, please set one of options --content-view-versions, --content-view-version-content-view-ids.
Expected results:
(1.) The option `--content-view-version-content-view-ids` is not listed, and (2.) the content view versions are removed from the content view.
Additional info:

Associated revisions

Revision aa6e9c31 (diff)
Added by Andrew Kofink over 2 years ago

Fixes #18352 - Remove CV from CVV search (#475)

Remove content view from content view version search parameters in the
ID resolver. Fix content-view remove command to properly resolve content
view version numbers, and add tests around the main content-view remove
functionality.

History

#1 Updated by Andrew Kofink over 2 years ago

  • Subject changed from `hammer content-view remove` has unnecessary option `--content-view-version-content-view-ids` to `hammer content-view remove` has unnecessary option `--content-view-version-content-view-ids`
  • Target version set to 163

#2 Updated by Justin Sherrill over 2 years ago

  • Legacy Backlogs Release (now unused) set to 114

#3 Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Andrew Kofink
  • Legacy Backlogs Release (now unused) deleted (114)
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/475 added

#4 Updated by Justin Sherrill over 2 years ago

  • Legacy Backlogs Release (now unused) set to 114

#5 Updated by The Foreman Bot over 2 years ago

  • Legacy Backlogs Release (now unused) deleted (114)

#6 Updated by Anonymous over 2 years ago

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

#7 Updated by Eric Helms over 2 years ago

  • Legacy Backlogs Release (now unused) set to 211

Also available in: Atom PDF