tools, codec-json: Initial whitespace is also valid json. Also fixed

reading for cli manip as well

1 files changed, 73 insertions, 13 deletions

diff --git a/modules/tools/c++/cli_analyzer.hpp b/modules/tools/c++/cli_analyzer.hpp
index 4aa3d6c..9a0998b 100644
--- a/modules/tools/c++/cli_analyzer.hpp
+++ b/modules/tools/c++/cli_analyzer.hpp
@@ -6,6 +6,7 @@
 #include <forstio/codec/json/json.hpp>
 
 #include <deque>
+#include <fstream>
 #include <iostream>
 
 namespace saw {
@@ -42,8 +43,6 @@ struct cli_modifier {
 			}
 
 			json_data_str = convert_to_string(json_data.get_buffer());
-
-			std::cout<<json_data_str<<std::endl;
 		}
 
 		return void_t{};
@@ -66,6 +65,28 @@ struct cli_modifier {
 	}
 };
 
+template<typename Encoding>
+struct cli_traverser<schema::String, Encoding> {
+	using Schema = schema::String;
+
+	template<typename Traversal>
+	static error_or<void> traverse(std::deque<std::string>& sch_path, data<Schema,Encoding>& enc_data, std::string& json_data){
+		if(not sch_path.empty()){
+			return make_error<err::invalid_state>("Schema path too long");
+		}
+
+		cli_modifier<Schema, Encoding> mod;
+		if constexpr (std::is_same_v<Traversal, cli_mode::read>){
+			return mod.read(sch_path, enc_data, json_data);
+		} else if constexpr (std::is_same_v<Traversal, cli_mode::write>) {
+			return mod.write(sch_path, enc_data, json_data);
+		} else {
+			return make_error<err::invalid_state>("We only support cli_mode::read and cli_mode::write");
+		}
+	}
+
+};
+
 template<typename... T, typename Encoding>
 struct cli_traverser<schema::Tuple<T...>, Encoding> {
 		using Schema = schema::Tuple<T...>;
@@ -146,27 +167,66 @@ struct parsed_args {
 };
 
 template<typename Schema, typename Encoding>
-int modify_data_on_cli(bool read_mode, const std::string_view& file_path, std::deque<std::string> sch_path, std::string& json_data){
-    /**
+error_or<void> modify_data_on_cli(bool read_mode, const std::string_view& file_path, std::deque<std::string> sch_path, std::string& json_data){
+  /**
 	 * Read data from file
 	 */
-
+	std::string file_data = [&]() -> std::string {
+		std::fstream fstr{std::string{file_path}, fstr.binary | fstr.in};
+		if(!fstr.is_open()){
+			return {};
+		}
+		std::stringstream sstr;
+		sstr << fstr.rdbuf();
+		return sstr.str();
+	}();
+	if(file_data.empty()){
+		return make_error<err::not_found>("File exists, but is empty.");
+	}
+	data<Schema, Encoding> enc_data{std::string_view{file_data}};
+	codec<Schema, Encoding> enc_codec;
 	
-	data<Schema, Encoding> enc_data;
 	data<Schema, encode::Native> native_data;
+	{
+		auto eov = enc_codec.template decode<encode::Native>(enc_data, native_data);
+		if(eov.is_error()){
+			return eov;
+		}
+	}
 
 	if (read_mode) {
-		auto eov = impl::cli_traverser<Schema, encode::Native>::template traverse<impl::cli_mode::read>(sch_path, native_data, json_data);
-		if(eov.is_error()){
-			return -1;
+		{
+			auto eov = impl::cli_traverser<Schema, encode::Native>::template traverse<impl::cli_mode::read>(sch_path, native_data, json_data);
+			if(eov.is_error()){
+				return eov;
+			}
 		}
 	} else {
-		auto eov = impl::cli_traverser<Schema, encode::Native>::template traverse<impl::cli_mode::write>(sch_path, native_data, json_data);
-		if(eov.is_error()){
-			return -1;
+		{
+			auto eov = impl::cli_traverser<Schema, encode::Native>::template traverse<impl::cli_mode::write>(sch_path, native_data, json_data);
+			if(eov.is_error()){
+				return eov;
+			}
+		}
+		{
+			auto eov = enc_codec.template encode<encode::Native>(native_data, enc_data);
+			if(eov.is_error()){
+				return eov;
+			}
+		}
+		{
+			std::fstream fstr{std::string{file_path}, fstr.binary | fstr.out | fstr.trunc };
+			if(!fstr.is_open()){
+				return make_error<err::invalid_state>("Couldn't open file");
+			}
+			std::stringstream sstr;
+			std::string enc_str = convert_to_string(enc_data.get_buffer());
+
+			fstr << enc_str;
+			fstr.close();
 		}
 	}
 
-	return 0;
+	return void_t{};
 }
 }