summaryrefslogtreecommitdiff
path: root/src/cstore
diff options
context:
space:
mode:
authorJohn Southworth <john.southworth@vyatta.com>2011-07-29 20:38:14 -0700
committerJohn Southworth <john.southworth@vyatta.com>2011-07-29 20:38:14 -0700
commitc47de319ccd918caccfc1f3ee394e48f6d4e4dc5 (patch)
tree932154d6c16c5be0f566b2bb413157fd1328622b /src/cstore
parentc760e4a19eb29fbe0f88916f9b845c9081120238 (diff)
downloadvyatta-cfg-c47de319ccd918caccfc1f3ee394e48f6d4e4dc5.tar.gz
vyatta-cfg-c47de319ccd918caccfc1f3ee394e48f6d4e4dc5.zip
Bugfix 2823: Configuration mode command interpreter, output adjustments for consistency
Diffstat (limited to 'src/cstore')
-rw-r--r--src/cstore/cstore.cpp11
1 files changed, 7 insertions, 4 deletions
diff --git a/src/cstore/cstore.cpp b/src/cstore/cstore.cpp
index b8ab82f..5f2eb79 100644
--- a/src/cstore/cstore.cpp
+++ b/src/cstore/cstore.cpp
@@ -342,7 +342,8 @@ Cstore::validateSetPath(const Cpath& path_comps)
* single-value node without value. now all value nodes
* (single-value, multi-value, and tag) must be set with value.
*/
- output_user("The specified configuration node requires a value\n");
+ string output = "Configuration path: ["+path_comps.to_string()+"] requires a value\n";
+ output_user(output.c_str());
return false;
} else {
/* typeless node
@@ -443,7 +444,8 @@ Cstore::getEditEnv(const Cpath& path_comps, string& env)
* follow the original implementation and do a "set".
*/
if (!validateSetPath(path_comps)) {
- output_user("The specified config path is not valid\n");
+ string output = "Configuration path: ["+path_comps.to_string()+"] is not valid\n";
+ output_user(output.c_str());
return false;
}
if (!set_cfg_path(path_comps, false)) {
@@ -2056,7 +2058,7 @@ Cstore::get_parsed_tmpl(const Cpath& path_comps, bool validate_vals,
{
tr1::shared_ptr<Ctemplate> rtmpl;
// default error message
- error = "The specified configuration node is not valid";
+ error = "Configuration path: ["+path_comps.to_string()+"] is not valid\n";
bool do_caching = false;
if (tmpl_path_at_root()) {
@@ -2491,7 +2493,8 @@ Cstore::set_cfg_path(const Cpath& path_comps, bool output)
if (path_exists) {
// whole path already exists
if (output) {
- output_user("The specified configuration node already exists\n");
+ string userout = "Configuration path: ["+path_comps.to_string()+"] already exists\n";
+ output_user(userout.c_str());
}
// treat as success
}