BIND 10 #2730: "config remove" should allow the same identifier as printed with "config show"

[BIND 10 Development]

#2730: "config remove" should allow the same identifier as printed with "config
show"
-------------------------------------+-------------------------------------
            Reporter:  cas           |                         Type:
              Status:  new           |  defect
           Milestone:  New Tasks     |                     Priority:
            Keywords:                |  medium
           Sensitive:  0             |                    Component:
         Sub-Project:  Core          |  Unclassified
Estimated Difficulty:  0             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
 The identifier for removing configuration items is different from the
 representation in "config show".

 The identifier for a list is show as "data_sources/classes/CH", the
 identifier needed for "config remove" is "data_sources/classes CH". That
 is confusing, and it prevents the use of copy and paste.

 Example (real world):
 {{{
 > config show all
 data_sources/classes/CH []      list
 data_sources/classes/IN []      list
 Logging/loggers[0]/name "*"     string
 Logging/loggers[0]/severity     "DEBUG" string
 Logging/loggers[0]/debuglevel   0       integer (default)
 Logging/loggers[0]/additive     false   boolean (default)
 Logging/loggers[0]/output_options[0]/destination        "file"  string
 Logging/loggers[0]/output_options[0]/output     "/var/log/bind10.log"
 string
 Logging/loggers[0]/output_options[0]/flush      false   boolean
 Logging/loggers[0]/output_options[0]/maxsize    204800  integer
 Logging/loggers[0]/output_options[0]/maxver     8       integer
 Cmdctl/key_file "/usr/local/etc/bind10/cmdctl-keyfile.pem"      string
 (default)
 Cmdctl/cert_file        "/usr/local/etc/bind10/cmdctl-certfile.pem"
 string  (default)
 Cmdctl/accounts_file    "/usr/local/etc/bind10/cmdctl-accounts.csv"
 string  (default)
 tsig_keys/keys  []      list    (default)
 Boss/components/b10-stats/special       null    string
 Boss/components/b10-stats/process       null    string
 Boss/components/b10-stats/kind  "dispensable"   string
 Boss/components/b10-stats/address       "Stats" string
 Boss/components/b10-stats/params        []      list
 Boss/components/b10-stats/priority      null    integer
 Boss/components/b10-cmdctl/special      "cmdctl"        string
 Boss/components/b10-cmdctl/process      null    string
 Boss/components/b10-cmdctl/kind "needed"        string
 Boss/components/b10-cmdctl/address      null    string
 Boss/components/b10-cmdctl/params       []      list
 Boss/components/b10-cmdctl/priority     null    integer
 Stats/poll-interval     60      integer (default)
 > config remove data_sources/classes/CH
 Error!  identifier in remove_value() does not contain a list index, and no
 value to remove
 > config remove data_sources/classes
 Error!  identifier in remove_value() does not contain a list index, and no
 value to remove
 > config remove data_sources
 Error!  identifier in remove_value() does not contain a list index, and no
 value to remove
 > config remove data_sources/classes CH
 > config commit
 }}}

-- 
Ticket URL: <http://bind10.isc.org/ticket/2730>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development