Support NIL values in searchable options
|Assigned To:||Martin Bacovsky|
|Target version:||Team Ivan - Iteration 25|
|Found in release:||Pull request:||https://github.com/theforeman/hammer-cli-foreman/pull/346, https://github.com/theforeman/hammer-cli/pull/264|
|Velocity based estimate||-|
Currently setting the NIL (unset) values is supported only for common options. The searchable options won't resolve properly:
hammer host update --id 83 --domain NIL Could not update the host: Error: domain not found
Requested result is the domain_id to be unset
Fixes #21768 - Allow nil searchables
Adds ability to correctly resolve name params set to NIL.
For better coexistence with hammer defaults the logic
was moved to option sources which is now preferred place
for manipulation with options. As a result when options
or all_options is called anywhere in the code it should
return fully resolved options in final state.
#6 Updated by Martin Bacovsky 3 months ago
- % Done changed from 0 to 100
- Status changed from Ready For Testing to Closed
Applied in changeset hammer-cli-foreman|c61d1365f1bbb7ab8d3a54ed14eeb10a6100b0eb.